eco-helpers 2.0.45 → 2.0.49

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: f85bd156250d1f44ca911d98007005cbbb2281ff4a40a2fc07a80680fd42a408
-  data.tar.gz: 4ccda90fbf7d1bbfc814d18cd033110b7af2ae366687af1fa2db10370788c52d
+  metadata.gz: 9b4a2ad99ea2e78519b611cdcdb74743ae9dd1a8f5707164ebcc686c21d83700
+  data.tar.gz: b9387499878102825b1396bf35394cc12c35c4b738380f3bfd52e2407c4239af
 SHA512:
-  metadata.gz: f4b7add6e04fb8b4dfa0c56371bc3f85e791659754e937f60586b88dec06186723f9286c87482144bd9b012298dbc03251388c49f1245008026a186cd6d16a88
-  data.tar.gz: 7a0bfda7154a8c0e7bb7e18100a96cd2a45f26a6d45309725e7d796dd9d0ad48cefb35be551c031706757d1241fab7505d74f401cf6a04764547bbe8c582e076
+  metadata.gz: d574a346ba439207b85cfd1e265ed4b3c3f3742dd6308adfc8a873717bea8fd0f9ceaec49fcde1ea5575068c4e00395a4a200b42d624b45e707bd89d68b4560a
+  data.tar.gz: 99a77ede34b322d1ebe6ad54822db841ae111e7063a4dbd6317dd092222a43919e8e059536db9f9c386bbbc31cd8043e64192f425a4fae055f3511ae6f622e67

data/CHANGELOG.md

@@ -1,17 +1,74 @@
 # Change Log
 All notable changes to this project will be documented in this file.
 
-## [2.0.45] - 2021-11-xx
+## [2.0.49] - 2022-02-14
+
+### Added
+  - `Eco::API::Session::Batch::RequestStats#message` now it shows which details have changed
+  - `Eco::API::UseCases::DefaultCases::ReinviteTransCase` the option `-force` will also send an invite to users that have accepted the invitation
+
+### Changed
+### Fixed
+
+## [2.0.48] - 2022-02-04
+
+### Changed
+  - upgraded `ecoportal-api-v2` dependency
+
+### Fixed
+  - `Eco::API::Common::People::EntryFactory#get_file_content`
+    - Call to `String#scrub` to prevent _invalid byte sequence_ error by replacing those characters by their coding identifier
+    - This has been implemented with a max tolerance of 5 byte sequence errors
+    - Errors are logged (although could not get to identify the line :/)
+
+## [2.0.47] - 2022-02-03
+
+### Added
+  - `Eco::API::UseCases::DefaultCases::CleanUnknownTags`
+      - It is possible to extend or override `register_tags`
+      - New command option `-include-register-tags` (will not ignore register tags; they will be removed too)
+  - `Eco::API::UseCases::DefaultCases::Samples::Sftp`
+      - Default sample case to list, get and archive SFTP remote files
+### Changed
+  - upgraded `ecoportal-api-v2` dependency
+
+### Fixed
+  - `Eco::API::Common::Session::S3Uploader#upload_file`: check if file exists before opening.
+
+## [2.0.46] - 2021-12-24
+
+### Added
+  - `Eco::API::UseCases::OozeSamples::RegisterUpdateCase`
+    - added counter of retrieved entries (to be able to compare with matched initially scoped results)
+    - added also counter of failed to retrieve oozes
+    - added also counter of double ups in search pagination
+    - counter for updated pages and pages that failed update
+  - `Eco::API::Session#enviro_name`: to identify in which environment we are running
+
+### Changed
+  - upgraded `ecoportal-api-v2` dependency
+  - `Eco::API::UseCases::OozeSamples::OozeBaseCase#update_ooze`
+    - To return `false` if there was request made to the server
+    - To return `Reponse` if there was request made to the server
+
+### Fixed
+  - Changed the `workflow` in
+    - `on(:end)` to do not recache people if there were no `:sync` or `:transform` cases
+  - `Eco::API::UseCases::OozeSamples::OozeRunBaseCase#update_ooze`
+    - It's parent had parameter (it as a child should so)
+  - `Eco::API::UseCases::OozeSamples::OozeBaseCase#process_ooze`
+    - It was not retrieving the `ooze` unless explicitly passed as parameter
+    - This did not play well with child implementations that expect it to retrieve it based on `ooze_id` cli argument
+
+## [2.0.45] - 2021-12-15
 
 ### Added
   - `Eco::API::UseCases::OozeSamples::RegisterExportCase`
