eco-helpers 2.7.11 → 2.7.13

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e3c92964baf09be5673e74f4a1762d03ce0efe159e7d6a2bc6da3839ac99f4f0
-  data.tar.gz: 26525ccfbdbb06f348d293a50a02de644f7a0a32f3f5285537c75d8d14fe3601
+  metadata.gz: 260621a685a9d1426362308eb8fc12d479f7dc3c5b9702844fcd2c9d6aac90fd
+  data.tar.gz: 62283e9e29d539cc2f475acd4294292efb147c7f6bef2ecfb8e6841f156199e9
 SHA512:
-  metadata.gz: 7620a74731fbfc7d17d1b21f7930df4753003a919fa9d79550083002f454c57716cceab6afec8738d9a37e5b2cd749cd42541f0913286d66c1387b7b88b27c5e
-  data.tar.gz: 1db57df2c9a4b8e5863580524d34e655c92d5e4ddd15e6a8cacbe8b9e9e8af73ae2d3fb05fd1f4dc6de119c2ee5e20b3971b234c073910153aec5137807ef37b
+  metadata.gz: a45aee37bc0e81dbb89d5bfe9cf6cff2b380797b5c6f115975625206dc444be5096ee9d03a22aa7e7491132290bbc879cd4e246ae2a3eb4430bdcc058e780574
+  data.tar.gz: 1c0e6ab8b9f09bd34ae920945c9d99e2fdbaa3e8be2856ca2edd5f9468368e8d7ff2f4d4c4a799511b4a5cfa83eb5dfddea227e7d7f63394dba09fbc8cc8bb26

data/CHANGELOG.md

@@ -1,18 +1,31 @@
 # Change Log
 All notable changes to this project will be documented in this file.
 
-
-## [2.7.11] - 2024-05-xx
+## [2.7.13] - 2024-06-13
 
 ### Added
+  - Case `-split-csv`
+  - Case `-remap-tags` on people case
+
+### Changed
+  - `close_handling_tags_remap_csv` will display table on dry-run too
+    - See `Eco::API::UseCases::GraphQL::Samples::Location::Command::TrackChangedIds`
+
+### Fixed
+
+## [2.7.12] - 2024-05-30
+
+### Changed
+  - Decouple `Eco::API::UseCases::Default::People` namespace
+
+## [2.7.11] - 2024-05-27
+
 ### Changed\
   - `Eco::API::Common::Session::SFTP#download`
     - It now returns the local filename(s) of the downloaded files.
   - `Eco::API::UseCases::GraphQL::Utils#sftp_download_files`
     - It now returns the local filename(s) of the downloaded files.
 
-### Fixed
-
 ## [2.7.10] - 2024-05-20
 
 ### Changed

data/Gemfile

@@ -4,3 +4,5 @@ git_source(:github) {|repo_name| "https://github.com/#{repo_name}" }
 
 # Specify your gem's dependencies in eco-helpers.gemspec
 gemspec
+
+# gem 'rubocop-rake', require: false

data/eco-helpers.gemspec

@@ -10,13 +10,13 @@ Gem::Specification.new do |spec|
   spec.authors       = ["Oscar Segura"]
   spec.email         = ["oscar@ecoportal.co.nz"]
   spec.date          = %q{2018-09-05}
-  spec.summary       = %q{eco-helpers to manage people api cases}
+  spec.summary       = %q(eco-helpers to manage people api cases)
   spec.homepage      = "https://www.ecoportal.com"
   spec.licenses      = %w[MIT]
 
   spec.required_ruby_version = '>= 2.7.2'
 
-  spec.files         = `git ls-files -z`.split("\x0").reject do |f|
+  spec.files = `git ls-files -z`.split("\x0").reject do |f|
     f.match(%r{^(test|spec|features)/})
   end
 
@@ -25,29 +25,31 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib"]
 
   spec.add_development_dependency "bundler",   ">= 2.4.12", "< 3"
-  spec.add_development_dependency "rspec",     ">= 3.12.0", "< 4"
   spec.add_development_dependency "rake",      ">= 13.0.3", "< 14"
