libis-workflow 2.1.5 → 2.1.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4ef72c9683a7f9bebb2148f90748b4cee50a1de85dd165f9402b496f6e333a2b
-  data.tar.gz: ff04df9816596546fea49280393dd25f72dab1c29c04a86932d5b7625808239b
+  metadata.gz: b9f56cd10757aa347d9853590fdbb493513c3ca8e5fc626388d4b08f5f95fe26
+  data.tar.gz: 4b4f623194705edca7496bebba401d89a039386cbd499c3ef22e5ce289b664c1
 SHA512:
-  metadata.gz: 368a2ee404589674c477bde0a252ce7df017d0cf4b25deb072caa286a53a34d141b943d870f8e2bce160a90683ee1216daf18655c41874fb95820be7c1fc1cb6
-  data.tar.gz: '089c486d5d7bfb719be4442f5aff7c30bbd1691eb54a36e58f204bb429c79b2e2fbc21f9faafed5f0055d06ab32d34c54572126cd23bbb01b3d9993a2adad4f3'
+  metadata.gz: e6faece7fc2cf155034716ab5d393f9b147b3c52bb16b0f3138d5c35ed927b9b6a721d5e0dfde2aea38bd2cf4d748dba83c2b32e3f6690883d2d96dd4d366bf5
+  data.tar.gz: c8e9c07728e6319a7536156e2d2e2b4c9e6891c53d14c69ba35194a86e5cdc4f118143bc5920b77f95dd2667415f082033e8756905aded19924fcc71d29fb1ae

data/README.md

@@ -258,7 +258,7 @@ perform on each work item:
 
     class MyTask < ::Libis::Workflow::Task
 
-      def process_item(item)
+      def process(item)
         if do_something(item)
           info "Did something"
         else
@@ -274,7 +274,7 @@ perform on each work item:
     end
 ```
 
-As seen above, the task should define a method called process_item that takes one argument. The argument will be a 
+As seen above, the task should define a method called process that takes one argument. The argument will be a 
 reference to the work item that it needs to perform an action on. The task has several option to progress after 
 performing its actions:
 * return. This is considered a normal and successful operation result. After a successful return the item's status will 
@@ -297,7 +297,7 @@ help of workflow input parameters and run options.
 
 #### Performing an action on the work item and all child items recursively
 
-With the 'recursive' parameter set to true, your task's process_item method will be called for the work item and then 
+With the 'recursive' parameter set to true, your task's process method will be called for the work item and then 
 once for each child and each child's children recursively.
 
 Note: you should not make both parent and child tasks recursive as this will cause the subitems to be processed 
@@ -318,7 +318,7 @@ parameter 'retry_interval', which is 30 by default.
 
 ### Pre- and postprocessing
 
-The default implementation of 'process' is to call 'pre_process' and then call 'process_item' on each child item, 
+The default implementation of running a task is to call 'pre_process' and then call 'process' for each child item, 
 followed by calling 'post_process'. The methods 'pre_process' and 'post_process' are no-operation methods by default, 
 but can be overwritten if needed.
 

data/lib/libis/workflow/task.rb

@@ -60,7 +60,7 @@ module Libis
         (parameter(:retry_count) + 1).times do
 
           i = run_item(item)
-          item = i if i.is_a?(Libis::Workflow::WorkItem)
+          item = i if i.is_a?(Libis::Workflow::Base::WorkItem)
 
           # noinspection RubyScope
           case item.status(namepath)
@@ -97,8 +97,8 @@ module Libis
 
       rescue Exception => e
         set_status item, :FAILED
-        fatal_error "Exception occured: #{e.message}", item
-        debug e.backtrace.join("\n")
+        fatal_error "Aborting ingest because of error: %s @ %s\n%s", item, e.message, e.backtrace.first, e.backtrace.map{|t| ' -- ' + t}.join("\n")
+        raise Libis::WorkflowAbort, "#{e.message} @ #{e.backtrace.first}"
 
       ensure
         item.save!
@@ -209,7 +209,7 @@ module Libis
             break if parameter(:abort_recursion_on_failure)
 
           rescue Libis::WorkflowAbort => e
-            fatal_error 'Fatal error processing subitem (%d/%d): %s', item, i + 1, items.size, e.message
+            fatal_error 'Fatal error processing subitem (%d/%d): %s @ %s\n%s', item, i + 1, items.size, e.message, e.backtrace.first, e.backtrace.map{|t| ' -- ' + t}.join("\n")
             item.set_status(namepath, :FAILED)
             break
 
@@ -218,7 +218,7 @@ module Libis
             raise Libis::WorkflowAbort, "#{e.message} @ #{e.backtrace.first}"
 
           else
-            item = new_item if new_item.is_a?(Libis::Workflow::WorkItem)
+            item = new_item if new_item.is_a?(Libis::Workflow::Base::WorkItem)
             parent_item.status_progress(namepath, i + 1)
 
           ensure

data/lib/libis/workflow/task_group.rb

@@ -50,7 +50,7 @@ module Libis
           end
           info 'Running subtask (%d/%d): %s', item, i+1, tasks.size, task.name
           new_item = task.run item
-          item = new_item if new_item.is_a?(Libis::Workflow::WorkItem)
+          item = new_item if new_item.is_a?(Libis::Workflow::Base::WorkItem)
           item.status_progress(self.namepath, i+1)
           item_status = item.status(task.namepath)
           status_count[item_status] += 1

data/lib/libis/workflow/version.rb

@@ -1,5 +1,5 @@
 module Libis
   module Workflow
-    VERSION = '2.1.5' unless const_defined? :VERSION # the guard is against a redefinition warning that happens on Travis
+    VERSION = '2.1.7' unless const_defined? :VERSION # the guard is against a redefinition warning that happens on Travis
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: libis-workflow
 version: !ruby/object:Gem::Version
-  version: 2.1.5
+  version: 2.1.7
 platform: ruby
 authors:
 - Kris Dekeyser
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-07-05 00:00:00.000000000 Z
+date: 2023-08-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake