eco-helpers 3.0.25 → 3.0.27

data/lib/eco/api/usecases/ooze_samples/helpers_migration/copying.rb

@@ -69,7 +69,8 @@ class Eco::API::UseCases::OozeSamples
       # @note
       #   - When used with `type: nil` it may pair fields of different type. There isn't refinement for this yet.
       def with_src_dst_field_pair(src, dst, label: nil, label_dst: label, type: nil, src_excluded: [], dst_excluded: []) # rubocop:disable Metrics/AbcSize
-        msg = "These helpers are to be included in class cases inheriting from Eco::API::UseCases::OozeSamples::RegisterUpdateCase"
+        msg  = "These helpers are to be included in class cases "
+        msg << "inheriting from Eco::API::UseCases::OozeSamples::RegisterUpdateCase"
         raise msg unless respond_to?(:with_fields, true)
 
         src_flds = with_fields(src, label: label, type: type)
@@ -82,7 +83,7 @@ class Eco::API::UseCases::OozeSamples
           msg     = "There isn't a '#{type}'"
           dst_msg = dst_flds.empty?? "destination field named '#{label_dst}'" : nil
           src_msg = src_flds.empty?? "source field named '#{label}'" : nil
-          msg    << " #{[dst_msg, src_msg].join(" or ")} (source: '#{src.id}')"
+          msg    << " #{[dst_msg, src_msg].join(' or ')} (source: '#{src.id}')"
 
           log(:warn) { msg }
           return nil, nil
@@ -132,52 +133,42 @@ class Eco::API::UseCases::OozeSamples
       def copy_field_content(src, dst) # rubocop:disable Metrics/AbcSize
         type = src.type
         case type
-        when "select"
+        when 'select'
           src.values.each {|val| dst.select(val)} # rubocop:disable Style/HashEachMethods
           dst.other_desc = src.other_desc if src.other && src.other_desc && dst.other
-        when "plain_text", "date", "number", "gauge"
+        when 'plain_text', 'date', 'number', 'gauge'
           dst.value = src.value
-        when "rich_text"
+        when 'rich_text'
           dst.content = src.content
-        when "cross_reference"
+        when 'cross_reference'
           src.reference_ids.each {|id| dst.add(id)}
-        when "people"
+        when 'people'
           dst.people_ids << src.people_ids.to_a
-        when "geo"
+        when 'geo'
           src_coo = src.coordinates
           dst_coo = dst.coordinates
           dst_coo.lat, dst_coo.lon = src_coo.lat, src_coo.lon # rubocop:disable Style/ParallelAssignment
-        when "file"
+        when 'file'
           src.items.each do |src_item|
             dst.add_file(src_item.file_container_id) do |dst_item|
               dst_item.label = src_item.label
             end
           end
-        when "image_gallery"
+        when 'image_gallery'
           src.images.each do |src_image|
             dst.add_image(src_image.upload_id) do |dst_image|
               dst_image.caption = src_image.caption
             end
           end
-        when "checklist"
+        when 'checklist'
           src.items.each do |src_item|
             dst.add_item(label: src_item.label) do |dst_item|
               dst_item.checked = src_item.checked
             end
           end
-        when "law"
+        when 'law', 'page_action', 'actions_list', 'signature'
           "won't copy"
-        when "page_action"
-          "won't copy"
-        when "actions_list"
-          "won't copy"
-        when "signature"
-          "can't copy"
-        when "tag_field"
-          "can't copy"
-        when "chart"
-          "won't copy"
-        when "frequency_rate_chart"
+        when 'tag_field', 'chart', 'frequency_rate_chart'
           "won't copy"
         end
       end

data/lib/eco/api/usecases/ooze_samples/helpers_migration/typed_fields_pairing.rb

@@ -12,7 +12,7 @@ class Eco::API::UseCases::OozeSamples
     class TypedFieldsPairing
       include Eco::API::UseCases::OozeSamples::Helpers::Shortcuts
 
-      EXCLUDED_TYPES = ["tag_field", "chart", "frequency_rate_chart"]
+      EXCLUDED_TYPES = %w[tag_field chart frequency_rate_chart].freeze
 
       attr_reader :src, :dst
       attr_reader :src_unpaired, :dst_unpaired