-  spec.add_development_dependency "yard",      ">= 0.9.34", "< 1"
   spec.add_development_dependency "redcarpet", ">= 3.6.0",  "< 4"
+  spec.add_development_dependency "rspec",     ">= 3.12.0", "< 4"
+  spec.add_development_dependency "rubocop",   "~> 1"
+  spec.add_development_dependency "rubocop-rake", "~> 0"
+  spec.add_development_dependency "yard",      ">= 0.9.34", "< 1"
 
-  spec.add_dependency 'ecoportal-api',         '>= 0.9.8', '< 0.10'
-  spec.add_dependency 'ecoportal-api-v2',      '>= 1.1.8', '< 1.2'
-  spec.add_dependency 'ecoportal-api-graphql', '>= 0.3.18', '< 0.4'
-  spec.add_dependency 'ed25519', '>= 1.2'
-  spec.add_dependency 'bcrypt_pbkdf', '>= 1.0'
+  spec.add_dependency 'amatch',        '>= 0.4.1',  '< 0.5'
   spec.add_dependency 'aws-sdk-s3',    '>= 1.142.0', '< 2'
   spec.add_dependency 'aws-sdk-ses',   '>= 1.58.0', '< 2'
+  spec.add_dependency 'bcrypt_pbkdf',  '>= 1.0'
+  spec.add_dependency 'docx',          '>= 0.8.0',  '< 0.9'
   spec.add_dependency 'dotenv',        '>= 2.8.1',  '< 3'
-  spec.add_dependency 'net-ssh',       '>= 7.2.1',  '< 8'
-  spec.add_dependency 'net-sftp',      '>= 4.0.0',  '< 5'
-  spec.add_dependency 'hashdiff',      '>= 1.1.0',  '< 1.2'
+  spec.add_dependency 'ecoportal-api',         '>= 0.9.8', '< 0.10'
+  spec.add_dependency 'ecoportal-api-graphql', '>= 0.3.18', '< 0.4'
+  spec.add_dependency 'ecoportal-api-v2',      '>= 1.1.8', '< 1.2'
+  spec.add_dependency 'ed25519', '>= 1.2'
+  spec.add_dependency 'fast_excel',    '>= 0.5.0',  '< 0.6'
   spec.add_dependency 'fuzzy_match',   '>= 2.1.0',  '< 2.2'
-  spec.add_dependency 'amatch',        '>= 0.4.1',  '< 0.5'
+  spec.add_dependency 'hashdiff',      '>= 1.1.0',  '< 1.2'
   spec.add_dependency 'jaro_winkler',  '>= 1.5.6',  '< 1.6'
+  spec.add_dependency 'net-sftp',      '>= 4.0.0',  '< 5'
+  spec.add_dependency 'net-ssh',       '>= 7.2.1',  '< 8'
   spec.add_dependency 'nokogiri',      '>= 1.13',   '< 1.17'
   spec.add_dependency 'roo',           '>= 2.10.1', '< 2.11'
   spec.add_dependency 'roo-xls',       '>= 1.2.0',  '< 1.3'
-  spec.add_dependency 'fast_excel',    '>= 0.5.0',  '< 0.6'
-  spec.add_dependency 'docx',          '>= 0.8.0',  '< 0.9'
   spec.add_dependency 'rubyzip',       '>= 2.3.2',  '< 2.4'
 end

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

@@ -3,17 +3,20 @@ module Eco
     module Common
       module Session
         class Logger < Eco::Language::BasicLogger
-          TIMESTAMP_PATTERN = '%Y-%m-%dT%H:%M:%S'
+          TIMESTAMP_PATTERN = '%Y-%m-%dT%H:%M:%S'.freeze
 
           attr_reader :cache
 
           def initialize(file_level: ::Logger::DEBUG, log_file: nil, enviro: nil, **kargs)
             super(**kargs)
-            raise "Required Environment object (enviro:). Given: #{enviro}" if enviro && !enviro.is_a?(Eco::API::Common::Session::Environment)
+
+            msg = "Required Environment object (enviro:). Given: #{enviro.class}"
+            raise ArgumentError, msg if enviro && !enviro.is_a?(Eco::API::Common::Session::Environment)
+
             @enviro = enviro
             @cache  = Logger::Cache.new
 
-            if log_file = fetch_log_file(log_file)
+            if (log_file = fetch_log_file(log_file))
               loggers[:file] = ::Logger.new(log_file).tap do |logger|
                 logger.formatter = format_proc(console: false) do |severity, datetime, msg, formatted_msg|
                   cache.add(severity, datetime, msg, formatted_msg)
@@ -34,15 +37,16 @@ module Eco
           private
 
           def config(attr)
-            return nil unless cnf = @enviro&.config&.logger
+            return unless (cnf = @enviro&.config&.logger)
+
             cnf.send(attr) if cnf.respond_to?(attr)
           end
 
           def if_config(attr)
-            unless (value = config(attr)).nil?
-              yield(value) if block_given?
-              value
-            end
+            return if (value = config(attr)).nil?
+
+            yield(value) if block_given?
+            value
           end
 
           def fetch_log_file(log_file)

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

@@ -24,7 +24,7 @@ module Eco
           end
 
           def empty?
-            count == 0
+            count.zero?
           end
 
           def each(&block)
@@ -90,15 +90,16 @@ module Eco
           # @return [Hash<Eco::API::Session::Batch::Job, Eco::API::Session::Batch::Status>]
           def launch(simulate: false)
             each_with_index do |job, idx|
-              if job.pending?
-                status[job] = job_status = job.launch(simulate: simulate)
-                callback    = @callbacks[job]
-                callback.call(job, job_status) if callback
-                Eco::API::Session::Batch::JobsGroups.counter(delay_between_jobs) if !simulate && idx < self.length - 1
-              end
+              next unless job.pending?
+
+              status[job] = job_status = job.launch(simulate: simulate)
+              callback    = @callbacks[job]
+              callback&.call(job, job_status)
+              Eco::API::Session::Batch::JobsGroups.counter(delay_between_jobs) if !simulate && idx < length - 1
             end
             launch(simulate: simulate) if pending?
-            return status
+
+            status
           end
 
           def find_jobs(type:)
@@ -113,7 +114,7 @@ module Eco
                 yield(job, job_status)
               end
               self
-            else
+            else # rubocop:disable Naming/MemoizedInstanceVariableName
               @jobs_status ||= {}
             end
           end

data/lib/eco/api/usecases/default/people/{clean_unknown_tags_case.rb → amend/clean_unknown_tags_case.rb}

@@ -1,4 +1,4 @@
-class Eco::API::UseCases::Default::People::CleanUnknownTags < Eco::API::Common::Loaders::UseCase
+class Eco::API::UseCases::Default::People::Amend::CleanUnknownTags < Eco::API::Common::Loaders::UseCase
   name "clean-unknown-tags"
   type :transform
 
@@ -35,7 +35,7 @@ class Eco::API::UseCases::Default::People::CleanUnknownTags < Eco::API::Common::
   private
 
   def default_tag(person)
-    return nil unless account = person.account
+    return unless (account = person.account)
     account.default_tag
   end
 

data/lib/eco/api/usecases/default/people/{clear_abilities_case.rb → amend/clear_abilities_case.rb}

@@ -1,8 +1,8 @@
-class Eco::API::UseCases::Default::People::ClearAbilitiesTransCase < Eco::API::Common::Loaders::UseCase
+class Eco::API::UseCases::Default::People::Amend::ClearAbilitiesTransCase < Eco::API::Common::Loaders::UseCase
   name "clear-abilities"
   type :transform
 
-  def main(people, _session, _options, _usecase)
+  def main(*_args)
     clear_abilities(people.users)
   end
 

data/lib/eco/api/usecases/default/people/{refresh_case.rb → amend/refresh_case.rb}

@@ -1,4 +1,4 @@
-class Eco::API::UseCases::Default::People::RefreshCase < Eco::API::Common::Loaders::UseCase
+class Eco::API::UseCases::Default::People::Amend::RefreshCase < Eco::API::Common::Loaders::UseCase
   name "refresh"
   type :transform
 

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

