eco-helpers 2.0.33 → 2.0.34

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 84ad4121fdba090ce8585df6a999c91a0a1ba5c8fa67d52f6dc4ec5d39da18b9
-  data.tar.gz: 96e540e97b6a54a92e246fd794821002ab015c9199d033bcb22abbc2bd729a69
+  metadata.gz: 420f5871108edb8c66a534f60a4f8303352245a5080c68398ccd1dc34974a606
+  data.tar.gz: c176a43c046adf47520d18af6db7a92225786625051f2982b6ffdd1540c722e7
 SHA512:
-  metadata.gz: a6a3c9b2920974830e3d9ea1619f9f4e2f8652112d0b0683526888cd9d29c29c61e35afe53d94bd0927ddb4b8bd621827cdf5a8a18830143f6549f7b3d8fc37d
-  data.tar.gz: ceaf87bc2b6190f70c91f9ad09da865ecb064bd1be9f783a75cb91f531a9918ee80a1d583ba74c21ab56ac9863eccd6d6c3321b78988961762335e80036f4ed6
+  metadata.gz: 1e1b2828dc5ba988364eb04f9433fff624694d6371ba5e3aa80bc4d4ee3c6cc62eb90cd2693a1e324a630043e19666d1499e5e8fdfff69c03adccd777e387494
+  data.tar.gz: 5b2d6ba34b14b4164512e3ffd5c65ab2c4edac3c0452ff4dcd422b32537fcff47c244f85b815c4f55b0d313b546cf786dabe49d76cc5feaf374b1039cdc7c266

data/CHANGELOG.md

@@ -1,7 +1,21 @@
 # Change Log
 All notable changes to this project will be documented in this file.
 
-## [2.0.33] - 2021-08-xx
+## [2.0.34] - 2021-08-xx
+
+### Added
+  - `Eco::API::UseCases::OozeSamples::OozeBaseCase` refactored ooze base case
+  - `Eco::API::UseCases::OozeSamples::RegisterUpdateCase` new case for full register update
+
+### Changed
+  - `Eco::API::UseCases::OozeSamples::OozeRunBaseCase` made child of `OozeBaseCase` and simplified to its essential
+  - `Eco::API::UseCases::OozeSamples::OozeUpdateCase` remove unnecessary code and helpers that are now inherited from `OoozeBaseCase`
+  - `Eco::API::UseCases::OozeSamples::OozeFromDocCase` remove unnecessary code
+
+### Fixed
+
+
+## [2.0.33] - 2021-08-16
 
 ### Added
   - `Eco::API::UseCases::OozeSamples::OozeFromDocCase` use case to abstract form from doc

data/eco-helpers.gemspec

@@ -31,7 +31,7 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency "redcarpet", ">= 3.5.1",  "< 3.6"
 
   spec.add_dependency 'ecoportal-api', '>= 0.8.3', '< 0.9'
-  spec.add_dependency 'ecoportal-api-v2', '>= 0.8.9', '< 0.9'
+  spec.add_dependency 'ecoportal-api-v2', '>= 0.8.11', '< 0.9'
   spec.add_dependency 'aws-sdk-s3',    '>= 1.83.0', '< 2'
   spec.add_dependency 'aws-sdk-ses',   '>= 1.36.0', '< 2'
   spec.add_dependency 'dotenv',        '>= 2.7.6', '< 2.8'

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

@@ -3,7 +3,7 @@ class Eco::API::UseCases::DefaultCases::UpsertCase < Eco::API::Common::Loaders::
   type :sync
 
   attr_reader :options
-  
+
   def main(entries, people, session, options, usecase)
     @options = options
     micro    = session.micro

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

@@ -54,6 +54,21 @@ module Eco
               end
             end
 
+            def object_reference(obj)
+              return "No reference" unless obj
+              "".tap do |ref|
+                case obj
+                when Ecoportal::API::V2::Page::Stage
+                  ref << "Stage "
+                when Ecoportal::API::V2::Pages::PageStage
+                  ref << "StagePage "
+                when Ecoportal::API::V2::Page
+                  ref << "Page "
+                end
+                ref << "'#{obj.name}' " if obj.respond_to?(:name)
+              end
+            end
+            
           end
         end
       end

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