@@ -20,11 +20,12 @@ class Eco::API::UseCases::OozeSamples
 
       def initialize(source, destination, src_excluded: nil, dst_excluded: nil,
           exclude_ximport: false)
-        @src = source
-        @dst = destination
-        @src_excluded = src_excluded || []
-        @dst_excluded = dst_excluded || []
-        @exclude_ximport = !!exclude_ximport
+        @src             = source
+        @dst             = destination
+        @src_excluded    = src_excluded || []
+        @dst_excluded    = dst_excluded || []
+        @exclude_ximport = exclude_ximport
+
         reset!
         process(exact: true)
         process
@@ -37,28 +38,36 @@ class Eco::API::UseCases::OozeSamples
       end
 
       def report_src_unpaired(only_present: true, only_indexed: false)
-        return nil if src_unpaired.empty?
-        msg  = "Source entry with unpaired fields (#{object_reference(src)}):\n"
+        return if src_unpaired.empty?
+
+        msg = "Source entry with unpaired fields (#{object_reference(src)}):\n"
+
         src_filtered = src_unpaired.values.flatten.reject do |src_fld|
           only_present && src_fld.empty?
         end.reject do |src_fld|
           only_indexed && src_fld.deindex
         end
-        return nil if src_filtered.empty?
+
+        return if src_filtered.empty?
+
         msg << src_filtered.map do |src_fld|
           "  * #{object_reference(src_fld)}"
         end.join("\n")
       end
 
       def report_dst_unpaired(only_required: false, exclude_ximport: true)
-        return nil if dst_unpaired.empty?
-        msg  = "Destination entry with unpaired fields (#{object_reference(src)}):\n"
+        return if dst_unpaired.empty?
+
+        msg = "Destination entry with unpaired fields (#{object_reference(src)}):\n"
+
         dst_filtered = dst_unpaired.values.flatten.reject do |dst_fld|
           only_required && !dst_fld.required
         end.reject do |dst_fld|
           exclude_ximport && ximport?(dst_fld)
         end
-        return nil if dst_filtered.empty?
+
+        return if dst_filtered.empty?
+
         msg << dst_filtered.map do |dst_fld|
           "  * #{object_reference(dst_fld)}"
         end.join("\n")
@@ -66,23 +75,27 @@ class Eco::API::UseCases::OozeSamples
 
       def report_multi_pairs(only_present: true, only_indexed: false, dst_exclude_ximport: true)
         return nil if src_dst_multi.empty?
-        "".tap do |msg|
-          msg << "Source fields in (#{object_reference(src)}) paired with multiple destination fields:\n"
-          msg_flds = ""
+        ''.tap do |msg|
+          msg     << "Source fields in (#{object_reference(src)}) paired with multiple destination fields:\n"
+          msg_flds = ''
           src_dst_multi.each do |type, pairs|
             pairs.each do |(src_fld, dst_flds)|
               next if (only_present && src_fld.empty?) || (only_indexed && src_fld.deindex)
+
               dst_filtered = dst_flds.reject do |dst_fld|
                 dst_exclude_ximport && ximport?(dst_fld)
               end
               next if dst_filtered.empty?
+
               msg_flds << "  * #{object_reference(src_fld)}\n"
               msg_flds << dst_filtered.map do |dst_fld|
                 "    * #{object_reference(dst_fld)}"
               end.compact.join("\n")
             end
           end
+
           return nil if msg_flds.empty?
+
           msg << msg_flds
         end
       end
@@ -91,8 +104,10 @@ class Eco::API::UseCases::OozeSamples
       def resolve(*flds)
         flds.each do |fld|
           next if unpaired_delete(fld, src_unpaired)
+
           unpaired_delete(fld, dst_unpaired)
         end
+
         multi_delete(*flds)
       end
 
@@ -120,11 +135,13 @@ class Eco::API::UseCases::OozeSamples
       def process(exact: false)
         src_unpaired.each do |type, src_pend|
           next unless (dst_pend = dst_unpaired[type]) && !dst_pend.empty?
+
           src_pend.dup.each do |src_fld|
             dst_candidates = dst_pend.select do |dst_fld|
               str_eq?(src_fld.label, dst_fld.label, exact: exact)
             end
             next if dst_candidates.empty?