@@ -0,0 +1,11 @@
+module Eco::API::UseCases::Default::People::Amend
+  class ReinviteSyncCase < ReinviteTransCase
+    name "reinvite"
+    type :sync
+
+    def main(entries, *_args)
+      found = micro.with_each_present(entries, people, options, log_starter: true)
+      reinvite(found.users)
+    end
+  end
+end

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

@@ -1,8 +1,8 @@
-class Eco::API::UseCases::Default::People::ReinviteTransCase < Eco::API::Common::Loaders::UseCase
+class Eco::API::UseCases::Default::People::Amend::ReinviteTransCase < Eco::API::Common::Loaders::UseCase
   name "reinvite"
   type :transform
 
-  def main(people, session, options, usecase)
+  def main(*_args)
     reinvite(people.users)
   end
 

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

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

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

@@ -1,6 +1,6 @@
 # @example
 #     -restore-db-from backup.json -skip-api-policies -skip-batch-policy -include-excluded -simulate
-class Eco::API::UseCases::Default::People::RestoreDBCase < Eco::API::Common::Loaders::UseCase
+class Eco::API::UseCases::Default::People::Amend::RestoreDBCase < Eco::API::Common::Loaders::UseCase
   name "restore-db"
   type :sync
 

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

@@ -0,0 +1,11 @@
+module Eco::API::UseCases::Default::People
+  module Amend
+  end
+end
+
+require_relative 'amend/clean_unknown_tags_case'
+require_relative 'amend/clear_abilities_case'
+require_relative 'amend/reinvite_trans_case'
+require_relative 'amend/reinvite_sync_case'
+require_relative 'amend/refresh_case'
+require_relative 'amend/restore_db_case'

data/lib/eco/api/usecases/default/people/migrate/cli/remap_tags_cli.rb

@@ -0,0 +1,23 @@
+class Eco::API::UseCases::Default::People::Migrate::RemapTags
+  class Cli < Eco::API::UseCases::Cli
+    desc 'Maps filter_tags and default_tag based on input file'
+
+    callback do |_people, _session, options, _usecase|
+      if (file = SCR.get_file('-remap-tags', required: true, should_exist: true))
+        options.deep_merge!(input: {file: {name: file}})
+      end
+    end
+
+    add_option('-remove-source-tag', 'Whether source mapped tags should be removed') do |options|
+      options.deep_merge!(usecase: {remove_source_tag: true})
+    end
+
+    add_option('-clear-unknown-tag', 'Whether unknown tags should be cleared') do |options|
+      options.deep_merge!(usecase: {clear_unknown_tags: true})
+    end
+
+    add_option('-clear-archived-nodes', 'Whether archived nodes should be cleared') do |options|
+      options.deep_merge!(usecase: {clear_archived_nodes: true})
+    end
+  end
+end

data/lib/eco/api/usecases/default/people/migrate/remap_tags_case.rb

