eco-helpers 3.0.4 → 3.0.6

data/lib/eco/csv/table.rb

@@ -5,6 +5,7 @@ module Eco
       #   - when `Array<Array>` => all `rows` as arrays where first array is the **header**
       def initialize(input)
         super(to_rows_array(input))
+
         delete_if do |row|
           values = row.fields
           values.all?(&:nil?) || values.map(&:to_s).all?(&:empty?)
@@ -21,6 +22,7 @@ module Eco
             row[header] = value.empty?? nil : value
           end
         end
+
         self
       end
 
@@ -41,9 +43,11 @@ module Eco
       # @return [Eco::CSV::Table]
       def transform_headers
         cols = columns
+
         cols.each do |col|
           col[0] = yield(col.first)
         end
+
         columns_to_table(cols)
       end
 
@@ -70,6 +74,7 @@ module Eco
             end
           end
         end
+
         self.class.new(out_rows)
       end
 
@@ -83,6 +88,7 @@ module Eco
       def transform_values
         transformed_rows = rows.map do |row|
           res = yield(row)
+
           case res
           when Array
             ::CSV::Row.new(row.headers, res)
@@ -90,6 +96,7 @@ module Eco
             res
           end
         end
+
         self.class.new(transformed_rows)
       end
 
@@ -136,6 +143,7 @@ module Eco
       def add_column(header_name, pos: -1)
         header_name = header_name.to_s.strip
         raise ArgumentError, "header_name can't be blank" if header_name.empty?
+
         new_col = Array.new(length).unshift(header_name)
         columns_to_table(columns.insert(pos, new_col))
       end
@@ -146,6 +154,7 @@ module Eco
       # @return [Eco::CSV::Table] with a new column named `name` with the row number
       def add_index_column(header_name = 'idx', pos: 0)
         header_name = header_name.to_s.strip
+
         add_column(header_name, pos: pos).tap do |table|
           table.each.with_index do |row, idx|
             row[header_name] = idx + 2
@@ -163,6 +172,7 @@ module Eco
       # It removes all rows where all columns' values are the same
       def delete_duplicates!
         unique_rows = []
+
         by_row!.delete_if do |row|
           unique_rows.any? {|done| equal_rows?(row, done)}.tap do |found|
             unique_rows << row unless found
@@ -218,6 +228,7 @@ module Eco
       def columns_to_table(columns_array)
         rows_data = columns_array.transpose
         return self.class.new(rows_data) if rows_data.length.positive?
+
         raise ArgumentError, "Expecting 'columns_array' to at least have header. Given: #{columns_array}"
       end
 
@@ -232,6 +243,7 @@ module Eco
         when Enumerable
           data = data.dup.compact
           return data unless data.count.positive?
+
           sample = data.first
 
           case sample
@@ -264,6 +276,7 @@ module Eco
 
       def if_array_to_pipe_string(value)
         return value unless value.is_a?(Array)
+
         value.join('|')
       end
     end

data/lib/eco/data/files/encoding.rb

@@ -4,7 +4,7 @@ module Eco
       module Encoding
         include Eco::Language::AuxiliarLogger
 
-        BOM_BYTES = [239, 187, 191]
+        BOM_BYTES = [239, 187, 191].freeze
 
         def has_bom?(path)
           return false if !path || file_empty?(path)
@@ -16,9 +16,10 @@ module Eco
 
         def get_file_content_with_encoding(file, encoding: nil)
           encoding ||= scoped_encoding(file)
+
           unless !encoding || encoding == 'utf-8'
             msg = "File encoding: '#{encoding}'"
-            logger.debug(msg)
+            log(:debug) { msg }
             puts msg
           end
 
@@ -29,7 +30,9 @@ module Eco
           else
             content = File.read(file, encoding: encoding)
           end
-          return nil unless content
+
+          return unless content
+
           content = content.encode("utf-8") unless encoding.include?('utf-8')
           content
         end
@@ -42,6 +45,7 @@ module Eco
             content = content.sub(bom, '')
             content.force_encoding('utf-8')
           end
+
           content
         end
 
@@ -52,21 +56,24 @@ module Eco
         # Gives the parameter as it should
         def scoped_encoding(path)
           unless file_exists?(path)
-            logger.error("File does not exist: #{path}")
+            log(:error) { "File does not exist: #{path}" }
             return nil
           end
+
           encoding ||= encoding(path)
-          encoding   = (encoding == "bom") ? "#{encoding}|utf-8": encoding
+          encoding   = "#{encoding}|utf-8" if encoding == "bom"
           encoding
         end
 
         def file_exists?(file)
-          return false if !file
-          return File.exist?(file) || File.exist?(File.expand_path(file))
+          return false unless file
+
+          File.exist?(file) || File.exist?(File.expand_path(file))
         end
 
         def file_empty?(path)
-          return true if !File.file?(path)
+          return true unless File.file?(path)
+
           File.zero?(path)
         end
       end

data/lib/eco/data/files/helpers.rb

@@ -30,11 +30,15 @@ module Eco
             replacement = "<#{bytes.unpack1('H*')}>"
 
             if tolerance <= 0
-              logger.error("There were more than 5 encoding errors in the file '#{file}'.")
+              log(:error) {
+                "There were more than 5 encoding errors in the file '#{file}'."
+              }
               return content
             else
               tolerance -= 1
-              logger.error("Encoding problem in file '#{file}': '#{replacement}'.")
+              log(:error) {
+                "Encoding problem in file '#{file}': '#{replacement}'."
+              }
               replacement
             end
           end

data/lib/eco/data/hashes/diff_result.rb

@@ -48,8 +48,10 @@ module Eco
         #   between `src_1` and `src_2`.
         def diff_attrs
           return (@diff_attrs = []) if new? || del?
+
           @diff_attrs ||= all_compared_attrs.each_with_object([]) do |kattr, out|
             next unless comparable_attr?(kattr)
+
             out << kattr unless eq?(attr_prev(kattr), attr(kattr))
           end
         end
@@ -71,6 +73,7 @@ module Eco
         def diff_attr?(attr)
           return true if new?
           return true if del?
+
           diff_attrs.include?(attr.to_s)
         end
 
@@ -80,6 +83,7 @@ module Eco
           target_attrs = [key] | all_compared_attrs
           return slice_attrs(src_2, *target_attrs) if new?
           return slice_attrs(src_1, key)           if del?
+
           slice_attrs(src_2, key, *diff_attrs)
         end
 
@@ -109,6 +113,7 @@ module Eco
           attr = attr.to_s
           return false unless all_compared_attrs.include?(attr)
           return true  unless self.class.compared_attr_when_present?(attr)
+
           key_present_in_both?(attr)
         end
 

data/lib/eco/data/locations/node_diff/nodes_diff.rb

@@ -28,6 +28,7 @@ class Eco::Data::Locations::NodeDiff
       @diffs ||= super.select do |dff|
         # discard entries that are to be inserted and archived at the same time
         next false if dff.insert? && dff.archive?(validate: false)
+
         dff.unarchive? || dff.id_name? || dff.insert? ||
           dff.move?    || dff.archive?
       end
@@ -67,32 +68,42 @@ class Eco::Data::Locations::NodeDiff
     def diffs_summary # rubocop:disable Metrics/AbcSize
       comp = "(#{source_2.count} input nodes VS #{source_1.count} live nodes)"
       return "There were no differences identified #{comp}" if diffs.empty?
+
       msg  = []
+
       msg << "Identified #{diffs.count} differences #{comp}:"
       msg << when_present(insert) do |count|
         "  • #{count} nodes to insert"
       end
+
       msg << when_present(update) do |count|
         "  • #{count} nodes to update"
       end
+
       msg << when_present(unarchive) do |count|
         "    • #{count} nodes to unarchive (includes ancestors of target nodes)"
       end
+
       msg << when_present(id) do |count|
         "    • #{count} nodes to change id\n"
       end
+
       msg << when_present(name) do |count|
         "    • #{count} nodes to change name"
       end
+
       msg << when_present(classifications) do |count|
         "    • #{count} nodes to change classifications"
       end
+
       msg << when_present(move) do |count|
         "    • #{count} nodes to move"
       end
+
       msg << when_present(archive) do |count|
         "  • #{count} nodes to archive"
       end
+
       msg.compact.join("\n")
     end
 
@@ -108,8 +119,10 @@ class Eco::Data::Locations::NodeDiff
 
     def when_present(list, default = nil)
       raise ArgumentError, "Expecting block but not given" unless block_given?
+
       count = list.count
       return yield(count) if count&.positive?
+
       default
     end
 

data/lib/eco/data/locations/node_diff.rb

@@ -50,12 +50,14 @@ module Eco::Data::Locations
     def id_name?
       return true if id?
       return true if name?
+
       classifications?
     end
 
     # Has the parent id changed?
     def move?
       return false unless update?
+
       parent_id?
     end
 
@@ -72,6 +74,7 @@ module Eco::Data::Locations
       return true  if     marked_for_unarchive?
       return false if     archived
       return false unless update?
+
       archived?
     end
 
@@ -86,6 +89,7 @@ module Eco::Data::Locations
       raise msg    if validate && archived && insert?
 
       return true  if del?
+
       archived
     end
   end

data/lib/eco/data/locations/node_level.rb

@@ -70,6 +70,7 @@ module Eco::Data::Locations
 
     def classifications
       return [] unless (al = actual_level).positive?
+
       original_headers[al - 1]
     end
 
@@ -111,8 +112,10 @@ module Eco::Data::Locations
     # Requires that all upper levels (lower positions) are filled-in
     def common_level_with(other)
       return nil unless other
+
       otags_array = other.tags_array.compact
       stags_array = tags_array.compact
+
       raise "Missing lower levels for #{other.id}: #{other.tags_array.pretty_inspect}" unless other.highest_levels_set?
       raise "Missing lower levels for #{id}:      #{tags_array.pretty_inspect}"        unless highest_levels_set?
 
@@ -174,6 +177,7 @@ module Eco::Data::Locations
     def highest_levels_set?
       return true if raw_level == 1
       return true unless raw_prev_empty_level?
+
       !!@highest_levels_set
     end
 
@@ -194,10 +198,12 @@ module Eco::Data::Locations
         target = i.to_a
       when Integer
         return false unless i >= 1 && i <= tag_attrs_count
+
         target = Array(i..tag_attrs_count)
       else
         return false
       end
+
       return false if target.empty?
 
       target.each do |n|

data/lib/eco/language/auxiliar_logger.rb

@@ -22,6 +22,8 @@ module Eco
 
       # Shortcut to logger.
       def log(level, &block)
+        return unless logger.respond_to?(:level)
+
         logger&.send(level, &block)
       end
     end

data/lib/eco/version.rb

@@ -1,3 +1,3 @@
 module Eco
-  VERSION = '3.0.4'.freeze
+  VERSION = '3.0.6'.freeze
 end

data/lib/eco-helpers.rb

@@ -1,7 +1,7 @@
 # rubocop:disable Naming/FileName
 require 'csv'
 require 'json'
-require 'pp'
+require 'pp' # rubocop:disable Lint/RedundantRequireStatement
 require 'dotenv'
 require 'dotenv/load'
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: eco-helpers
 version: !ruby/object:Gem::Version
-  version: 3.0.4
+  version: 3.0.6
 platform: ruby
 authors:
 - Oscar Segura
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-08-14 00:00:00.000000000 Z
+date: 2024-08-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -269,7 +269,7 @@ dependencies:
         version: '2.0'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.0.8
+        version: 2.0.9
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -279,7 +279,7 @@ dependencies:
         version: '2.0'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 2.0.8
+        version: 2.0.9
 - !ruby/object:Gem::Dependency
   name: ed25519
   requirement: !ruby/object:Gem::Requirement
@@ -663,12 +663,15 @@ files:
 - lib/eco/api/usecases/cli/option.rb
 - lib/eco/api/usecases/default.rb
 - lib/eco/api/usecases/default/locations.rb
+- lib/eco/api/usecases/default/locations/cli/codes_to_tags_cli.rb
+- lib/eco/api/usecases/default/locations/cli/csv_to_tree_cli.rb
 - lib/eco/api/usecases/default/locations/cli/tagtree_extract_cli.rb
+- lib/eco/api/usecases/default/locations/cli/tagtree_paths_cli.rb
 - lib/eco/api/usecases/default/locations/cli/tagtree_upload_cli.rb
 - lib/eco/api/usecases/default/locations/codes_to_tags_case.rb
-- lib/eco/api/usecases/default/locations/create_tag_paths_case.rb
 - lib/eco/api/usecases/default/locations/csv_to_tree_case.rb
 - lib/eco/api/usecases/default/locations/tagtree_extract_case.rb
+- lib/eco/api/usecases/default/locations/tagtree_paths_case.rb
 - lib/eco/api/usecases/default/locations/tagtree_upload_case.rb
 - lib/eco/api/usecases/default/meta.rb
 - lib/eco/api/usecases/default/meta/cli/graphql_schema_cli.rb
@@ -678,16 +681,22 @@ files:
 - lib/eco/api/usecases/default/people/amend/clean_unknown_tags_case.rb
 - lib/eco/api/usecases/default/people/amend/clear_abilities_case.rb
 - lib/eco/api/usecases/default/people/amend/cli/clean_unknown_tags_cli.rb
+- lib/eco/api/usecases/default/people/amend/cli/reinvite_sync_cli.rb
+- lib/eco/api/usecases/default/people/amend/cli/reinvite_trans_cli.rb
+- lib/eco/api/usecases/default/people/amend/cli/restore_db_cli.rb
 - lib/eco/api/usecases/default/people/amend/refresh_case.rb
 - lib/eco/api/usecases/default/people/amend/reinvite_sync_case.rb
 - lib/eco/api/usecases/default/people/amend/reinvite_trans_case.rb
-- lib/eco/api/usecases/default/people/amend/reinvite_trans_cli.rb
 - lib/eco/api/usecases/default/people/amend/restore_db_case.rb
 - lib/eco/api/usecases/default/people/migrate.rb
 - lib/eco/api/usecases/default/people/migrate/cli/remap_tags_cli.rb
 - lib/eco/api/usecases/default/people/migrate/remap_tags_case.rb
 - lib/eco/api/usecases/default/people/treat.rb
 - lib/eco/api/usecases/default/people/treat/analyse_people_case.rb
+- lib/eco/api/usecases/default/people/treat/cli/analyse_people_cli.rb
+- lib/eco/api/usecases/default/people/treat/cli/org_data_convert_cli.rb
+- lib/eco/api/usecases/default/people/treat/cli/supers_cyclic_identify_cli.rb
+- lib/eco/api/usecases/default/people/treat/cli/supers_hierachy_cli.rb
 - lib/eco/api/usecases/default/people/treat/org_data_convert_case.rb
 - lib/eco/api/usecases/default/people/treat/supers_cyclic_identify_case.rb
 - lib/eco/api/usecases/default/people/treat/supers_hierarchy_case.rb

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

@@ -1,25 +0,0 @@
-class Eco::API::UseCases::Default::Locations::TagPaths < Eco::API::Common::Loaders::UseCase
-  name "create-tag-paths"
-  type :other
-
-  def main(session, options, usecase)
-    CSV.open("tag_paths.csv", "w") do |csv|
-      csv << ["Tag", "Path"]
-      tag_paths.each do |values|
-        csv << values
-      end
-    end
-  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
-end

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

@@ -1,4 +0,0 @@
-class Eco::API::UseCases::Default::People::Amend::ReinviteTransCase
-  class Cli < Eco::API::UseCases::Cli
-  end
-end