+
             if dst_candidates.count == 1
               pair_add(src_fld, dst_candidates.first)
             else
@@ -134,14 +151,14 @@ class Eco::API::UseCases::OozeSamples
         end
       end
 
-      def str_eq?(str1, str2, exact: false)
-        same_string?(str1, str2, **str_eq_params.merge(exact: exact))
+      def str_eq?(str_1, str_2, exact: false)
+        same_string?(str_1, str_2, **str_eq_params, exact: exact)
       end
 
       def str_eq_params
         {
-          exact: exact?,
-          mild: only_letters?,
+          exact:  exact?,
+          mild:   only_letters?,
           ignore: false
         }.tap do |params|
           params.merge!(ignore: bracked_regex) if non_bracked?
@@ -158,13 +175,15 @@ class Eco::API::UseCases::OozeSamples
         type = fld.type
         return false unless unpaired.key?(type)
         return false unless (flds = unpaired[type]).delete(fld)
+
         unpaired.delete(type) if flds.empty?
         true
       end
 
       def multi_add(type, src_fld, *dst_flds)
         (src_dst_multi[type] ||= []) << (pair = [src_fld, dst_flds])
-        @src_multi[src_fld] = pair
+        @src_multi[src_fld]          = pair
+
         dst_flds.each do |dst_fld|
           (@dst_multi[dst_fld] ||= []) << pair
         end
@@ -172,8 +191,9 @@ class Eco::API::UseCases::OozeSamples
 
       def multi_delete(*flds)
         flds.each do |fld|
-          next unless type_pairs = src_dst_multi[fld.type]
-          if pairs = @dst_multi.delete(fld)
+          next unless (type_pairs = src_dst_multi[fld.type])
+
+          if (pairs = @dst_multi.delete(fld))
             pairs.each do |pair|
               src_fld = pair.first
               dst_flds = pair.last
@@ -186,12 +206,14 @@ class Eco::API::UseCases::OozeSamples
           elsif @src_multi.key?(fld)
             pair = @src_multi.delete(fld)
             type_pairs.delete(pair)
+
             pair.last.each do |dst_fld|
               pairs = @dst_multi[dst_fld]
               pairs.delete(pair)
               @dst_multi.delete(dst_fld) if pairs.empty?
             end
           end
+
           src_dst_multi.delete(fld.type) if type_pairs.empty?
         end
       end
@@ -215,20 +237,20 @@ class Eco::API::UseCases::OozeSamples
       end
 
       def ximport?(fld)
-        fld && fld.tooltip.to_s.include?("ximport")
+        fld && fld.tooltip.to_s.include?('ximport')
       end
 
       def reset!
-        @src_unpaired = by_type(src_components)
-        @dst_unpaired = by_type(dst_components)
+        @src_unpaired  = by_type(src_components)
+        @dst_unpaired  = by_type(dst_components)
         @src_dst_pairs = []
         reset_multi!
       end
 
       def reset_multi!
         @src_dst_multi = {}
-        @src_multi = {}
-        @dst_multi = {}
+        @src_multi     = {}
+        @dst_multi     = {}
       end
     end
   end

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

@@ -1,6 +1,6 @@
 # @attr_reader target [Ecoportal::API::V2::Page] current target ooze.
 class Eco::API::UseCases::OozeSamples::OozeBaseCase < Eco::API::Common::Loaders::UseCase
-  name "ooze-base"
+  name 'ooze-base'
   type :other
 
   include Eco::API::UseCases::OozeSamples::Helpers
@@ -8,7 +8,7 @@ class Eco::API::UseCases::OozeSamples::OozeBaseCase < Eco::API::Common::Loaders:
 
   attr_reader :target
 
-  SAVE_PATCH = "ooze_patch_update.json".freeze
+  SAVE_PATCH = 'ooze_patch_update.json'.freeze
   DRY_COUNT  = 5
 
   def main(_session, options, _usecase)
@@ -37,7 +37,7 @@ class Eco::API::UseCases::OozeSamples::OozeBaseCase < Eco::API::Common::Loaders:
   def create_page(page = nil, template_id: nil, retrieve: false)
     template_id = template_id.id        if template_id.is_a?(Ecoportal::API::V2::Page)
     template_id ||= page.template_id    if page.respond_to?(:template_id)