@@ -0,0 +1,269 @@
+class Eco::API::UseCases::Default::People::Migrate::RemapTags < Eco::API::Common::Loaders::UseCase
+  name 'remap-tags'
+  type :transform
+
+  require_relative 'cli/remap_tags_cli'
+
+  REGISTER_TAGS = %w[
+    EVENT INJURY RISK CONTRACTOR PERMIT
+    AUDIT JSEA
+    TRAINING INDUCTION
+    MEETING PPE CHEMICAL
+    PLANT ASSET
+    POLICY IDEA REPORTS
+  ].freeze
+
+  def main(*_args)
+    tune_options!
+
+    people.each do |person|
+      update_job.add(person)
+
+      update_filter_tags!(person)
+      update_default_tag!(person)
+      clear_unkown_tags!(person)    if clear_unknown_tags?
+      clear_archived_nodes!(person) if clear_archived_nodes?
+    end
+  end
+
+  def update_job
+    @update_job ||= session.new_job('main', 'update', :update, usecase) do |_job, _job_status|
+      report_removed_src_tags
+      report_archived_cleared
+      report_unknown_cleared
+    end
+  end
+
+  private
+
+  # If the map on `value` exists, it returns the new value,
+  # otherwise mirrors the `value` (to prevent data loss)
+  def apply_map(value)
+    case value
+    when Array
+      value.map {|val| apply_map(val)}
+    when String
+      if (new_tag = id_maps[value])
+        new_tag
+      else
+        value
+      end
+    end
+  end
+
+  def update_filter_tags!(person)
+    mapped_tags = apply_map(person.filter_tags)
+
+    if remove_source_tag?
+      removed = person.filter_tags - mapped_tags
+      removed_src_tag!(*removed)
+      person.filter_tags = mapped_tags
+    else
+      person.filter_tags |= mapped_tags
+    end
+  end
+
+  # We always map the default_tag
+  def update_default_tag!(person)
+    return unless (account = person.account)
+
+    default_tag = apply_map(account.default_tag)
+    return unless default_tag.nil? || active_node?(default_tag)
+
+    account.default_tag = default_tag
+  end
+
+  # @note it still keeps archived nodes
+  def clear_unkown_tags!(person)
+    person.filter_tags = person.filter_tags.select do |tag|
+      known_tag?(tag).tap do |known|
+        cleared_unknown_tag!(tag) unless known
+      end
+    end
+  end
+
+  # @note it only targets archived nodes
+  def clear_archived_nodes!(person)
+    person.filter_tags = person.filter_tags.reject do |tag|
+      archived_node?(tag).tap do |known|
+        cleared_archived_node!(tag) unless known
+      end
+    end
+  end
+
+  def report_removed_src_tags
+    return if removed_src_tags.empty?
+
+    log(:info) {
+      msg  = "Here list of the #{removed_src_tags.count} removed source-mapped tags..."
+      msg << "\n  • "
+      msg << removed_src_tags.sort.join("\n  • ")
+      msg
+    }
+  end
+
+  # Clean OLD Tags
+  def report_unknown_cleared
+    report_cleared(cleared_unknown_tags, msg: 'Cleared up the following non existing tags:')
+  end
+
+  def report_archived_cleared
+    report_cleared(cleared_archived_nodes, msg: 'Cleared up the following archived nodes:')
+  end
+
+  # Whether it should remove the original tag
+  # @note we might want to preserve it at the moment, and remove them
+  #   at a later stage (i.e. after migrating the pages)
+  def remove_source_tag?
+    options.dig(:usecase, :remove_source_tag) || false
+  end
+
+  # Whether it should clear out any unkonwn tag
+  def clear_unknown_tags?
+    options.dig(:usecase, :clear_unknown_tags) || false
+  end
+
+  # Whether it should clear out any archived node
+  def clear_archived_nodes?
+    options.dig(:usecase, :clear_archived_nodes) || false
+  end
+
+  def report_cleared(data, msg:)
+    return if data.empty?
+
+    cleared_sorted = data.sort_by {|tag, count| [count * -1, tag]}
+
+    msg << "\n  • "
+    msg += cleared_sorted.map do |tag, count|
+      "'#{tag}' (#{count})"
+    end.join("\n  • ")
+
+    log(:info) { msg }
+    data.clear
+  end
+
+  def removed_src_tag!(*tags)
+    @removed_src_tags = removed_src_tags | tags
+  end
+
+  def removed_src_tags
+    @removed_src_tags ||= []
+  end
+
+  def cleared_unknown_tag!(tag)
+    cleared_unknown_tags[tag] ||= 0
+    cleared_unknown_tags[tag] += 1
+  end
+
+  def cleared_unknown_tags
+    @cleared_unknown_tags ||= {}
+  end
+
+  def cleared_archived_node!(tag)
+    cleared_archived_nodes[tag] ||= 0
+    cleared_archived_nodes[tag] += 1
+  end
+
+  def cleared_archived_nodes
+    @cleared_archived_nodes ||= {}
+  end
+
+  # @note validations on src -> dst location ids isn't done because
+  #   we might be in a case where we are moving from old locations to
+  #   location ids, but it might be a case where we are mapping an actual
+  #   location id to a custom tag as well... so trying to keep this standard.
+  def id_maps
+    @id_maps ||= input_csv.each.with_object({}) do |row, out|
+      from, to = row.fields.to_a.first(2)
+      froms    = from.split('|').map(&:strip).map(&:upcase).reject(&:empty?)
+      tos      = to.split('|').map(&:strip).map(&:upcase).reject(&:empty?)
+      from     = froms.first
+      to       = tos.first
+      next if from == to
+      next unless from && to
+
+      out[from] = to
+    end
+  end
+
+  def input_csv
+    @input_csv ||= Eco::CSV.read(input_file, encoding: input_encoding)
+  end
+
+  def input_encoding
+    options.dig(:input, :file, :encoding) || 'utf-8'
+  end
+
+  def input_file
+    @input_file ||= options.dig(:input, :file, :name).tap do |file|
+      if file.nil?
+        log(:warn) { "No input file specified" }
+        exit(1)
+      elsif File.exist?(file)
+        log(:info) { "Using input file '#{file}'" }
+      else
+        log(:error) { "File not found '#{file}'" }
+        exit(1)
+      end
+    end
+  end
+
+  def default_tag(person)
+    return unless (account = person.account)
+    account.default_tag
+  end
+
+  def known_tag?(value)
+    register_tag?(value) || tag?(value)
+  end
+
+  def unknown_tag?(value)
+    !known_tag?(value)
+  end
+
+  def register_tag?(value)
+    register_tags.include?(value)
+  end
+
+  def archived_tag?(value)
+    return false if register_tag?(value)
+    return false unless (node = tagtree.node(value))
+    node.archived
+  end
+
+  def tag?(value)
+    return false if value.nil?
+    return true  if tagtree.tag?(value)
+    register_tags.any? { |reg| value.upcase == reg }
+  end
+
+  def active_node?(value)
+    return false if value.nil?
+    return false unless tagtree.tag?(value)
+    !archived_node?(value)
+  end
+
+  def archived_node?(value)
+    return false unless (node = tagtree.node(value))
+    node.archived
+  end
+
+  # Get all the location structures merged into a single one
+  def tagtree
+    @tagtree ||= session.tagtree(
+      live:             true,
+      include_archived: true,
+      merge:            true
+    )
+  end
+
+  def register_tags
+    @register_tags ||= self.class::REGISTER_TAGS.compact.map(&:upcase)
+  end
+
+  def tune_options!
+    # options.deep_merge!(include: {excluded:     true})
+    options.deep_merge!(skip:    {api_policies: true})
+    options.deep_merge!(skip:    {batch_policy: true})
+  end
+end

