eco-helpers 2.1.5 → 2.1.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: eaeec07a9cf6c0b584a546d6626f661d73f608332a72ef938bcfe7513adc6f2c
-  data.tar.gz: 285234b6c42a281c900ae3c05dc5a71f9e6a55d4aa52a884457a6956a02c456e
+  metadata.gz: 57558123bd83d606799604772ef717673f58bd089efc90458116a3126b1742f4
+  data.tar.gz: 99ec5e1bf52f4cd3ce3bbab5da60c64bfb42aa7742dfbe85897b40bfd6c76471
 SHA512:
-  metadata.gz: bc26467b2cb2b102a03aff3e5c189c60d3d63290c107e69c609e5536e70631e50e1a2c21c5859260e048c132584288fc728fd185681df4540a3c67e822f581dc
-  data.tar.gz: 0a5a23a848dac8114d1945efac8f58508066de077cac286e4b6bd473965356ce2aa0b961b1b2e542183afb58f04b1ceeb441f14452e0477976b565c9ff414a51
+  metadata.gz: 4d5bbe4a67ec3b9f18025877ecc3ce8735a8e816be686a17deee58be2e9f1d1c35b54b73a004440210905b113230224c8ce9f2b225746e7bf7c276b274f67b5e
+  data.tar.gz: 3d492d4ead671bc288621135e197ee3ce42ed40a10b73546ca7b8f3c3b9d137c2048105beda464d2d7b0d443dde510f3c86edac0bdb68f5262ae434f063864f8

data/CHANGELOG.md

@@ -1,13 +1,29 @@
 # Change Log
 All notable changes to this project will be documented in this file.
 
-## [2.1.6] - 2022-10-xx
+## [2.1.8] - 2022-11-xx
 
 ### Added
 ### Changed
 ### Fixed
 
-## [2.1.5] - 2022-10-xx
+## [2.1.7] - 2022-11-17
+
+### Added
+  - `Eco::API::UseCases::OozeSamples::RegisterMigrationCase` method `excluded_field_hooks`
+    - This enables the ability to prevent fields from being paired/copied/written on
+
+### Changed
+  - **refactored** `#with_src_dst_field_pair` helper
+    - from `Eco::API::UseCases::OozeSamples::HelpersMigration::Copying`
+
+## [2.1.6] - 2022-11-17
+
+### Fixed
+  - `Eco::API::UseCases::OozeSamples::Helpers::Shortcuts#to_i` (missing param on method)
+  - `Eco::API::UseCases::OozeSamples::HelpersMigration::TypedFieldsPairing#multi_delete` typo
+
+## [2.1.5] - 2022-11-17
 
 ### Added
   - `Eco::API::UseCases::OozeSamples::RegisterUpdateCase` added creation count
@@ -16,7 +32,7 @@ All notable changes to this project will be documented in this file.
     - `with_rescue` that prompts user to continue
     - this is for usage in child classes
   - new use case and helpers thereof `Eco::API::UseCases::OozeSamples::RegisterMigrationCase`
-  
+
 ### Changed
   - upgraded `ecoportal-api-v2` **gem**
   - **refactored** `Eco::API::UseCases::OozeSamples::Helpers::Shortcuts#same_string?`

data/lib/eco/api/usecases/ooze_samples/helpers/shortcuts.rb

@@ -126,7 +126,7 @@ module Eco
               end
             end
 
-            def to_i
+            def to_i(value)
               Float(value).to_i
             end
 

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

@@ -9,8 +9,8 @@ class Eco::API::UseCases::OozeSamples
         session.logger
       end
 
-      def copy_generic_paired_fields(src, dst, exclude_ximport: false)
-        HelpersMigration::TypedFieldsPairing.new(src, dst, exclude_ximport: false).tap do |pairing|
+      def copy_generic_paired_fields(src, dst, src_excluded: [], dst_excluded: [], exclude_ximport: false)
+        HelpersMigration::TypedFieldsPairing.new(src, dst, src_excluded: src_excluded, dst_excluded: dst_excluded, exclude_ximport: false).tap do |pairing|
           pairing.each_pair do |src_fld, dst_fld|
             copy_field_content(src_fld, dst_fld)
             yield(src_fld, dst_fld, pairing) if block_given?