-    template_id ||= page["template_id"] if page.is_a?(Hash)
+    template_id ||= page['template_id'] if page.is_a?(Hash)
     page = new_page(template_id)        unless page || !template_id
 
     yield(page) if block_given?
@@ -190,8 +190,8 @@ class Eco::API::UseCases::OozeSamples::OozeBaseCase < Eco::API::Common::Loaders:
   def new_target(object, warn_pending_changes: true)
     if dirty?(target)
       if warn_pending_changes
-        msg =  "You you are switching to a new target #{object_reference(object)}"
-        msg += " after doing unsaved changes to #{object_reference(target)}"
+        msg  =  "You you are switching to a new target #{object_reference(object)}"
+        msg << " after doing unsaved changes to #{object_reference(target)}"
         log(:warn) { msg }
       end
 
@@ -202,17 +202,17 @@ class Eco::API::UseCases::OozeSamples::OozeBaseCase < Eco::API::Common::Loaders:
   end
 
   def dirty?(object)
-    object && patch_doc(object)["page"]
+    object && patch_doc(object)['page']
   end
 
   def backup_patch!(ooze = target)
-    unless (patch = (patch_doc(ooze) || {})["page"])
+    unless (patch = (patch_doc(ooze) || {})['page'])
       log(:info) { "No changes to update for #{object_reference(ooze)}." }
       return
     end
 
     # store the request
-    File.open(SAVE_PATCH, "w") do |file|
+    File.open(SAVE_PATCH, 'w') do |file|
       #file << (patch_doc || {}).to_json
       file << JSON.pretty_generate(patch || {})
     end
@@ -222,7 +222,7 @@ class Eco::API::UseCases::OozeSamples::OozeBaseCase < Eco::API::Common::Loaders:
 
   def display_patch(entry = target)
     return if options.dig(:feedback, :only_stats)
-    return unless (patch = (patch_doc(entry) || {})["page"])
+    return unless (patch = (patch_doc(entry) || {})['page'])
 
     pp patch
   end

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

@@ -6,7 +6,7 @@ class Eco::API::UseCases::OozeSamples::OozeRunBaseCase < Eco::API::UseCases::Ooz
   SAVE_PATCH = 'ooze_patch_update.json'.freeze
 
   def main(session, options, usecase)
-    super(session, options, usecase) do
+    super do
       if method(:process_ooze).parameters.count.zero?
         ooze # retrieve ooze
         process_ooze

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

@@ -15,7 +15,7 @@ class Eco::API::UseCases::OozeSamples::RegisterExportCase < Eco::API::Common::Lo
 
   include Eco::API::UseCases::OozeSamples::Helpers
 
-  name "register-export-case"
+  name 'register-export-case'
   type :other
 
   attr_reader :session, :options, :usecase
@@ -58,29 +58,29 @@ class Eco::API::UseCases::OozeSamples::RegisterExportCase < Eco::API::Common::Lo
   # @param ooze_id [String]
   # @return [Ecoportal::API::V2::Page]
   def build_full_ooze(ooze_id) # rubocop:disable Metrics/AbcSize
-    return unless (page = ooze(ooze_id))
+    return      unless (page = ooze(ooze_id))
     return page unless page.is_a?(Ecoportal::API::V2::Pages::PageStage)
 
     pending_stage_ids = page.stages.ordered.map(&:id) - [page.current_stage_id]
     state    = page.state
     secs_doc = page.sections.doc
     flds_doc = page.components.doc
-    hsecs    = secs_doc.each_with_object({}) {|sec, h| h[sec["id"]] = sec}
-    hflds    = flds_doc.each_with_object({}) {|fld, h| h[fld["id"]] = fld}
+    hsecs    = secs_doc.each_with_object({}) {|sec, h| h[sec['id']] = sec}
+    hflds    = flds_doc.each_with_object({}) {|fld, h| h[fld['id']] = fld}
 
     pending_stage_ids.each do |id|
       next unless (other_stage = stage(id, ooze: page))
 
-      state = "inprogress" unless other_stage.state == "complete"
+      state = 'inprogress' unless other_stage.state == 'complete'
       other_stage.sections.doc.each do |sec_doc|