data/lib/eco/api/usecases/default/people/migrate.rb

@@ -0,0 +1,6 @@
+module Eco::API::UseCases::Default::People
+  module Migrate
+  end
+end
+
+require_relative 'migrate/remap_tags_case'

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

@@ -1,4 +1,4 @@
-class Eco::API::UseCases::Default::People::Analyse < Eco::API::Common::Loaders::UseCase
+class Eco::API::UseCases::Default::People::Treat::Analyse < Eco::API::Common::Loaders::UseCase
   name "analyse-people"
   type :export
 

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

@@ -1,4 +1,4 @@
-class Eco::API::UseCases::Default::People::OrgDataConvertCase < Eco::API::Common::Loaders::UseCase
+class Eco::API::UseCases::Default::People::Treat::OrgDataConvertCase < Eco::API::Common::Loaders::UseCase
   name "org-data-convert"
   type :import
 

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

@@ -1,4 +1,4 @@
-class Eco::API::UseCases::Default::People::SupersCyclicIdentify < Eco::API::Common::Loaders::UseCase
+class Eco::API::UseCases::Default::People::Treat::SupersCyclicIdentify < Eco::API::Common::Loaders::UseCase
   name "identify-cyclic-supers"
   type :export
 
@@ -62,7 +62,7 @@ class Eco::API::UseCases::Default::People::SupersCyclicIdentify < Eco::API::Comm
 
     "".tap do |str|
       entry = set.shift
-      str << "  " * lev
+      str << ("  " * lev)
       str << lev&.positive? ? "+-#{lev}- " : ""
       str << entry.name
       str << " (#{entry.external_id}|#{entry.email}|#{entry.id})\n"