@@ -0,0 +1,125 @@
+# @attr_reader target [Ecoportal::API::V2::Page] current target ooze.
+class Eco::API::UseCases::OozeSamples::OozeBaseCase < Eco::API::Common::Loaders::UseCase
+  name "ooze-base"
+  type :other
+
+  include Eco::API::UseCases::OozeSamples::Helpers
+
+  attr_reader :session, :options, :usecase
+  attr_reader :target
+
+  SAVE_PATCH = "ooze_patch_update.json"
+
+  def main(session, options, usecase)
+    raise "You need to inherit from this class ('#{self.class}') and call super with a block" unless block_given?
+    @session = session; @options = options; @usecase = usecase
+    @target  = nil
+    yield
+  end
+
+  # Write here your script
+  def process_ooze(ooze = target)
+    raise "You need to inherit from this class ('#{self.class}') and call super with a block" unless block_given?
+    yield(ooze)
+    update_ooze(ooze)
+  end
+
+  private
+
+  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)
+        logger.info("Got #{object_reference(ooze)}")
+      else
+        exit_error "Could not get ooze '#{ooze_id}'"
+      end
+    end
+  end
+
+  def stage(id_name = nil, ooze: target)
+    if ooze_id = ooze && ooze.id
+      exit_error "#{object_reference(ooze)} does not have stages!" unless ooze.stages?
+    else
+      exit_error "There's no target ooze to get retrieve stages from"
+    end
+
+    if stg = ooze.stages[id_name] || ooze.stages.get_by_name(id_name)
+      return ooze if ooze.respond_to?(:current_stage_id) && (ooze.current_stage_id == stg.id)
+      return apiv2.pages.get(ooze_id, stage_id: stg.id).tap do |stage|
+        if stage
+          new_target(stage)
+          logger.info("Got #{object_reference(stage)} from #{object_reference(ooze)}")
+        else
+          exit_error "Could not get stage '#{id_name}' in ooze '#{ooze_id}'"
+        end
+      end
+    end
+    exit_error "Stage '#{id_name}' doesn't exist in ooze '#{ooze_id}'"
+  end
+
+  def update_ooze(ooze = target)
+    if !options[:simulate]
+      apiv2.pages.update(ooze).tap do |response|
+        if response.success?
+          ooze.consolidate!
+          logger.info("Successful update of '#{ooze.name}' (#{ooze.id})")
+        else
+          logger.error("Could not update '#{ooze.name}' (#{ooze.id}): #{response.body}")
+        end
+      end
+    else
+      unless options.dig(:feedback, :only_stats)
+        if patch = (patch_doc(ooze) || {})["page"]
+          pp patch
+        end
+      end
+      backup_patch!(ooze)
+      exit(0) if dry_count > 5
+    end
+  end
+
+  def new_target(object)
+    if target && patch_doc(target)["page"]
+      logger.warn "You you are switching to a new target #{object_reference(object)} after doing unsaved changes to #{object_reference(target)}"
+    end
+    @target = object
+  end
+
+  def backup_patch!(ooze = target)
+    unless patch = (patch_doc(ooze) || {})["page"]
+      logger.info "No changes to update for #{object_reference(ooze)}."
+      return
+    end
+
+    patch = patch_doc(ooze)
+    # store the request
+    File.open(SAVE_PATCH, "w") do |file|
+      #file << (patch_doc || {}).to_json
+      file << JSON.pretty_generate(patch || {})
+    end
+    # TODO: ooze_requests folder with subfolders: for each run
+    puts "Saved patch at: #{File.expand_path(SAVE_PATCH)}"
+  end
+
+  def patch_doc(ooze = target)
+    apiv2.pages.get_body(ooze)
+  end
+
+  def dry_count
+    @dry_count ||= 0
+    @dry_count += 1
+  end
+
+  def apiv2
+   @apiv2 ||= session.api(version: :oozes)
+  end
+
+  def exit_error(msg)
+    logger.error(msg)
+    exit(1)
+  end
+
+
+end

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

@@ -4,13 +4,6 @@ class Eco::API::UseCases::OozeSamples::OozeFromDocCase < Eco::API::UseCases::Ooz
   name "ooze-forms-case"
   type :other
 
-  def main(session, options, usecase)
-    raise "You need to inherit from this class and call super with a block" unless block_given?
-    super(session, options, usecase) do
-      yield
-    end
-  end
-
   private
 
   def with_column(num = 0)

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

@@ -1,4 +1,5 @@
-class Eco::API::UseCases::OozeSamples::OozeRunBaseCase < Eco::API::Common::Loaders::UseCase
+# Base class to run a script against a single ooze.
+class Eco::API::UseCases::OozeSamples::OozeRunBaseCase < Eco::API::UseCases::OozeSamples::OozeBaseCase
   name "ooze-run-base"
   type :other
 
@@ -6,20 +7,28 @@ class Eco::API::UseCases::OozeSamples::OozeRunBaseCase < Eco::API::Common::Loade
 
   SAVE_PATCH = "ooze_patch_update.json"
 