-        unless hsecs.key?(id = sec_doc["id"])
+        unless hsecs.key?(id = sec_doc['id'])
           hsecs[id] = sec_doc
           secs_doc << sec_doc
         end
       end
 
       other_stage.components.doc.each do |comp_doc|
-        unless hflds.key?(id = comp_doc["id"])
+        unless hflds.key?(id = comp_doc['id'])
           hflds[id] = comp_doc
           flds_doc << comp_doc
         end
@@ -93,21 +93,27 @@ class Eco::API::UseCases::OozeSamples::RegisterExportCase < Eco::API::Common::Lo
   end
 
   def batched_search_results
-    raise "Missing block. It yields in slices of #{self.class.batch_size} results" unless block_given?
+    msg = "Missing block. It yields in slices of #{self.class.batch_size} results"
+    raise msg unless block_given?
+
     results_preview
     results = []
+
     apiv2.registers.search(register_id, search_options) do |page_result|
       results << page_result
+
       if results.length >= self.class.batch_size
         yield(results)
         results = []
       end
     end
+
     yield(results) unless results.empty?
   end
 
   def ooze(ooze_id = nil, stage_id: nil)
     return target unless ooze_id
+
     apiv2.pages.get(ooze_id, stage_id: stage_id).tap do |ooze|
       if ooze
         new_target(ooze)
@@ -143,21 +149,22 @@ class Eco::API::UseCases::OozeSamples::RegisterExportCase < Eco::API::Common::Lo
   end
 
   def default_proceed
-    "Y"
+    'Y'
   end
 
   def results_preview
     apiv2.registers.search(register_id, search_options.merge(only_first: true)).tap do |search_results|
       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):",
+        'Do you want to proceed (y/N):',
         explanation: str_results,
         default:     default_proceed,
         timeout:     10
       ) do |res|
-        next if res.upcase.start_with?("Y")
-        puts "..."
-        log(:info) { "Aborting script..." }
+        next if res.upcase.start_with?('Y')
+
+        puts '...'
+        log(:info) { 'Aborting script...' }
         exit(0)
       end
     end
@@ -165,20 +172,21 @@ class Eco::API::UseCases::OozeSamples::RegisterExportCase < Eco::API::Common::Lo
 
   def search_options
     @search_options ||= {}.tap do |opts|
-      opts.merge!(sort: "created_at")
-      opts.merge!(dir:  "asc")
+      opts.merge!(sort: 'created_at')
+      opts.merge!(dir:  'asc')
       opts.merge!(query:   conf_search) if conf_search
       opts.merge!(filters: conf_filters)
     end
   end
 
   def conf_filters
-    return filters if respond_to?(:filters)
+    return filters if respond_to?(:filters, true)
+
     []
   end
 
   def conf_search
-    search if respond_to?(:search)
+    search if respond_to?(:search, true)
   end
 
   def register_id

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

@@ -47,9 +47,9 @@ class Eco::API::UseCases::OozeSamples::RegisterMigrationCase < Eco::API::UseCase
       @csv = []
       super(session, options, usecase, &block)
     else
-      CSV.open(out_csv_filename, "w") do |csv|
+      CSV.open(out_csv_filename, 'w') do |csv|
         @csv = csv
-        csv << ["src_page_id", "dst_page_id"]
+        csv << %w[src_page_id dst_page_id]
         super(session, options, usecase, &block)
       end
       log(:info) { "File '#{out_csv_filename}' has been created." }
@@ -64,11 +64,11 @@ class Eco::API::UseCases::OozeSamples::RegisterMigrationCase < Eco::API::UseCase
   #   # More custom logics
   # end
 
-  def process_ooze
+  def process_ooze # rubocop:disable Metrics/AbcSize
     with_target_stage(self.class::SOURCE_STAGE) do |source|
       draft_reference = object_reference(source)
       creating_new_page(draft_reference, template_id: elf.class::TEMPLATE_ID) do |draft|
-        draft.base_tags << ["IMPORTED", "MIGRATED"]
+        draft.base_tags << %w[IMPORTED MIGRATED]
 
         pairing_tracking = copy_pairings(source, draft)
 
@@ -78,17 +78,24 @@ class Eco::API::UseCases::OozeSamples::RegisterMigrationCase < Eco::API::UseCase
           yield(draft, source, fields_tracker: pairing_tracking)
         end
 