-  
+
 ### Changed
   - Upgrade `ecoportal-api` dependency
   - Upgrade `ecoportal-api-v2` dependency
 
-### Fixed
-
 ## [2.0.44] - 2021-11-25
 
 ### Changed

data/eco-helpers.gemspec

@@ -31,7 +31,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "redcarpet", ">= 3.5.1",  "< 3.6"
 
   spec.add_dependency 'ecoportal-api', '>= 0.8.4', '< 0.9'
-  spec.add_dependency 'ecoportal-api-v2', '>= 0.8.21', '< 0.9'
+  spec.add_dependency 'ecoportal-api-v2', '>= 0.8.25', '< 0.9'
   spec.add_dependency 'aws-sdk-s3',    '>= 1.83.0', '< 2'
   spec.add_dependency 'aws-sdk-ses',   '>= 1.36.0', '< 2'
   spec.add_dependency 'dotenv',        '>= 2.7.6', '< 2.8'

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

@@ -196,7 +196,23 @@ module Eco
             ext        = File.extname(file)
             encoding ||= Eco::API::Common::Session::FileManager.encoding(file)
             encoding   = (encoding != "utf-8")? "#{encoding}|utf-8": encoding
-            content    = File.read(file, encoding: encoding)
+            read_with_tolerance(file, encoding: encoding)
+          end
+
+          def read_with_tolerance(file, encoding:)
+            if content = File.read(file, encoding: encoding)
+              tolerance = 5
+              content.scrub do |bytes|
+                replacement = '<' + bytes.unpack('H*')[0] + '>'
+                if tolerance <= 0
+                  logger.error("There were more than 5 encoding errors in the file '#{file}'.")
+                  return content
+                else
+                  logger.error("Encoding problem in file '#{file}': '#{replacement}'.")
+                  replacement
+                end
+              end
+            end
           end
 
           def fatal(msg)

data/lib/eco/api/common/session/s3_uploader.rb

@@ -32,8 +32,10 @@ module Eco
           # @param path [String] the target file to be uploaded
           # @return [String] S3 path to the uploaded `path` file
           def upload_file(path)
-            File.open(path, "rb") do |f|
-              upload(File.basename(path), f)
+            if File.exist?(path)
+              File.open(path, "rb") do |f|
+                upload(File.basename(path), f)
+              end
             end
           end
 

data/lib/eco/api/microcases/people_load.rb

@@ -30,7 +30,7 @@ module Eco
             people_load(modifier: modifier - [:newest, :file])
           end
         when modifier.include?(:api)
-          logger.info("Going to get all the people via API")
+          logger.info("Going to get all the people via API (load)")
 
           start = Time.now
           session.batch.get_people.tap do |people|

data/lib/eco/api/session/batch/request_stats.rb

@@ -9,6 +9,7 @@ module Eco
           ACCOUNT_ATTRS  = (Eco::API::Common::People::PersonParser::ACCOUNT_ATTRS + ["permissions_custom"]).uniq
           DETAILS_ATTRS  = ["fields"]
           BLANKED_PREFIX = "blanked_"
+          DETAILS_FIELDS = "details_fields"
 
           class << self
 
@@ -74,44 +75,43 @@ module Eco
           end
 
           def message(percent: false)
-            key_val_delimiter = ": "; attr_delimiter = " ++ "
-            pairs_to_line = Proc.new do |pairs|
-              pairs.map do |p|
-                [p.first.to_s, "#{p.last.to_s}" + (percent ? "%" : "")].join(key_val_delimiter)
-              end.join(attr_delimiter)
-            end
-
             lines = []