@@ -18,11 +18,11 @@ class Eco::API::UseCases::OozeSamples
         end
       end
 
-      def copy_hooked_fields(src, dst, hook:, type:, fields_tracker: nil)
+      def copy_hooked_fields(src, dst, hook:, type:, fields_tracker: nil, src_excluded: [], dst_excluded: [])
         case hook
         when String
-          with_src_dest_field_pair(src, dst, label: to_regex(hook), type: type) do |src_fld, dst_fld|
-            copy_field_content(src_fld, dst_fld, type).tap do |result|
+          with_src_dst_field_pair(src, dst, label: to_regex(hook), type: type, src_excluded: src_excluded, dst_excluded: dst_excluded) do |src_fld, dst_fld|
+            copy_field_content(src_fld, dst_fld).tap do |result|
               fields_tracker.resolve(src_fld, dst_fld) if fields_tracker
               yield(src_fld, dst_fld, fields_tracker) if block_given?
             end
@@ -30,8 +30,8 @@ class Eco::API::UseCases::OozeSamples
         when Hash
           hook.each do |hook_src, hook_dst|
             src_lab, dst_lab = to_regex(hook_src), to_regex(hook_dst)
-            with_src_dest_field_pair(src, dst, label: src_lab, label_dst: dst_lab, type: type) do |src_fld, dst_fld|
-              copy_field_content(src_fld, dst_fld, type).tap do |result|
+            with_src_dst_field_pair(src, dst, label: src_lab, label_dst: dst_lab, type: type, src_excluded: src_excluded, dst_excluded: dst_excluded) do |src_fld, dst_fld|
+              copy_field_content(src_fld, dst_fld).tap do |result|
                 fields_tracker.resolve(src_fld, dst_fld) if fields_tracker
                 yield(src_fld, dst_fld, fields_tracker) if block_given?
               end
@@ -44,33 +44,59 @@ class Eco::API::UseCases::OozeSamples
         /^#{str}$/i
       end
 
-      def with_src_dest_field_pair(src, dst, label: nil, label_dst: label, type: nil)
+      # @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: [])
         unless respond_to?(:with_fields, true)
           raise "These helpers are to be included in class cases inheriting from Eco::API::UseCases::OozeSamples::RegisterUpdateCase"
         end
         src_flds = with_fields(src, label: label, type: type)
         dst_flds = with_fields(dst, label: label_dst, type: type)
+        src_fld, dst_fld = src_flds.first, dst_flds.first
+
+        # Not found
+        if dst_flds.empty? || src_flds.empty?
+          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}')"
+          logger.warn(msg)
+          return nil, nil
+        end
+
+        # Fields exclusion
+        dst_flds -= dst_excluded unless dst_excluded.empty?
+        src_flds -= src_excluded unless src_excluded.empty?
+        src_fld, dst_fld = src_flds.first, dst_flds.first
+        return src_fld, dst_fld if dst_flds.empty? || src_flds.empty?
 
         if dst_flds.count > 1
           logger.warn("There are #{dst_flds.count} destination '#{type}' fields named '#{label_dst}' (source: '#{src.id}'). Using first...")
-        elsif dst_flds.empty?
-          logger.warn("There isn't a destination '#{type}' field named '#{label_dst}' (source: '#{src.id}')")
-          return nil, nil
         end
 
         if src_flds.count > 1
-          logger.warn("There are #{src_flds.count} source '#{type}' fields named '#{label}' (source: '#{src.id}'). Using first...")
-        elsif src_flds.empty?
-          logger.warn("There isn't a source '#{type}' field named '#{label}' (source: '#{src.id}')")
-          return nil, dst
+          msg = "There are #{src_flds.count} source '#{type}' fields named '#{label}' (source: '#{src.id}'). "
+          src_content = src_flds.reject(&:empty?)
+          if src_content.empty?
+            msg << "None with content. Using first..."
+          else
+            src_fld = src_content.first
+            if src_content.count == 1
+              msg << "Using the only one with content (idx: #{src_flds.index(src_fld)})..."
+            else
+              msg << "#{src_content.count} have content. Using first..."
+            end
+          end
+          logger.warn(msg)
         end
 