-  def main(session, options, usecase)
-    raise "You need to inherit from this class and call super with a block" unless block_given?
+  def main(session, options, usecase, &block)
+    raise "You need to inherit from this class and call super with a block" unless block
     @session = session; @options = options; @usecase = usecase
-    yield
-    end_script!
+    process_ooze(&block)
+  end
+
+  def process_ooze(ooze = target)
+    raise "You need to inherit from this class and call super with a block" unless block_given?
+    super(ooze) do
+      yield(ooze)
+      exit_if_no_changes!
+    end
   end
 
   private
 
-  def end_script!
-    exit_if_no_changes!
-    backup_patch!
-    launch_request unless options[:simulate]
-    exit(0)
+  def ooze
+    super(ooze_id, stage_id: stage_id)
+  end
+
+  def stage(stage_id_name = stage_id, ooze: target)
+    super(stage_id_name, ooze: ooze)
   end
 
   def ooze_id
@@ -30,54 +39,9 @@ class Eco::API::UseCases::OozeSamples::OozeRunBaseCase < Eco::API::Common::Loade
     options.dig(:source, :stage_id)
   end
 
-  def target
-    @target ||= ooze
-  end
-
-  def new_target(object)
-    if @target && patch_doc["page"]
-      logger.warn "You you are switching to a new target #{object_reference(object)} after doing unsaved changes to #{object_reference(@target)}"
-    end
-    @target = object
-  end
-
-  def stage(id_name = stage_id)
-    if stg = ooze.stages[id_name] || ooze.stages.get_by_name(id_name)
-      return apiv2.pages.get(ooze_id, stage_id: stg.id).tap do |stage|
-        if stage
-          new_target(stage)
-          logger.info("Got #{object_reference(stage)} from #{object_reference(ooze)}")
-        else
-          logger.error("Could not get stage '#{id_name}' in ooze '#{ooze_id}'")
-          exit(1)
-        end
-      end
-    end
-    raise "Stage '#{id_name}' doesn't exist in ooze '#{ooze_id}'"
-  end
-
-  def ooze
-    @ooze ||= apiv2.pages.get(ooze_id, stage_id: stage_id).tap do |ooze|
-      if ooze
-        new_target(ooze)
-        logger.info("Got #{object_reference(ooze)}")
-      else
-        logger.error("Could not get ooze '#{ooze_id}'")
-        exit(1)
-      end
-    end
-  end
-
-  def launch_request
+  def update_ooze
     prompt_to_confirm!
-
-    apiv2.pages.update(target).tap do |response|
-      if response.success?
-        logger.info("All went OK")
-      else
-        logger.error(response.body)
-      end
-    end
+    super
   end
 
   def exit_if_no_changes!
@@ -87,46 +51,10 @@ class Eco::API::UseCases::OozeSamples::OozeRunBaseCase < Eco::API::Common::Loade
     end
   end
 
-  def apiv2
-   @apiv2 ||= session.api(version: :oozes)
-  end
-
-  def patch_doc
-    apiv2.pages.get_body(target)
-  end
-
-  def backup_patch!
-    # store the request
-    File.open(SAVE_PATCH, "w") do |file|
-      #file << (patch_doc || {}).to_json
-      file << JSON.pretty_generate(patch_doc || {})
-    end
-    puts "Saved patch at: #{File.expand_path(SAVE_PATCH)}"
-  end
-
-  def object_reference(obj)
-    return "No reference" unless obj
-    "".tap do |ref|
-      case obj
-      when Ecoportal::API::V2::Page::Stage
-        ref << "Stage "
-      when Ecoportal::API::V2::Pages::PageStage
-        ref << "StagePage "
-      when Ecoportal::API::V2::Page
-        ref << "Page "
-      end
-      ref << "'#{obj.name}' " if obj.respond_to?(:name)
-    end
-  end
-
   def prompt_to_confirm!
     micro.prompt_user("Do you want to proceed (y/N)?", default: "Y") do |response|
       exit(1) unless response.upcase.start_with?("Y")
     end
   end
 
-  def logger
-    session.logger
-  end
-
 end

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

@@ -2,15 +2,6 @@ class Eco::API::UseCases::OozeSamples::OozeUpdateCase < Eco::API::UseCases::Ooze
   name "single-ooze-case"
   type :other
 
-  include Eco::API::UseCases::OozeSamples::Helpers
-
-  def main(session, options, usecase)
-    raise "You need to inherit from this class and call super with a block" unless block_given?
-    super(session, options, usecase) do
-      yield
-    end
-  end
-
   private
 
   def with_fields(type: nil, label: nil)

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