-            lines << pairs_to_line.call(core_pairs(percent: percent))
-            lines << pairs_to_line.call(account_pairs(percent: percent))
-            lines << pairs_to_line.call(details_pairs(percent: percent))
+            lines << pairs_to_line(core_pairs(percent: percent))
+            lines << pairs_to_line(account_pairs(percent: percent))
+            lines << pairs_to_line(details_pairs(percent: percent))
             lines.join("\n")
           end
 
           private
 
-          def attr(attr, percent: false, total: count)
-            i = @stats["#{attr}"]
+          def attr_value(attr, percent: false, total: count, details: false)
+            target = details ? (@stats[DETAILS_FIELDS] || {}) : @stats
+            i = target["#{attr}"]
             return i unless percent
             percentage(i, total: total)
           end
 
           def core(percent: false)
-            attr("core", percent: percent)
+            attr_value("core", percent: percent)
           end
 
           def account(percent: false)
-            attr("account", percent: percent)
+            attr_value("account", percent: percent)
           end
 
           def details(percent: false)
-            attr("details", percent: percent)
+            attr_value("details", percent: percent)
+          end
+
+          def stats
+            @stats ||= Hash.new(0)
           end
 
           def build(requests)
-            Hash.new(0).tap do |stats|
+            stats.tap do |stats|
               stats[type] = count
               unless !requests || !requests.is_a?(Enumerable) || requests.empty?
+                stats[DETAILS_FIELDS] = Hash.new(0)
                 requests.each_with_index do |request|
                   add_core_stats(stats, request || {})
                   add_account_stats(stats, request || {})
@@ -149,16 +149,15 @@ module Eco
             if request.key?("details")
               stats["details"] += 1
               stats["details_remove"] += 1 if !request["details"]
+
+              det_attrs = {}
               if fields = request.dig("details", "fields")
                 stats["fields"] += fields.length
+                det_attrs = fields.each_with_object(det_attrs) {|fld, hash| hash[fld["alt_id"]] = fld["value"]}
               end
-            end
-          end
 
-          def pairs(attrs, percent: false, total: count)
-            pairs = attrs.map do |a|
-              (v = attr(a, percent: percent, total: count)) > 0 ? [a, v] : nil
-            end.compact
+              attrs_to_stat(stats[DETAILS_FIELDS], det_attrs, det_attrs.keys)
+            end
           end
 
           def core_pairs(percent: false)
@@ -172,22 +171,23 @@ module Eco
           end
 
           def details_pairs(percent: false)
-            details_pairs  = [["details", details(percent: percent)]]
-            details_pairs += [["fields", fields_average]] if attr("fields") && fields_average
-            details_pairs += pairs(["details_remove"], percent: percent, total: details)
+            det_pairs  = [["details", details(percent: percent)]]
+            det_pairs += [["fields", fields_average]] if attr_value("fields") && fields_average
+            det_pairs += pairs(["details_remove"], percent: percent, total: details)
+            det_pairs += pairs(details_field_attrs, percent: percent, total: details, details: true)
           end
 
-          def fields_average
-            if (fields_num = attr("fields")) && (total = details) > 0
-              (fields_num.to_f / total.to_f).round(2)
-            end
+          def pairs(attrs, percent: false, total: count, details: false)
+            pairs = attrs.map do |a|
+              (v = attr_value(a, percent: percent, total: count, details: details)) > 0 ? [a, v] : nil
+            end.compact
           end
 
-          def percentage(num, total: count)
-            total ||= count
-            if num
-              (num.to_f / total * 100).round(2)
-            end
+          def pairs_to_line(pairs, percent: false)
+            key_val_delimiter = ": "; attr_delimiter = " ++ "
+            pairs.map do |p|
+              [p.first.to_s, "#{p.last.to_s}" + (percent ? "%" : "")].join(key_val_delimiter)
+            end.join(attr_delimiter)
           end
 
           def core_attrs
@@ -202,6 +202,10 @@ module Eco
             @details_attrs ||= self.class.details_attrs
           end
 
+          def details_field_attrs
+            @stats[DETAILS_FIELDS].keys
+          end
+
           def blank_attrs(attrs)
             self.class.blank_attrs(attrs)
           end
@@ -225,6 +229,18 @@ module Eco
             end
           end
 
+          def fields_average
+            if (fields_num = attr_value("fields")) && (total = details) > 0
+              (fields_num.to_f / total.to_f).round(2)
+            end
+          end
+
+          def percentage(num, total: count)
+            total ||= count
+            if num
+              (num.to_f / total * 100).round(2)
+            end
+          end
         end
       end
     end

data/lib/eco/api/session.rb

@@ -17,6 +17,11 @@ module Eco
         logger.debug("LINE COMMAND: #{$0} #{ARGV.join(" ")}")
       end
 
+      # @retrun [String] the name of the current environment
+      def enviro_name
+        config.active_enviro
+      end
+
       # @return [Eco::API::Session::Batch] provides helper to launch batch operations.
       def batch
         @batch  ||= Batch.new(enviro)

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

@@ -2,6 +2,8 @@ class Eco::API::UseCases::DefaultCases::CleanUnknownTags < Eco::API::Common::Loa
   name "clean-unknown-tags"
   type :transform
 
+  UNKNOWN_LOCS_FILE = "unknown_locations.txt"
+
   REGISTER_TAGS = [
     "EVENT", "INJURY", "RISK", "CONTRACTOR", "PERMIT",
     "AUDIT", "JSEA",
@@ -20,18 +22,52 @@ class Eco::API::UseCases::DefaultCases::CleanUnknownTags < Eco::API::Common::Loa
     people.each do |person|
       unknown_tags = person.filter_tags.select {|tag| !tag?(tag)}
       person.filter_tags -= unknown_tags
+      unknown_tag!(*unknown_tags)
       update.add(person)
     end
   end
 
   private
 
+  def unknown_tag!(*tags)
+    tags.each do |value|
+      unless unknown.include?(value)
+        unknown << value
+        File.open(unknown_tags_file, 'a') do |fd|
+          fd.write("#{value}\n")
+        end
+        puts "Custom detected: '#{value}'"
+      end
+    end
+  end
+
+  def register_tags
+    REGISTER_TAGS
+  end
+
+  def unknown
+    @unknown ||= []
+  end
+
   def tag?(value)
-    tagtree.tag?(value) || REGISTER_TAGS.any? {|reg| value == reg}
+    tagtree.tag?(value).yield_self do |is_loc|
+      next true if is_loc
+      if clean_register_tags?
+        register_tags.any? {|reg| value == reg}
+      end
+    end
   end
 
   def tagtree
     @tagtree ||= ASSETS.config.tagtree
   end
 
+  def unknown_tags_file
+    File.join(".","#{session.enviro_name}_#{UNKNOWN_LOCS_FILE}")
+  end
+
+  def clean_register_tags?
+    !options.dig(:usecase, :include_register_tags)
+  end
+
 end

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

@@ -2,16 +2,25 @@ class Eco::API::UseCases::DefaultCases::ReinviteTransCase < Eco::API::Common::Lo
   name "reinvite"
   type :transform
 
+  def main(people, session, options, usecase)
+    reinvite(people.users, session, options, usecase)
+  end
+
+  private
+
   def reinvite(users, session, options, usecase)
     invite   = session.new_job("main", "invite", :update, usecase, :account)
     users.each do |person|
-      person.account.send_invites = true
+      if force_invite?
+        person.account.send_invites = true
+      else
+        person.account.force_send_invites = true
+      end
       invite.add(person)
     end
   end
 
-  def main(people, session, options, usecase)
-    reinvite(people.users, session, options, usecase)
+  def force_invite?
+    options.dig(:force, :invite)
   end
-
 end

data/lib/eco/api/usecases/default_cases/samples/sftp_case.rb

@@ -0,0 +1,109 @@
+class Eco::API::UseCases::DefaultCases::Samples::Sftp < Eco::API::Common::Loaders::UseCase
+  name "sftp-sample"
+  type :other
+
+  attr_reader :session
+
+  def main(session, options, usecase)
+    @session = session
+    options[:end_get] = false
+    raise "The SFTP is not configured" unless session.sftp?
+    case options.dig(:sftp, :command)
+    when :list
+      list_folder
+    when :get
+      get_files
+    when :get_last
+      get_last
+    when :archive
+      archive_files
+    end
+  end
+
+  private
+
+  def file_pattern
+    raise "You should redefine the file_pattern function as a RegEx expression that matches the target remote file"
+  end
+
+  # Ex: "/IN/Personnel"
+  def remote_subfolder
+    raise "You should redefine remote_subfolder as the folder where the target file sits. Ex: /IN/Personnel"
+  end
+
+  def archive_subfolder
+    "Archive"
+  end
+
+  def with_remote_files
+    sftp.files(remote_folder, pattern: file_pattern).each do |remote_file|
+      yield(remote_file) if block_given?
+    end
+  end
+
+  def list_folder
+    puts "Listing remote folder: '#{remote_folder}':"
+    with_remote_files {|file| puts file.longname}
+  end
+
+  def get_files
+    with_remote_files.tap do |files|
+      unless files.empty?
+        file_names = files.map {|file| to_remote_path(file.name)}
+        puts "Getting the following files:"
+        puts file_names
+        sftp.download(file_names, local_folder: local_folder)
+      end
+    end
+  end
+
+  def get_last
+    with_remote_files.last.tap do |file|
+      if file
+        file_name = to_remote_path(file.name)
+        puts "Getting the following file: #{file_name}"
+        sftp.download(file_name, local_folder: local_folder)
+      end
+    end
+  end
+
+  def archive_files
+    with_remote_files do |file|
+      source = to_remote_path(file.name) # should probably be file.longname
+      dest   = to_remote_path("#{archive_subfolder}/#{file.name}")
+      move_file(source, dest)
+    end.tap do |files|
+      puts "Moved the file(s) to the #{archive_subfolder} folder" unless files.empty?
+    end
+  end
+
+  def move_file(source, dest)
+    sftp.move(source, dest, 0x0001) do |response|
+      if response.ok?
+        puts "#{source}\n  -to-> #{dest}"
+      else
+        puts "Could not move file #{source}"
+      end
+    end
+  end
+
+  def to_remote_path(file)
+    remote_folder + "/" + file
+  end
+
+  def local_folder
+    "."
+  end
+
+  def remote_folder
+    @remote_folder ||= sftp_config.remote_folder + remote_subfolder
+  end
+
+  def sftp_config
+    session.config.sftp
+  end
+
+  def sftp
+    session.sftp
+  end
+end

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

@@ -0,0 +1,12 @@
+module Eco
+  module API
+    class UseCases
+      class DefaultCases
+        module Samples
+        end
+      end
+    end
+  end
+end
+
+require_relative 'samples/sftp_case'

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

@@ -9,6 +9,7 @@ module Eco
   end
 end
 
+require_relative 'default_cases/samples'
 require_relative 'default_cases/abstract_policygroup_abilities_case.rb'
 require_relative 'default_cases/analyse_people_case'
 require_relative 'default_cases/append_usergroups_case'

data/lib/eco/api/usecases/ooze_samples/ooze_base_case.rb

@@ -20,10 +20,11 @@ class Eco::API::UseCases::OozeSamples::OozeBaseCase < Eco::API::Common::Loaders:
   end
 
   # Write here your script
-  def process_ooze(ooze = target)
+  def process_ooze(ooz = target)
+    ooz ||= ooze
     raise "You need to inherit from this class ('#{self.class}') and call super with a block" unless block_given?
-    yield(ooze)
-    update_ooze(ooze)
+    yield(ooz)
+    update_ooze(ooz)
   end
 
   protected
@@ -109,10 +110,12 @@ class Eco::API::UseCases::OozeSamples::OozeBaseCase < Eco::API::Common::Loaders:
     exit_error "Stage '#{id_name}' doesn't exist in ooze '#{ooze_id}'"
   end
 
+  # It fill update the ooze only if it's dirty (it carries changes)
+  # @return [Boolean, Response] `false` if there was not request against the server, `Response` otherwise
   def update_ooze(ooze = target)
     if !options[:simulate]
-      return unless dirty?(ooze)
-      
+      return false unless dirty?(ooze)
+
       ooze.validate.tap do |validation|
         raise validation if validation.is_a?(String)
       end
@@ -133,6 +136,7 @@ class Eco::API::UseCases::OozeSamples::OozeBaseCase < Eco::API::Common::Loaders:
       end
       backup_patch!(ooze)
       exit(0) if dirty?(ooze) && dry_count > DRY_COUNT
+      false
     end
   end
 

data/lib/eco/api/usecases/ooze_samples/ooze_run_base_case.rb

@@ -39,9 +39,9 @@ class Eco::API::UseCases::OozeSamples::OozeRunBaseCase < Eco::API::UseCases::Ooz
     options.dig(:source, :stage_id)
   end
 
-  def update_ooze
+  def update_ooze(ooz = target)
     prompt_to_confirm!
-    super
+    super(ooz)
   end
 
   def exit_if_no_changes!

data/lib/eco/api/usecases/ooze_samples/register_update_case.rb

@@ -17,12 +17,30 @@ class Eco::API::UseCases::OozeSamples::RegisterUpdateCase < Eco::API::UseCases::
   name "register-update-case"
   type :other
 
+  attr_reader :retrieved_oozes, :non_retrieved_oozes
+  attr_reader :total_search_oozes, :dupped_search_oozes
+  attr_reader :ooze_result_ids, :updated_oozes, :failed_update_oozes
+
   def main(session, options, usecase, &block)
+    @retrieved_oozes = 0
+    @non_retrieved_oozes = 0
+    @dupped_search_oozes = 0
+    @ooze_result_ids     = {}
+    @updated_oozes       = 0
+    @failed_update_oozes = 0
+
     super(session, options, usecase) do
       with_each_entry do |ooze|
         process_ooze(&block)
       end
     end
+    msg  = "Run end:\n"
+    msg += "  • Search results: #{total_search_oozes}\n"
+    msg += "  • Duplicated search results #{dupped_search_oozes}\n"
+    msg += "  • Retrieved a total of #{retrieved_oozes}\n"
+    msg += "  • Could not get #{non_retrieved_oozes} oozes.\n"
+    msg += "  • Updated #{updated_oozes} oozes (failed update on #{failed_update_oozes} oozes).\n"
+    logger.info(msg)
   end
 
   def process_ooze(ooze = target)
@@ -47,7 +65,14 @@ class Eco::API::UseCases::OozeSamples::RegisterUpdateCase < Eco::API::UseCases::
   def with_each_entry
     batched_search_results do |page_results|
       page_results.each do |page_result|
+        if ooze_result_ids[page_result.id]
+          @dupped_search_oozes += 1
+        else
+          ooze_result_ids[page_result.id] = true
+        end
+
         if pending = queue_shift(page_result.id)
+
           if dirty?(pending)
             msg  = "Inconsistent search results. "
             msg << "Launching update on '#{object_reference(pending)}' to be able to queue it back"
@@ -56,7 +81,11 @@ class Eco::API::UseCases::OozeSamples::RegisterUpdateCase < Eco::API::UseCases::
           end
         end
         if ooz = ooze(page_result.id)
+          @retrieved_oozes += 1
           yield(ooz)
+        else
+          @non_retrieved_oozes += 1
+          console.warn "Could not get page #{page_result.id}"
         end
       end
       update_oozes
@@ -113,7 +142,15 @@ class Eco::API::UseCases::OozeSamples::RegisterUpdateCase < Eco::API::UseCases::
 
   def update_oozes(batched_oozes = batch_queue)
     batched_oozes.each do |ooze|
-      update_ooze(ooze)
+      update_ooze(ooze).tap do |result|
+        if result.is_a?(Ecoportal::API::Common::Response)
+          if result.success?
+            @updated_oozes += 1
+          else
+            @failed_update_oozes +=1
+          end
+        end
+      end
     end
     batched_oozes.clear
   end
@@ -127,6 +164,7 @@ class Eco::API::UseCases::OozeSamples::RegisterUpdateCase < Eco::API::UseCases::
 
   def results_preview
     apiv2.registers.search(register_id, search_options.merge(only_first: true)).tap do |search_results|
+      @total_search_oozes = search_results.total
       str_results = "Total target entries: #{search_results.total} (out of #{search_results.total_before_filtering})"
       session.prompt_user("Do you want to proceed (y/N):", explanation: str_results, default: "N", timeout: 10) do |res|
         unless res.upcase.start_with?("Y")

data/lib/eco/cli/config/default/usecases.rb

@@ -90,6 +90,9 @@ ASSETS.cli.config do |cnf|
     desc  = "Cleans from filter_tags those tags that are not present in the tagtree (as per tagtree.json file)."
     desc += " It will preserve standard register tags of most common registers (i.e. EVENT, RISK)."
     cases.add("-clean-unknown-tags", :transform, desc, case_name: "clean-unknown-tags")
+    .add_option("-include-register-tags", "Will also try to remove register tags") do |options|
+      options.deep_merge!(usecase: {include_register_tags: true})
+    end
 
     desc = "Removes the landing page or sets it to -page-id"
     cases.add("-reset-landing-page", :transform, desc, case_name: "reset-landing-page")
@@ -180,9 +183,15 @@ ASSETS.cli.config do |cnf|
 
     desc = "Re-sends invites to all filtered users that have not accepted the invite as yet"
     cases.add("-reinvite", :transform, desc, case_name: "reinvite")
+    .add_option("-force", "If also send an invite email to those that have accepted the invite") do |options|
+      options.deep_merge!(force: {invite: true})
+    end
 
     desc = "Re-sends invites to target users that have not accepted the invite as yet"
     cases.add("-reinvite-from", :sync, desc, case_name: "reinvite")
+    .add_option("-force", "If also send an invite email to those that have accepted the invite") do |options|
+      options.deep_merge!(force: {invite: true})
+    end
 
     desc = "Deletes everybody that has been filtered from the people manager"
     cases.add("-delete", :transform, desc, case_name: "delete")

data/lib/eco/cli/config/default/workflow.rb

@@ -3,6 +3,8 @@ ASSETS.cli.config do |config|
 
     io = nil
     rescued = false
+    cases_with_input  = nil
+    cases_with_output = nil
 
     # default rescue
     wf.rescue do |exception, io|
@@ -165,9 +167,16 @@ ASSETS.cli.config do |config|
     wf.on(:end) do |wf_end, io|
       get_people     = io.options.dig(:people, :get)
       partial_update = get_people && get_people.dig(:type) == :partial
+
       unless !io.options[:end_get] || io.options[:dry_run] || partial_update
-        # Prevent getting people when there were no use cases that used them
-        unless io.people
+        people_update_cases = config.usecases.active(io: io).any? do |usecase, data|
+          [:transform, :sync].any? {|type| usecase.type == type}
+        end
+
+        if !people_update_cases
+          # Prevent getting people when there were no use cases that used them
+          io.session.logger.info("Won't be recaching people, as there haven't been any targetted updates")
+        elsif !io.people
           people = io.session.micro.people_cache
           io     = io.new(people: people)
         end

data/lib/eco/version.rb

@@ -1,3 +1,3 @@
 module Eco
-  VERSION = "2.0.45"
+  VERSION = "2.0.49"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: eco-helpers
 version: !ruby/object:Gem::Version
-  version: 2.0.45
+  version: 2.0.49
 platform: ruby
 authors:
 - Oscar Segura
@@ -136,7 +136,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.8.21
+        version: 0.8.25
     - - "<"
       - !ruby/object:Gem::Version
         version: '0.9'
@@ -146,7 +146,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.8.21
+        version: 0.8.25
     - - "<"
       - !ruby/object:Gem::Version
         version: '0.9'
@@ -546,6 +546,8 @@ files:
 - lib/eco/api/usecases/default_cases/remove_account_trans_case.rb
 - lib/eco/api/usecases/default_cases/reset_landing_page_case.rb
 - lib/eco/api/usecases/default_cases/restore_db_case.rb
+- lib/eco/api/usecases/default_cases/samples.rb
+- lib/eco/api/usecases/default_cases/samples/sftp_case.rb
 - lib/eco/api/usecases/default_cases/set_default_tag_case.rb
 - lib/eco/api/usecases/default_cases/set_supervisor_case.rb
 - lib/eco/api/usecases/default_cases/supers_cyclic_identify_case.rb