-        if msg = pairing_tracking.report_src_unpaired(only_present: !self.class::REPORT_SRC_EMTPY, only_indexed: !self.class::REPORT_SRC_DEINDEX)
-          log(:warn) { msg }
-        end
-
-        if msg = pairing_tracking.report_multi_pairs(only_present: !self.class::REPORT_SRC_EMTPY, only_indexed: !self.class::REPORT_SRC_DEINDEX, dst_exclude_ximport: !self.class::REPORT_DST_XIMPORT)
-          log(:warn) { msg }
-        end
-
-        if msg = pairing_tracking.report_dst_unpaired(only_required: false, exclude_ximport: !self.class::REPORT_DST_XIMPORT)
-          log(:warn) { msg }
-        end
+        msg = pairing_tracking.report_src_unpaired(
+          only_present: !self.class::REPORT_SRC_EMTPY,
+          only_indexed: !self.class::REPORT_SRC_DEINDEX
+        )
+        log(:warn) { msg } if msg
+
+        msg = pairing_tracking.report_multi_pairs(
+          only_present:        !self.class::REPORT_SRC_EMTPY,
+          only_indexed:        !self.class::REPORT_SRC_DEINDEX,
+          dst_exclude_ximport: !self.class::REPORT_DST_XIMPORT
+        )
+        log(:warn) { msg } if msg
+
+        msg = pairing_tracking.report_dst_unpaired(
+          only_required:   false,
+          exclude_ximport: !self.class::REPORT_DST_XIMPORT
+        )
+        log(:warn) { msg } if msg
       end
     end
   end
@@ -101,8 +108,18 @@ class Eco::API::UseCases::OozeSamples::RegisterMigrationCase < Eco::API::UseCase
       src_excluded: excluded_fields(src),
       dst_excluded: excluded_fields(dst)
     }
-    copy_generic_paired_fields(src, dst, **params.merge(exclude_ximport: self.class::EXCLUDE_XIMPORT), &after_copy).tap do |pairing_tracking|
-      copy_mapped_fields(src, dst, **params.merge(fields_tracker: pairing_tracking), &after_copy)
+    copy_generic_paired_fields(
+      src,
+      dst,
+      **params.merge(exclude_ximport: self.class::EXCLUDE_XIMPORT),
+      &after_copy
+    ).tap do |pairs_track|
+      copy_mapped_fields(
+        src,
+        dst,
+        **params.merge(fields_tracker: pairs_track),
+        &after_copy
+      )
     end
   end
 
@@ -110,7 +127,8 @@ class Eco::API::UseCases::OozeSamples::RegisterMigrationCase < Eco::API::UseCase
     return [] unless respond_to?(:excluded_field_hooks, true)
     excluded_field_hooks.each_with_object([]) do |(type, hooks), excluded|
       hooks.each do |hook|
-        type = (type == "all")? nil : type
+        type = nil if type == 'all'
+
         excluded.push(*with_fields(entry, label: to_regex(hook), type: type))
       end
     end
@@ -125,7 +143,8 @@ class Eco::API::UseCases::OozeSamples::RegisterMigrationCase < Eco::API::UseCase
   def copy_mapped_fields(src, dst, **kargs, &after_copy)
     return [] unless field_maps?
     field_maps.each do |type, field_hooks|
-      type = (type == "all")? nil : type
+      type = nil if type == 'all'
+
       field_hooks.each do |hook_name|
         copy_hooked_fields(src, dst, hook: hook_name, type: type, **kargs, &after_copy)
       end
@@ -137,17 +156,15 @@ class Eco::API::UseCases::OozeSamples::RegisterMigrationCase < Eco::API::UseCase
   end
 
   def callback(sym)
-    if callback?(sym)
-      method(callback_name(sym))
-    end
+    method(callback_name(sym)) if callback?(sym)
   end
 
   def callback_name(sym)
-    "#{sym}_callback".to_sym
+    :"#{sym}_callback"
   end
 
   def with_target_stage(name)
-    if stg = target_stage(name)
+    if (stg = target_stage(name))
       yield(stg)
     else
       log(:warn) { "Entry '#{target.id}' does not have stage '#{name}'" }