forest_admin_agent 1.0.0.pre.beta.78 → 1.0.0.pre.beta.79

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 27acf5eecdcc0214bb2f2c88c761a165e4c769e8790d10be2ed999e4e2b99c6f
-  data.tar.gz: 6fc3368da9efe7a50120dfcf27fb731100be817075343f4d489efdbd6a225a26
+  metadata.gz: 0c6837279d632500c6245b172c625bae5ae5e4e459a396e90ed10cd7ac1f5591
+  data.tar.gz: 61dedb7cf077dd3ed87b178d59d78b557538fb83032cd3530cd791ff3442030f
 SHA512:
-  metadata.gz: f6325a6a4130edf6c3589bcdcf1c0247693fa17a95dd6dea045058b48e4289e85b13d0a4c9f9d67414aa1d91c6b795e254b20f966611ceff8448c27ea24f85a6
-  data.tar.gz: c68b8311af7cd615eaa1ef6b84dc9126098988d725885e6499acd9acb743d4023474fda0ec10176c7d3a6eb2b7ab405936db8335a11290b60290b6ca80fa76ef
+  metadata.gz: 5dada3205734f2ba783a141428937b26d176ccfbf82e012b59742417e61c10497f3d6aa25a1399b4f604fe765d21e66f1d07fc1dfd02340a36777b9b79746f9a
+  data.tar.gz: 38d2f8294e77b004df22d448f185a6153b85c08d1c0591a388bffa7252e794d72e8ceecf8d19d4ec75322db8fb506a2de0fa80704026e759557a59b555dc5792

data/lib/forest_admin_agent/builder/agent_factory.rb

@@ -80,13 +80,10 @@ module ForestAdminAgent
         return unless @has_env_secret
 
         cache = @container.resolve(:cache)
-        @options[:customize_error_message] = @options[:customize_error_message]
-                                             &.source
-                                             &.strip
-                                             &.delete_prefix('config.customize_error_message =')
-                                             &.strip
 
-        @options[:logger] = @options[:logger]&.source&.strip&.delete_prefix('config.logger =')&.strip
+        @options[:customize_error_message] =
+          clean_option_value(@options[:customize_error_message], 'config.customize_error_message =')
+        @options[:logger] = clean_option_value(@options[:logger], 'config.logger =')
 
         cache.set('config', @options.to_h)
       end
@@ -95,6 +92,14 @@ module ForestAdminAgent
         @logger = Services::LoggerService.new(@options[:loggerLevel], @options[:logger])
         @container.register(:logger, @logger)
       end
+
+      def clean_option_value(option, prefix)
+        return unless option
+
+        source = option.source
+        cleaned_option = source&.strip if source
+        cleaned_option&.delete_prefix(prefix)&.strip
+      end
     end
   end
 end

data/lib/forest_admin_agent/services/smart_action_checker.rb

@@ -37,7 +37,7 @@ module ForestAdminAgent
 
       def can_approve?
         if smart_action[:userApprovalEnabled].include?(role_id) &&
-           (smart_action[:userApprovalConditions].empty? || match_conditions(:userApprovalConditions)) &&
+           (condition_by_role_id(smart_action[:userApprovalConditions]).blank? || match_conditions(:userApprovalConditions)) &&
            (attributes[:signed_approval_request][:data][:attributes][:requester_id] != caller.id ||
              smart_action[:selfApprovalEnabled].include?(role_id))
           return true
@@ -48,15 +48,17 @@ module ForestAdminAgent
 
       def can_trigger?
         if smart_action[:triggerEnabled].include?(role_id) && !smart_action[:approvalRequired].include?(role_id)
-          return true if smart_action[:triggerConditions].empty? || match_conditions(:triggerConditions)
+          if condition_by_role_id(smart_action[:triggerConditions]).blank? || match_conditions(:triggerConditions)
+            return true
+          end
         elsif smart_action[:approvalRequired].include?(role_id) && smart_action[:triggerEnabled].include?(role_id)
-          if smart_action[:approvalRequiredConditions].empty? || match_conditions(:approvalRequiredConditions)
+          if condition_by_role_id(smart_action[:approvalRequiredConditions]).blank? || match_conditions(:approvalRequiredConditions)
             raise RequireApproval.new(
               'This action requires to be approved.',
               REQUIRE_APPROVAL_ERROR,
               smart_action[:userApprovalEnabled]
             )
-          elsif smart_action[:triggerConditions].empty? || match_conditions(:triggerConditions)
+          elsif condition_by_role_id(smart_action[:triggerConditions]).blank? || match_conditions(:triggerConditions)
             return true
           end
         end
@@ -71,11 +73,11 @@ module ForestAdminAgent
                            else
                              Nodes::ConditionTreeLeaf.new(pk, 'IN', attributes[:ids])
                            end
-        condition = smart_action[condition_name][0][:filter]
+        condition = condition_by_role_id(smart_action[condition_name])
         conditional_filter = filter.override(
           condition_tree: ConditionTreeFactory.intersect(
             [
-              ConditionTreeParser.from_plain_object(collection, condition),
+              ConditionTreeParser.from_plain_object(collection, condition[:filter]),
               filter.condition_tree,
               condition_filter
             ]
@@ -90,6 +92,10 @@ module ForestAdminAgent
           INVALID_ACTION_CONDITION_ERROR
         )
       end
+
+      def condition_by_role_id(condition)
+        condition.find { |c| c['roleId'] == role_id }
+      end
     end
   end
 end

data/lib/forest_admin_agent/utils/schema/schema_emitter.rb

@@ -7,7 +7,7 @@ module ForestAdminAgent
       class SchemaEmitter
         LIANA_NAME = "agent-ruby"
 
-        LIANA_VERSION = "1.0.0-beta.78"
+        LIANA_VERSION = "1.0.0-beta.79"
 
         def self.get_serialized_schema(datasource)
           schema_path = Facades::Container.cache(:schema_path)

data/lib/forest_admin_agent/version.rb

@@ -1,3 +1,3 @@
 module ForestAdminAgent
-  VERSION = "1.0.0-beta.78"
+  VERSION = "1.0.0-beta.79"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: forest_admin_agent
 version: !ruby/object:Gem::Version
-  version: 1.0.0.pre.beta.78
+  version: 1.0.0.pre.beta.79
 platform: ruby
 authors:
 - Matthieu
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2024-10-28 00:00:00.000000000 Z
+date: 2024-11-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport