eco-helpers 3.0.4 → 3.0.6

data/lib/eco/api/usecases/default/locations/csv_to_tree_case.rb

@@ -2,11 +2,13 @@ class Eco::API::UseCases::Default::Locations::CsvToTree < Eco::API::Common::Load
   name "csv-to-tree"
   type :other
 
+  require_relative 'cli/csv_to_tree_cli'
+
   include Eco::Data::Locations::DSL
 
   TIME_FORMAT = '%Y%m%dT%H%M%S'.freeze
 
-  def main(_session, options, _usecase)
+  def main(*_args)
     options[:end_get] = false
     tree_struct = org_tree(input_csv)
 
@@ -18,10 +20,6 @@ class Eco::API::UseCases::Default::Locations::CsvToTree < Eco::API::Common::Load
 
   private
 
-  def output_file
-    @output_file ||= "#{active_enviro}_tree_#{timestamp}.json"
-  end
-
   def input_csv
     @input_csv ||= Eco::CSV.read(input_file, encoding: input_encoding)
   end
@@ -34,6 +32,18 @@ class Eco::API::UseCases::Default::Locations::CsvToTree < Eco::API::Common::Load
     options.dig(:input, :file, :encoding) || 'utf-8'
   end
 
+  def output_file
+    @output_file ||= File.join(output_folder, output_base_name)
+  end
+
+  def output_base_name
+    "#{active_enviro}_tree_#{timestamp}.json"
+  end
+
+  def output_folder
+    File.dirname(input_file)
+  end
+
   def timestamp(date = Time.now)
     date.strftime(TIME_FORMAT)
   end

data/lib/eco/api/usecases/default/locations/tagtree_extract_case.rb

@@ -116,6 +116,7 @@ class Eco::API::UseCases::Default::Locations::TagtreeExtract < Eco::API::UseCase
 
   def input_csv
     return unless file_as_source?
+
     @input_csv ||= Eco::CSV.read(file_as_source, encoding: input_encoding)
   end
 
@@ -125,6 +126,7 @@ class Eco::API::UseCases::Default::Locations::TagtreeExtract < Eco::API::UseCase
 
   def file_as_source?
     return true unless file_as_source.nil?
+
     false
   end
 
@@ -132,6 +134,7 @@ class Eco::API::UseCases::Default::Locations::TagtreeExtract < Eco::API::UseCase
     @file_as_source ||= options.dig(:source, :file).tap do |file|
       next if file.nil?
       next if File.exist?(file)
+
       log(:error) { "File '#{file}' does not exist" }
       exit 1
     end
@@ -177,6 +180,7 @@ class Eco::API::UseCases::Default::Locations::TagtreeExtract < Eco::API::UseCase
   def csv(filename, header = [])
     CSV.open(filename, "w") do |csv|
       csv << header unless header.empty?
+
       yield(csv)
     end
   ensure
@@ -203,9 +207,11 @@ class Eco::API::UseCases::Default::Locations::TagtreeExtract < Eco::API::UseCase
     sheet = workbook.add_worksheet(name)
     #end
     sheet.auto_width = true
+
     yield(sheet)
 
     return unless header
+
     sheet.set_row(0, 30, workbook.bold_format)
     sheet.freeze_panes(1, 0)
   end

data/lib/eco/api/usecases/default/locations/tagtree_paths_case.rb

@@ -0,0 +1,51 @@
+class Eco::API::UseCases::Default::Locations::TagTreePaths < Eco::API::Common::Loaders::UseCase
+  name "tagtree-paths"
+  type :other
+
+  require_relative 'cli/tagtree_paths_cli'
+
+  OUT_FOLDER      = "sftp".freeze
+  OUT_TIME_FORMAT = '%Y%m%dT%H%M%S'.freeze
+  OUT_FILENAME    = "tag_paths".freeze
+
+  def main(*_args)
+    CSV.open(output_filename, "w") do |csv|
+      csv << %w[Tag Path]
+      tag_paths.each do |values|
+        csv << values
+      end
+    end
+
+    log(:info) {
+      "Generated file '#{output_filename}'"
+    }
+  end
+
+  private
+
+  def tag_paths
+    @tag_paths ||= tagtree.tags.reverse.map do |tag|
+      [tag, tagtree.path(tag).join("|")]
+    end
+  end
+
+  def tagtree
+    session.tagtree
+  end
+
+  def output_file_format
+    'csv'
+  end
+
+  def output_filename(name = self.class::OUT_FILENAME)
+    File.join(output_folder, "#{timestamp}_#{config.active_enviro}_#{name}.#{output_file_format}")
+  end
+
+  def output_folder
+    "#{config.active_enviro}/#{self.class::OUT_FOLDER}"
+  end
+
+  def timestamp(date = Time.now)
+    date.strftime(self.class::OUT_TIME_FORMAT)
+  end
+end

data/lib/eco/api/usecases/default/locations.rb

@@ -10,7 +10,7 @@ module Eco
 end
 
 require_relative 'locations/codes_to_tags_case'
-require_relative 'locations/create_tag_paths_case'
+require_relative 'locations/tagtree_paths_case'
 require_relative 'locations/csv_to_tree_case'
 require_relative 'locations/tagtree_extract_case'
 require_relative 'locations/tagtree_upload_case'

data/lib/eco/api/usecases/default/people/amend/cli/reinvite_sync_cli.rb

@@ -0,0 +1,8 @@
+class Eco::API::UseCases::Default::People::Amend::ReinviteSyncCase
+  class Cli < Eco::API::UseCases::Default::People::Amend::ReinviteTransCase::Cli
+    str = "Re-sends invites to target users that have not accepted the invite as yet"
+    desc str
+
+    cli_name '-reinvite-from'
+  end
+end

data/lib/eco/api/usecases/default/people/amend/cli/reinvite_trans_cli.rb

@@ -0,0 +1,11 @@
+class Eco::API::UseCases::Default::People::Amend::ReinviteTransCase
+  class Cli < Eco::API::UseCases::Cli
+    str = "Re-sends invites to all filtered users that have not accepted the invite as yet"
+    desc str
+
+    str = "If also send an invite email to those that have accepted the invite"
+    add_option("-force", str) do |options|
+      options.deep_merge!(force: {invite: true})
+    end
+  end
+end

data/lib/eco/api/usecases/default/people/amend/cli/restore_db_cli.rb

@@ -0,0 +1,26 @@
+class Eco::API::UseCases::Default::People::Amend::RestoreDBCase
+  class Cli < Eco::API::UseCases::Cli
+    str = "Restores the people manager by using a backup.json file"
+    desc str
+
+    callback do |input, session, _opts, _case|
+      unless input && input.is_a?(Eco::API::Organization::People)
+        file  = SCR.get_file(cli_name, required: true)
+        input = Eco::API::Organization::People.new(JSON.parse(File.read(file)))
+        session.log(:info) {
+          "Source DB: loaded #{input.length} entries."
+        }
+      end
+    end
+
+    str = "If it should DELETE people that do not exist in the backup file"
+    add_option("-include-delete", str) do |options|
+      options.deep_merge!(include: {delete: true})
+    end
+
+    str = "If it should CREATE people that do not exist in the people manager"
+    add_option("-include-create", str) do |options|
+      options.deep_merge!(include: {create: true})
+    end
+  end
+end

data/lib/eco/api/usecases/default/people/amend/reinvite_sync_case.rb

@@ -3,6 +3,8 @@ module Eco::API::UseCases::Default::People::Amend
     name "reinvite"
     type :sync
 
+    require_relative 'cli/reinvite_sync_cli'
+
     def main(entries, *_args)
       found = micro.with_each_present(entries, people, options, log_starter: true)
       reinvite(found.users)

data/lib/eco/api/usecases/default/people/amend/reinvite_trans_case.rb

@@ -2,6 +2,8 @@ class Eco::API::UseCases::Default::People::Amend::ReinviteTransCase < Eco::API::
   name "reinvite"
   type :transform
 
+  require_relative 'cli/reinvite_trans_cli'
+
   def main(*_args)
     reinvite(people.users)
   end

data/lib/eco/api/usecases/default/people/amend/restore_db_case.rb

@@ -4,6 +4,8 @@ class Eco::API::UseCases::Default::People::Amend::RestoreDBCase < Eco::API::Comm
   name "restore-db"
   type :sync
 
+  require_relative 'cli/restore_db_cli'
+
   def main(entries, people, session, options, usecase) # rubocop:disable Metrics/AbcSize
     require_people_as_entries!(entries)
 
@@ -52,6 +54,7 @@ class Eco::API::UseCases::Default::People::Amend::RestoreDBCase < Eco::API::Comm
 
   def account_copy(src, dst) # rubocop:disable Metrics/AbcSize
     return unless src
+
     dst.default_tag        = src.default_tag        unless options.dig(:exclude, :filter_tags)
     dst.policy_group_ids   = src.policy_group_ids   unless options.dig(:exclude, :policy_groups)
     dst.permissions_custom = src.permissions_custom unless options.dig(:exclude, :abilities)

data/lib/eco/api/usecases/default/people/treat/analyse_people_case.rb

@@ -2,6 +2,8 @@ class Eco::API::UseCases::Default::People::Treat::Analyse < Eco::API::Common::Lo
   name "analyse-people"
   type :export
 
+  require_relative 'cli/analyse_people_cli'
+
   attr_reader :people
 
   def main(_people, _session, options, _usecase)
@@ -26,12 +28,14 @@ class Eco::API::UseCases::Default::People::Treat::Analyse < Eco::API::Common::Lo
   def identify_unnamed
     similarity_analytics.unnamed.tap do |unnamed|
       next unless unnamed.empty?
+
       log(:info) { "There were no people with no name!!" }
     end
   end
 
   def identify_duplicates
     analysed = similarity_screening
+
     if case_options[:ignore_matching_words]
       log(:info) { "Fine tune results by ignoring matching words..." }
       analysed = strict_similarity(analysed)
@@ -155,7 +159,9 @@ class Eco::API::UseCases::Default::People::Treat::Analyse < Eco::API::Common::Lo
 
   def save!(data)
     ext = File.extname(output_file).downcase.delete(".")
-    session.logger.info("Generating file '#{output_file}'")
+
+    log(:info) { "Generating file '#{output_file}'" }
+
     File.open(output_file, "w") do |fd|
       case ext
       when "txt"
@@ -184,11 +190,15 @@ class Eco::API::UseCases::Default::People::Treat::Analyse < Eco::API::Common::Lo
   # A way to use command line to specify part
   # => i.e. person.details[first-name]
   def attribute_access(person, expression)
-    parts = expression.split(".")
+    parts = expression.split('.')
+
     parts_to_value(person, parts).tap do |value|
-      unless value.is_a?(String) || !value
-        raise "Something is wrong with #{expression} to parts #{parts}. Expecting String, obtained: #{value.class}"
-      end
+      next if value.is_a?(String)
+      next unless value
+
+      msg  = "Something is wrong with #{expression} to parts #{parts}. "
+      msg << "Expecting String, obtained: #{value.class}"
+      raise msg
     end
   end
 

data/lib/eco/api/usecases/default/people/treat/cli/analyse_people_cli.rb

@@ -0,0 +1,66 @@
+class Eco::API::UseCases::Default::People::Treat::Analyse
+  class Cli < Eco::API::UseCases::Cli
+    str = "Provides a set of tools to analyse a set of people (i.e. detect duplicates)"
+    desc str
+
+    callback do |_people, _session, options|
+      options.deep_merge!(output: {file: "people_analysis.txt"}) unless options.dig(:output, :file)
+      #unless options.dig(:usecase, :analyse_people, :use_field)
+      #  options.deep_merge!(usecase: {analyse_people: {use_field: :name}})
+      #end
+    end
+
+    add_option("-to", "Specify the output file.") do |options|
+      file = SCR.get_file("-to", required: true, should_exist: false)
+      options.deep_merge!(output: {file: file})
+    end
+
+    str = "Generates a list of people with possible duplicates."
+    add_option("-identify-duplicates", str) do |options|
+      options.deep_merge!(usecase: {analyse_people: {identify_duplicates: true}})
+    end
+
+    str = "Works with -identify-duplicates. Sets field to be used in the comparison."
+    add_option("-use-field", str) do |options|
+      expression = SCR.get_arg("-use-field", with_param: true)
+      options.deep_merge!(usecase: {analyse_people: {use_field: expression}})
+    end
+
+    str = "Works with -identify-duplicates. Adds an additional layer of comparison."
+    add_option("-facet-field", str) do |options|
+      expression = SCR.get_arg("-facet-field", with_param: true)
+      options.deep_merge!(usecase: {analyse_people: {facet_field: expression}})
+    end
+
+    str = "Works with -identify-duplicates. Skips the rearrangement stage."
+    add_option("-only-screening", str) do |options|
+      options.deep_merge!(usecase: {analyse_people: {only_screening: true}})
+    end
+
+    str = "Works with -identify-duplicates. Re-adjust scores ignoring matching words."
+    add_option("-ignore-matching-words", str) do |options|
+      options.deep_merge!(usecase: {analyse_people: {ignore_matching_words: true}})
+    end
+
+    str = "Works with -identify-duplicates. Re-adjust the comparing strings to do not have repeated words."
+    add_option("-unique-words", str) do |options|
+      options.deep_merge!(usecase: {analyse_people: {unique_words: true}})
+    end
+
+    add_option("-identify-unnamed", "Identifies all people with no names.") do |options|
+      options.deep_merge!(usecase: {analyse_people: {identify_unnamed: true}})
+    end
+
+    str = "Generates a json file with all the people involved in the final results of the analysis."
+    add_option("-backup-people-results", str) do |options|
+      file = SCR.get_file("-backup-people-results", required: true, should_exist: false)
+      options.deep_merge!(usecase: {analyse_people: {backup_people: File.expand_path(file)}})
+    end
+
+    str = "Genarates a CSV file with all people of the final results."
+    add_option("-to-csv", str) do |options|
+      file = SCR.get_file("-to-csv", required: true, should_exist: false) || "Results.csv"
+      options.deep_merge!(usecase: {analyse_people: {csv_file: File.expand_path(file)}})
+    end
+  end
+end

data/lib/eco/api/usecases/default/people/treat/cli/org_data_convert_cli.rb

@@ -0,0 +1,35 @@
+class Eco::API::UseCases::Default::People::Treat::OrgDataConvertCase
+  class Cli < Eco::API::UseCases::Cli
+    str  = "Usage '#{cli_name} backup.json -restore-db-from'."
+    str << " Transforms an input .json file to the values of the destination environment "
+    str << " (names missmatch won't solve: i.e. usergroups)"
+    desc str
+
+    callback do |input, session, options|
+      unless input.is_a?(Eco::API::Organization::People)
+        file  = SCR.get_file(cli_name, required: true)
+        input = Eco::API::Organization::People.new(JSON.parse(File.read(file)))
+        session.log(:info) {
+          "Source DB: loaded #{input.length} entries."
+        }
+      end
+
+      unless options[:source_enviro]
+        session.log(:error) {
+          "You need to specify a -source-enviro for the conversion to work out"
+        }
+        exit(1)
+      end
+    end
+
+    str = "The defined -source-enviro API configuration that the backup file was generated from"
+    add_option("-source-enviro", str) do |options|
+      options.merge!(source_enviro: SCR.get_arg("-source-enviro", with_param: true))
+    end
+
+    str = "Prevents the script to crash when backup file has missing usergroups in the org"
+    add_option("-ignore-missing-policy-groups", str) do |options|
+      options.deep_merge!(ignore: {missing: {policy_groups: true}})
+    end
+  end
+end

data/lib/eco/api/usecases/default/people/treat/cli/supers_cyclic_identify_cli.rb

@@ -0,0 +1,15 @@
+class Eco::API::UseCases::Default::People::Treat::SupersCyclicIdentify
+  class Cli < Eco::API::UseCases::Cli
+    str = "Provides a set of tools to analyse a set of people (i.e. detect duplicates)"
+    desc str
+
+    # callback do |_people, _session, options|
+    #   options.deep_merge!(output: {file: "cyclic_supers.txt"}) unless options.dig(:output, :file)
+    # end
+
+    add_option("-to", "Specify the output file") do |options|
+      file = SCR.get_file("-to", required: true, should_exist: false)
+      options.deep_merge!(output: {file: file})
+    end
+  end
+end

data/lib/eco/api/usecases/default/people/treat/cli/supers_hierachy_cli.rb

@@ -0,0 +1,15 @@
+class Eco::API::UseCases::Default::People::Treat::SupersHierarchy
+  class Cli < Eco::API::UseCases::Cli
+    str = "Draws the Supervisors hiearchy in a file"
+    desc str
+
+    # callback do |_people, _session, options|
+    #   options.deep_merge!(output: {file: "supers_hierarchy.txt"}) unless options.dig(:output, :file)
+    # end
+
+    add_option("-to", "Specify the output file") do |options|
+      file = SCR.get_file("-to", required: true, should_exist: false)
+      options.deep_merge!(output: {file: file})
+    end
+  end
+end

data/lib/eco/api/usecases/default/people/treat/org_data_convert_case.rb

@@ -2,6 +2,12 @@ class Eco::API::UseCases::Default::People::Treat::OrgDataConvertCase < Eco::API:
   name "org-data-convert"
   type :import
 
+  require_relative 'cli/org_data_convert_cli'
+
+  def main(people, _session, _options, _usecase)
+    org_data_convert(people)
+  end
+
   def org_data_convert(people)
     log(:info) {
       "Going to convert data from '#{source_enviro}' to '#{ASSETS.active_config}'"
@@ -22,14 +28,11 @@ class Eco::API::UseCases::Default::People::Treat::OrgDataConvertCase < Eco::API:
     people
   end
 
-  def main(people, _session, _options, _usecase)
-    org_data_convert(people)
-  end
-
   private
 
   def account_convert!(person)
     return unless (account = person.account)
+
     account.policy_group_ids = policy_groups_convert(account.policy_group_ids)
     account.doc.tap do |doc|
       %w[login_provider_ids starred_ids landing_page_id].map do |attr|
@@ -40,6 +43,7 @@ class Eco::API::UseCases::Default::People::Treat::OrgDataConvertCase < Eco::API:
 
   def details_convert!(person)
     return unless person.details
+
     person.details.as_json.tap do |details_doc|
       details_doc["schema_id"] = session.schema.id
       person.add_details(session.schema)
@@ -50,11 +54,13 @@ class Eco::API::UseCases::Default::People::Treat::OrgDataConvertCase < Eco::API:
   def policy_groups_convert(ids)
     ids.map do |id|
       name = source_policy_groups.to_name(id)
+
       unless name || ignore_missing_policy_groups?
         msg  = "Ops, do not know #{id} usergroup for source environment"
         msg << "\nUse the option -ignore-missing-policy-groups if you do not care"
         error(msg)
       end
+
       session.policy_groups.to_id(name).tap do |new_id|
         unless new_id || ignore_missing_policy_groups?
           msg  = "Ops, do not know #{name} usergroup for destination environment"
@@ -88,7 +94,7 @@ class Eco::API::UseCases::Default::People::Treat::OrgDataConvertCase < Eco::API:
   end
 
   def error(msg)
-    logger.error(msg)
+    log(:error) { msg }
     exit(1)
   end
 end

data/lib/eco/api/usecases/default/people/treat/supers_cyclic_identify_case.rb

@@ -1,7 +1,13 @@
 class Eco::API::UseCases::Default::People::Treat::SupersCyclicIdentify < Eco::API::Common::Loaders::UseCase
-  name "identify-cyclic-supers"
+  name 'identify-cyclic-supers'
   type :export
 
+  require_relative 'cli/supers_cyclic_identify_cli'
+
+  OUT_FOLDER      = "sftp".freeze
+  OUT_TIME_FORMAT = '%Y%m%dT%H%M%S'.freeze
+  OUT_FILENAME    = "cyclic_supers".freeze
+
   attr_reader :people
 
   def main(_people, _session, options, _usecase)
@@ -15,10 +21,6 @@ class Eco::API::UseCases::Default::People::Treat::SupersCyclicIdentify < Eco::AP
     Eco::API::Common::People::SupervisorHelpers.identify_cyclic_chains(people)
   end
 
-  def file
-    @file ||= options.dig(:output, :file) || "cyclic_supers.txt"
-  end
-
   def save!(data)
     if data.empty?
       log(:info) { "There were no cyclic supervisors identified!!" }
@@ -29,7 +31,7 @@ class Eco::API::UseCases::Default::People::Treat::SupersCyclicIdentify < Eco::AP
 
     case ext
     when "txt"
-      create_file(data, file: file, format: :txt)
+      create_file(data, file: output_filename(ext: 'txt'), format: :txt)
     when "html"
       log(:warn) { "html is still not supported" }
       exit(1)
@@ -45,7 +47,10 @@ class Eco::API::UseCases::Default::People::Treat::SupersCyclicIdentify < Eco::AP
     File.open(file, "w") do |fd|
       fd << sets_to_str(sets, format: format)
     end
-    puts "Generated file #{file}"
+
+    log(:info) {
+      "Generated file #{file}"
+    }
   end
 
   def sets_to_str(sets, format: :txt)
@@ -70,4 +75,20 @@ class Eco::API::UseCases::Default::People::Treat::SupersCyclicIdentify < Eco::AP
       str << set_to_str(set, lev: lev + 1, format: format)
     end
   end
+
+  def output_filename(ext: nil)
+    @output_filename ||= options.dig(:output, :file) || default_output_filename(ext: ext)
+  end
+
+  def default_output_filename(name = self.class::OUT_FILENAME, ext: nil)
+    File.join(output_folder, "#{timestamp}_#{config.active_enviro}_#{name}.#{ext}")
+  end
+
+  def output_folder
+    "#{config.active_enviro}/#{self.class::OUT_FOLDER}"
+  end
+
+  def timestamp(date = Time.now)
+    date.strftime(self.class::OUT_TIME_FORMAT)
+  end
 end

data/lib/eco/api/usecases/default/people/treat/supers_hierarchy_case.rb

@@ -2,13 +2,19 @@ class Eco::API::UseCases::Default::People::Treat::SupersHierarchy < Eco::API::Co
   name "supers-hierarchy"
   type :export
 
+  require_relative 'cli/supers_hierachy_cli'
+
+  OUT_FOLDER      = "sftp".freeze
+  OUT_TIME_FORMAT = '%Y%m%dT%H%M%S'.freeze
+  OUT_FILENAME    = "supers_hierarchy".freeze
+
   attr_reader :people
 
   def main(people, _session, options, _usecase)
     options[:end_get] = false
     @people = people
 
-    save!(hierarchy)
+    save!(hierarchy, file: output_filename)
   end
 
   private
@@ -17,11 +23,7 @@ class Eco::API::UseCases::Default::People::Treat::SupersHierarchy < Eco::API::Co
     Eco::API::Common::People::SupervisorHelpers.supervisors_tree(people)
   end
 
-  def file
-    @file ||= options.dig(:output, :file) || "supers_hierarchy.txt"
-  end
-
-  def save!(data)
+  def save!(data, file:)
     ext = File.extname(file).downcase.delete(".")
 
     case ext
@@ -42,18 +44,39 @@ class Eco::API::UseCases::Default::People::Treat::SupersHierarchy < Eco::API::Co
     File.open(file, "w") do |fd|
       fd << tree_to_str(tree, format: format)
     end
-    puts "Generated file #{file}"
+
+    log(:info) {
+      "Generated file #{file}"
+    }
   end
 
   def tree_to_str(tree, lev: 0, format: :txt)
-    raise "Required Hash tree structure. Given: #{tree.class}" unless tree.is_a?(Hash)
+    msg = "Required Hash tree structure. Given: #{tree.class}"
+    raise msg unless tree.is_a?(Hash)
+
     "".tap do |str|
       tree.each do |entry, subtree|
         str << "#{"  " * lev}#{lev.positive? ? "+-#{lev}- " : ""}"
-        str << entry.name
+        str << (entry.name || entry.email || entry.external_id || entry.id)
         str << " (#{entry.external_id}|#{entry.email}|#{entry.id})\n"
         str << tree_to_str(subtree, lev: lev + 1, format: format) unless !subtree || subtree.empty?
       end
     end
   end
+
+  def output_filename(ext: 'txt')
+    @output_filename ||= options.dig(:output, :file) || default_output_filename(ext: ext)
+  end
+
+  def default_output_filename(name = self.class::OUT_FILENAME, ext: nil)
+    File.join(output_folder, "#{timestamp}_#{config.active_enviro}_#{name}.#{ext}")
+  end
+
+  def output_folder
+    "#{config.active_enviro}/#{self.class::OUT_FOLDER}"
+  end
+
+  def timestamp(date = Time.now)
+    date.strftime(self.class::OUT_TIME_FORMAT)
+  end
 end

data/lib/eco/api/usecases/default/people/utils/set_default_tag_case.rb

@@ -23,7 +23,7 @@ class Eco::API::UseCases::Default::People::Utils::SetDefaultTagCase < Eco::API::
     return unless tagtree?
 
     msg = "There is no locs struct defined. Aborting..."
-    logger.fatal(msg)
+    log(:fatal) { msg }
     raise msg
   end
 
@@ -39,6 +39,7 @@ class Eco::API::UseCases::Default::People::Utils::SetDefaultTagCase < Eco::API::
 
   def tagtree?
     return false unless (tree = session.tagtree)
+
     !tree.empty?
   end
 end

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

@@ -2,7 +2,7 @@ class Eco::API::UseCases::DefaultCases::ToCsvCase < Eco::API::Common::Loaders::U
   name "to-csv"
   type :export
 
-  OUT_FILENAME = 'pm'
+  OUT_FILENAME = 'pm'.freeze
 
   attr_reader :people
 
@@ -30,7 +30,7 @@ class Eco::API::UseCases::DefaultCases::ToCsvCase < Eco::API::Common::Loaders::U
   private
 
   def create_file!(filename = file, data = people)
-    logger.info("going to create file: #{filename}")
+    session.log(:info) { "going to create file: #{filename}" }
 
     CSV.open(filename, "w") do |csv|
       csv << spot_header(data.first)