@@ -0,0 +1,104 @@
+# Use case to update a register
+# @note
+#   - You can define methods `filters` and `search` to change the target entries of the register
+#   - You need to define the `process_ooze` method
+# This case expects `options[:source][:register_id]`
+class Eco::API::UseCases::OozeSamples::RegisterUpdateCase < Eco::API::UseCases::OozeSamples::OozeBaseCase
+
+  class << self
+    attr_reader :batch_size
+
+    # @return [Integer] the number of pages to be processed in each batch
+    def batch_size
+      @batch_size || 25
+    end
+  end
+
+  name "register-update-case"
+  type :other
+
+  def main(session, options, usecase, &block)
+    super(session, options, usecase) do
+      with_each_entry do |ooze|
+        process_ooze(&block)
+      end
+    end
+  end
+
+  def process_ooze(ooze = target)
+    raise "You need to define this method"
+  end
+
+  private
+
+  def with_each_entry
+    batched_search_results do |page_results|
+      batched_oozes = page_results.map do |page_result|
+        # TODO: Logics to get the correct PageStage
+        if ooze = new_target(ooze(page_result.id))
+          yield(ooze)
+        end
+        ooze
+      end
+      update_oozes(batched_oozes)
+    end
+  end
+
+  def batched_search_results
+    raise "Missing block. It yields in slices of #{self.class.batch_size} results" unless block_given?
+    results_preview
+    i = 0; results = []
+    apiv2.registers.search(register_id, search_options) do |page_result|
+      i += 1
+      results << page_result
+      if i <= self.class.batch_size
+        yield(results)
+        results = []
+        i = 0
+      end
+    end
+    yield(results) unless results.empty?
+  end
+
+  def update_oozes(batched_oozes)
+    batched_oozes.each do |ooze|
+      update_ooze(ooze)
+    end
+  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):", explanation: str_results, default: "N", timeout: 10) do |res|
+        if res.upcase.start_with?("N")
+          puts "..."
+          logger.info "Aborting script..."
+          exit(0)
+        end
+      end
+    end
+  end
+
+  def search_options
+    @search_options ||= {}.tap do |opts|
+      opts.merge!(sort: "created_at")
+      opts.merge!(dir:  "asc")
+      opts.merge!(query:   conf_filters) if conf_search
+      opts.merge!(filters: conf_filters)
+    end
+  end
+
+  def conf_filters
+    return filters if self.respond_to?(:filters)
+    []
+  end
+
+  def conf_search
+    return search if self.respond_to?(:search)
+  end
+
+  def register_id
+    options.dig(:source, :register_id)
+  end
+
+end

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

@@ -9,6 +9,8 @@ module Eco
 end
 
 require_relative 'ooze_samples/helpers'
+require_relative 'ooze_samples/ooze_base_case'
 require_relative 'ooze_samples/ooze_run_base_case'
 require_relative 'ooze_samples/ooze_update_case'
 require_relative 'ooze_samples/ooze_from_doc_case'
+require_relative 'ooze_samples/register_update_case'

data/lib/eco/version.rb

@@ -1,3 +1,3 @@
 module Eco
-  VERSION = "2.0.33"
+  VERSION = "2.0.34"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: eco-helpers
 version: !ruby/object:Gem::Version
-  version: 2.0.33
+  version: 2.0.34
 platform: ruby
 authors:
 - Oscar Segura
@@ -136,7 +136,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.8.9
+        version: 0.8.11
     - - "<"
       - !ruby/object:Gem::Version
         version: '0.9'
@@ -146,7 +146,7 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 0.8.9
+        version: 0.8.11
     - - "<"
       - !ruby/object:Gem::Version
         version: '0.9'
@@ -559,9 +559,11 @@ files:
 - lib/eco/api/usecases/ooze_samples.rb
 - lib/eco/api/usecases/ooze_samples/helpers.rb
 - lib/eco/api/usecases/ooze_samples/helpers/shortcuts.rb
+- lib/eco/api/usecases/ooze_samples/ooze_base_case.rb
 - lib/eco/api/usecases/ooze_samples/ooze_from_doc_case.rb
 - lib/eco/api/usecases/ooze_samples/ooze_run_base_case.rb
 - lib/eco/api/usecases/ooze_samples/ooze_update_case.rb
+- lib/eco/api/usecases/ooze_samples/register_update_case.rb
 - lib/eco/api/usecases/use_case.rb
 - lib/eco/api/usecases/use_case_chain.rb
 - lib/eco/api/usecases/use_case_io.rb