-        [src_flds.first, dst_flds.first].tap do |(src_fld, dst_fld)|
+        [src_fld, dst_fld].tap do |(src_fld, dst_fld)|
           yield(src_fld, dst_fld) if block_given?
         end
       end
 
-      def copy_field_content(src, dst, type = src.type)
+      def copy_field_content(src, dst)
+        type = src.type
         case type
         when "select"
           src.values.each {|val| dst.select(val)}

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

@@ -192,7 +192,7 @@ class Eco::API::UseCases::OozeSamples
               @dst_multi.delete(dst_fld) if pairs.empty?
             end
           end
-          src_dst_multi.delete(type) if type_pairs.empty?
+          src_dst_multi.delete(fld.type) if type_pairs.empty?
         end
       end
 

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

@@ -7,12 +7,17 @@
 #         1. field label name (String or RegExp body/source),
 #         2. or an object with key-value being source-destination label names (also supports RegExp body/source)
 #     - Example: {"select": ["severity of .*:", {"type of event:" => "type of incident:"}]}
+#     - "all" as type will target all type of fields.
+#       - Caution with this! Copying between fields of different types is not yet supported.
 #   - You may either
 #     1. define the `process_ooze` method calling super with a block: super.do ||
 #     2. of define the function `custom_processing(draft, source, fields_tracker: nil)`
 #   - You can define a callback function, `paired_fields_post_callback(src_fld, dst_fld, pairing_tracking = nil)`
 #     - It will be called after copying by using any of the pairing methods, and allows to introduce modifications
 #       such as selection option mappings
+#   - You may define a method called `excluded_field_hooks` with same syntax as the json pairing file
+#     - This will exclude as source and destination targets any fields matching those patterns.
+#     - This is specially useful to discard rich text field guidelines (i.e. `{"rich_text" => [".*guideline:"]}`)
 # This case expects `options[:source][:register_id]`
 class Eco::API::UseCases::OozeSamples::RegisterMigrationCase < Eco::API::UseCases::OozeSamples::RegisterUpdateCase
   name "register-migration-case"
@@ -101,8 +106,22 @@ class Eco::API::UseCases::OozeSamples::RegisterMigrationCase < Eco::API::UseCase
 
   def copy_pairings(src, dst)
     after_copy = callback(:paired_fields_post)
-    copy_generic_paired_fields(src, dst, exclude_ximport: self.class::EXCLUDE_XIMPORT, &after_copy).tap do |pairing_tracking|
-      copy_mapped_fields(src, dst, fields_tracker: pairing_tracking, &after_copy)
+    params = {
+      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)
+    end
+  end
+
+  def excluded_fields(entry)
+    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
+        excluded.push(*with_fields(entry, label: to_regex(hook), type: type))
+      end
     end
   end
 
@@ -112,11 +131,12 @@ class Eco::API::UseCases::OozeSamples::RegisterMigrationCase < Eco::API::UseCase
     raise msg
   end
 
-  def copy_mapped_fields(src, dst, fields_tracker: nil, &after_copy)
+  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
       field_hooks.each do |hook_name|
-        copy_hooked_fields(src, dst, hook: hook_name, type: type, fields_tracker: fields_tracker, &after_copy)
+        copy_hooked_fields(src, dst, hook: hook_name, type: type, **kargs, &after_copy)
       end
     end
   end

data/lib/eco/version.rb

@@ -1,3 +1,3 @@
 module Eco
-  VERSION = "2.1.5"
+  VERSION = "2.1.7"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: eco-helpers
 version: !ruby/object:Gem::Version
-  version: 2.1.5
+  version: 2.1.7
 platform: ruby
 authors:
 - Oscar Segura