eco-helpers 1.3.10 → 1.3.15

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 861fdc294bb8624a1cebd3a880a94ae5c0067adb8b61b7e4467afd6b52e0c87a
-  data.tar.gz: e3db0ea2abda25621738bbe3cb7e50ebae1b68e7daaf16344ae1a74aa200aa9c
+  metadata.gz: 1527493b64563ca009c0338bc5e3b5311371a4c59947526ad60fa453d577abfd
+  data.tar.gz: cabc02e6a45958ff8cfa46fc7ff76e451074d492f42a3091f28e6633c131b64b
 SHA512:
-  metadata.gz: d815d693a7208363238c96adb14250f6010c55b85158c236b157982754e73438f657e2d7639f4801a2530e1ba9952ec6496a6998b3a758a9728c056956fb4688
-  data.tar.gz: 10e1fdab93ce81884d01d474a82b41274c75bd168b52e1b525f53d127b15f8a9fa15759ac5b1b27de2b90997fe883628fec66dcffd443296f2abdb58d881065f
+  metadata.gz: 6c8eaa3f1a56daaee787c26dd5c2681aea73e689fa55636c946a46441e8b894adb63922584d69f292176ffe35f05da3878dd0a410e0fd3bbf0d906937bca774b
+  data.tar.gz: 345b6faa4aa18aaf0db1930eddcf45bf228a1ed2f6246efe7f839c86f8af396d81513209fa2a6e04e36734da28842b6b193d2269d7aee5be1b2c7be23e120eaf

data/CHANGELOG.md

@@ -1,6 +1,47 @@
 # Change Log
 All notable changes to this project will be documented in this file.
 
+## [1.3.15] - 2020-06-11
+
+### Added
+  - default usecase to export to `csv` (`-detailed`) now includes `"Supervisor Name"` column
+### Changed
+### Fixed
+
+## [1.3.14] - 2020-06-10
+
+### Added
+  - `Eco::API::Common::People::SupervisorHelpers` now has its methods as class methods
+### Changed
+  - upgraded `ecoportal-api` gem dependency to minimum version `0.5.6`
+### Fixed
+
+
+## [1.3.13] - 2020-05-29
+
+### Added
+### Changed
+### Fixed
+  - `Eco::API::Organization::TagTree#tag?` to accept `nil` by returning `false`
+  - `Eco::API::Common::People::DefaultParsers::DateParser` will parse to `Date` class
+    * it was parsing to `Time` class, while the native gem `ecoportal-api` parses as `Date` ([reference](https://gitlab.com/ecoPortal/ecoportal-api/-/blob/master/lib/ecoportal/api/v1/schema_field.rb))
+## [1.3.11] - 2020-05-19
+
+### Added
+### Changed
+  - stop using `email` as `external_id` on `People#person` & `People#find`
+    * this should result in more accurate searches when using `:strict` options
+### Fixed
+
+
+## [1.3.11] - 2020-05-12
+
+### Added
+### Changed
+  - remove popping up comments on `Eco::API::Organization::PolicyGroups#`
+### Fixed
+
+
 ## [1.3.10] - 2020-05-12
 
 ### Added

data/eco-helpers.gemspec

@@ -28,7 +28,7 @@ Gem::Specification.new do |s|
   s.add_development_dependency "yard",      "~> 0.9", ">= 0.9.18"
   s.add_development_dependency "redcarpet", "~> 3.5", ">= 3.5.0"
 
-  s.add_dependency 'ecoportal-api', '~> 0.5',  '>= 0.5.5'
+  s.add_dependency 'ecoportal-api', '~> 0.5',  '>= 0.5.6'
   s.add_dependency 'aws-sdk-s3',    '~> 1.30', '>= 1.30.1'
   s.add_dependency 'aws-sdk-ses',   '~> 1.14', '>= 1.14.0'
   s.add_dependency 'dotenv',        '~> 2.6',  '>= 2.6.0'

data/lib/eco/api/common/people/default_parsers/date_parser.rb

@@ -19,7 +19,7 @@ module Eco
 
             def parse_date(value)
               begin
-                (value.to_s.empty?) ? nil : Time.parse(value)
+                (value.to_s.empty?) ? nil : Date.parse(value)
               rescue
                 nil
               end

data/lib/eco/api/common/people/supervisor_helpers.rb

@@ -7,134 +7,144 @@ module Eco
         #   * `external_id`
         #   * `id`
         module SupervisorHelpers
+          def self.included(base)
+            base.send(:include, ClassMethods)
+          end
 
-          # Reorders as follows:
-          #   1. supervisors, people with no supervisor or where their supervisor not present
-          #   2. subordinates
-          def sort_by_supervisors(values, supervisors_first: true)
-            raise "Expected non hash Enumerable. Given: #{values.class}" if values.is_a?(Hash)
-            return [] unless values && values.is_a?(Enumerable)
-            roam = Proc.new do |tree|
-              [].tap do |out|
-                sub_outs = tree.empty?? [] : tree.map {|sup, subtree| roam.call(subtree)}
-                tree.each do |sup, subtree|
-                  sout = subtree.empty?? [] :roam.call(subtree)
-                  supervisors_first ? sout.unshift(sup) : sout.push(sup)
-                  out.concat(sout)
+          module ClassMethods
+            # Reorders as follows:
+            #   1. supervisors, people with no supervisor or where their supervisor not present
+            #   2. subordinates
+            def sort_by_supervisors(values, supervisors_first: true)
+              raise "Expected non hash Enumerable. Given: #{values.class}" if values.is_a?(Hash)
+              return [] unless values && values.is_a?(Enumerable)
+              roam = Proc.new do |tree|
+                [].tap do |out|
+                  sub_outs = tree.empty?? [] : tree.map {|sup, subtree| roam.call(subtree)}
+                  tree.each do |sup, subtree|
+                    sout = subtree.empty?? [] :roam.call(subtree)
+                    supervisors_first ? sout.unshift(sup) : sout.push(sup)
+                    out.concat(sout)
+                  end
                 end
               end
-            end
 
-            roam.call(supervisors_tree(values))
-          end
+              roam.call(supervisors_tree(values))
+            end
 
-          def tree_to_str(tree, lev: 0)
-            raise "Required Hash tree structure. Given: #{tree.class}" unless tree.is_a?(Hash)
-            "".tap do |str|
-              tree.each do |entry, subtree|
-                str << "#{"  " * lev}+-- #{entry.id || entry.external_id}\n"
-                str << tree_to_str(subtree, lev: lev + 1) unless !subtree || subtree.empty?
+            def tree_to_str(tree, lev: 0)
+              raise "Required Hash tree structure. Given: #{tree.class}" unless tree.is_a?(Hash)
+              "".tap do |str|
+                tree.each do |entry, subtree|
+                  str << "#{"  " * lev}+-- #{entry.id || entry.external_id}\n"
+                  str << tree_to_str(subtree, lev: lev + 1) unless !subtree || subtree.empty?
+                end
               end
             end
-          end
 
-          def print_tree(tree, lev: 0)
-            puts tree_to_str(tree)
-          end
+            def print_tree(tree, lev: 0)
+              puts tree_to_str(tree)
+            end
 
-          # Generates a `Hash` tree structure, where:
-          #   * **keys** are nodes
-          #   * **values** are `Hash` subtree structures of `key` subordinates
-          # @note it is resilient to cyclic supervisors (it will just add the last at the top)
-          # @param values [Enumerable<Object>] of objects with methods:
-          #  `id`, `external_id` and `supervisor_id`
-          # @return [Hash] the tree structure
-          def supervisors_tree(values)
-            raise "Expected non hash Enumerable. Given: #{values.class}" if values.is_a?(Hash)
-            return {} unless values && values.is_a?(Enumerable)
-            idx = get_super_indexes(values)
-
-            processed = []
-            subtree = Proc.new do |entry, level, toptree|
-              if processed.include?(entry)
-                next {} unless toptree.key?(entry) && level > 0
-                # needs to be moved as a child
-                subnodes = toptree.delete(entry)
-                processed.delete(entry)
-              end
+            # Generates a `Hash` tree structure, where:
+            #   * **keys** are nodes
+            #   * **values** are `Hash` subtree structures of `key` subordinates
+            # @note it is resilient to cyclic supervisors (it will just add the last at the top)
+            # @param values [Enumerable<Object>] of objects with methods:
+            #  `id`, `external_id` and `supervisor_id`
+            # @return [Hash] the tree structure
+            def supervisors_tree(values)
+              raise "Expected non hash Enumerable. Given: #{values.class}" if values.is_a?(Hash)
+              return {} unless values && values.is_a?(Enumerable)
+              idx = get_super_indexes(values)
+
+              processed = []
+              subtree = Proc.new do |entry, level, toptree|
+                if processed.include?(entry)
+                  next {} unless toptree.key?(entry) && level > 0
+                  # needs to be moved as a child
+                  subnodes = toptree.delete(entry)
+                  processed.delete(entry)
+                end
 
-              subnodes ||= {}.tap do |tree|
-                subs = idx[:subordinates].call(entry)
-                processed.push(entry)
-                next nil unless subs && !subs.empty?
-                subs.each do |sub|
-                  sub_tree = subtree.call(sub, level + 1, toptree)
-                  tree.merge!(sub_tree)
+                subnodes ||= {}.tap do |tree|
+                  subs = idx[:subordinates].call(entry)
+                  processed.push(entry)
+                  next nil unless subs && !subs.empty?
+                  subs.each do |sub|
+                    sub_tree = subtree.call(sub, level + 1, toptree)
+                    tree.merge!(sub_tree)
+                  end
                 end
-              end
 
-              {entry => subnodes}
-            end
+                {entry => subnodes}
+              end
 
-            {}.tap do |tree|
-              idx[:by_sup].keys.each do |sup_id|
-                if sup = idx[:supers][sup_id]
-                  tree.merge!(subtree.call(sup, 0, tree))
-                else
-                  idx[:by_sup][sup_id].each do |sub|
-                    tree.merge!(subtree.call(sub, 0, tree))
+              {}.tap do |tree|
+                idx[:by_sup].keys.each do |sup_id|
+                  if sup = idx[:supers][sup_id]
+                    tree.merge!(subtree.call(sup, 0, tree))
+                  else
+                    idx[:by_sup][sup_id].each do |sub|
+                      tree.merge!(subtree.call(sub, 0, tree))
+                    end
                   end
                 end
               end
             end
-          end
 
-          private
-
-          def get_super_indexes(values)
-            raise "Expected non hash Enumerable. Given: #{values.class}" if values.is_a?(Hash)
-            {}.tap do |indexes|
-              indexes[:by_id]  = values.map do |e|
-                e.id && [e.id, e]
-              end.compact.to_h
-              indexes[:by_ext] = values.map do |e|
-                e.external_id && [e.external_id, e]
-              end.compact.to_h
-
-              indexes[:by_sup] = {}.tap do |by_s|
-                values.group_by do |e|
-                  e.supervisor_id
-                end.tap do |by_sup|
-                  by_s[nil] = by_sup.delete(nil) if by_sup.key?(nil)
-                  by_s.merge!(by_sup)
+            private
+
+            def get_super_indexes(values)
+              raise "Expected non hash Enumerable. Given: #{values.class}" if values.is_a?(Hash)
+              {}.tap do |indexes|
+                indexes[:by_id]  = values.map do |e|
+                  e.id && [e.id, e]
+                end.compact.to_h
+                indexes[:by_ext] = values.map do |e|
+                  e.external_id && [e.external_id, e]
+                end.compact.to_h
+
+                indexes[:by_sup] = {}.tap do |by_s|
+                  values.group_by do |e|
+                    e.supervisor_id
+                  end.tap do |by_sup|
+                    by_s[nil] = by_sup.delete(nil) if by_sup.key?(nil)
+                    by_s.merge!(by_sup)
+                  end
                 end
-              end
 
-              indexes[:supers] = {}.tap do |sups|
-                indexes[:by_ext].select do |ext, e|
-                  ext && indexes[:by_sup].key?(ext)
-                end.tap {|found| sups.merge!(found)}
-                indexes[:by_id].select do |id, e|
-                  id && indexes[:by_sup].key?(id)
-                end.tap {|found| sups.merge!(found)}
-              end
+                indexes[:supers] = {}.tap do |sups|
+                  indexes[:by_ext].select do |ext, e|
+                    ext && indexes[:by_sup].key?(ext)
+                  end.tap {|found| sups.merge!(found)}
+                  indexes[:by_id].select do |id, e|
+                    id && indexes[:by_sup].key?(id)
+                  end.tap {|found| sups.merge!(found)}
+                end
 
-              indexes[:is_super]     = Proc.new do |entry|
-                !!(indexes[:supers][entry.id] || indexes[:supers][entry.external_id])
-              end
+                indexes[:is_super]     = Proc.new do |entry|
+                  !!(indexes[:supers][entry.id] || indexes[:supers][entry.external_id])
+                end
 
-              indexes[:subordinates] = Proc.new do |entry|
-                subs = nil
-                if sup = indexes[:supers][entry.id] || indexes[:supers][entry.external_id]
-                  subs ||= indexes[:by_sup][sup.id]          unless !sup.id
-                  subs ||= indexes[:by_sup][sup.external_id] unless !sup.external_id
+                indexes[:subordinates] = Proc.new do |entry|
+                  subs = nil
+                  if sup = indexes[:supers][entry.id] || indexes[:supers][entry.external_id]
+                    subs ||= indexes[:by_sup][sup.id]          unless !sup.id
+                    subs ||= indexes[:by_sup][sup.external_id] unless !sup.external_id
+                  end
+                  subs
                 end
-                subs
               end
+
             end
 
           end
 
+          class << self
+            include SupervisorHelpers::ClassMethods
+          end
+
         end
       end
     end

data/lib/eco/api/organization/people.rb

@@ -65,8 +65,8 @@ module Eco
               pers = candidates.first
             end
 
+            pers = @by_external_id[email&.downcase.strip]&.first    if !pers && !email.to_s.strip.empty?
           end
-          pers = @by_external_id[email&.downcase.strip]&.first    if !pers && !email.to_s.strip.empty?
 
           pers
         end

data/lib/eco/api/organization/policy_groups.rb

@@ -44,7 +44,6 @@ module Eco
         end
 
         def policy_group(id_name)
-          pp "the id: #{policy_group_id(id_name)}"
           @by_id.fetch(policy_group_id(id_name), nil)
         end
 
@@ -73,7 +72,6 @@ module Eco
         end
 
         def policy_group_id(id_name)
-          pp "by name ('#{id_name}'): #{@by_name[id_name&.downcase]}"
           (@by_name[id_name&.downcase] || @by_id[id_name])&.id
         end
 

data/lib/eco/api/organization/tag_tree.rb

@@ -62,7 +62,7 @@ module Eco
         # @param key [String] tag to verify.
         # @return [Boolean]
         def tag?(key)
-          @hash_tags.key?(key.upcase)
+          @hash_tags.key?(key&.upcase)
         end
 
         # Finds a subtree node.

data/lib/eco/api/usecases/default_cases/to_csv_detailed_case.rb

@@ -24,21 +24,31 @@ module Eco
               CSV.open(file, "w") do |csv|
                 deps    = {"supervisor_id" => {people: people}}
                 header  = session.new_entry(people.first, dependencies: deps).to_hash.keys
+                header += ["Subordinates"]
+                header += ["Supervisor Name"]
                 header += abilities
                 header += ["Login Methods"] if login_providers.any_optional?
-                header += ["Subordinates"]
                 header += ["Landing Page"]
 
                 csv << header
                 people.each do |person|
                   data = session.new_entry(person, dependencies: deps).to_hash.values
+
+                  data.push(person.subordinates)
+                  super_id = person.supervisor_id
+                  if supervisor = people.person(id: super_id, external_id: super_id)
+                    data.push(supervisor.name)
+                  else
+                    data.push("")
+                  end
+
                   person_abilities = (person.account && person.account.permissions_custom) || {}
                   data += abilities.map {|key| person_abilities[key] || "no access"}
+
                   if login_providers.any_optional?
                     logins = (person.account && person.account.login_provider_ids) || []
                     data.push(login_providers.to_name(logins).join("|"))
                   end
-                  data.push(person.subordinates)
 
                   if person.account && landing_id = person.account.landing_page_id
                     data.push(landing_id)

data/lib/eco/version.rb

@@ -1,3 +1,3 @@
 module Eco
-  VERSION = "1.3.10"
+  VERSION = "1.3.15"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: eco-helpers
 version: !ruby/object:Gem::Version
-  version: 1.3.10
+  version: 1.3.15
 platform: ruby
 authors:
 - Oscar Segura
@@ -107,7 +107,7 @@ dependencies:
         version: '0.5'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.5.5
+        version: 0.5.6
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -117,7 +117,7 @@ dependencies:
         version: '0.5'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.5.5
+        version: 0.5.6
 - !ruby/object:Gem::Dependency
   name: aws-sdk-s3
   requirement: !ruby/object:Gem::Requirement