openhab-scripting 4.28.2 → 4.29.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7c9d9745b2590c4152baf18a82dbe0063ac4c73c94f6f452d9d848790d8eb06a
-  data.tar.gz: 276152f1a79cf44fb539f4bb0c73618263db8d4a944cf367fa0ef1db167d989a
+  metadata.gz: 346933e8c00b9208f18ce61d4a87911cdc315a7b7ee1698aec1977dc47ba9bea
+  data.tar.gz: 5a94e6d5a1d43a4fd3e6586bd70a9f3286918bee1c300fe490d2460dbdce3494
 SHA512:
-  metadata.gz: d66efa237735b6a754e97eb933a301cb5dd639fa7cc715fa7d36c3e6f7ed14c11a3da874ea707792cb0913bb9e57396b33532746b7a63b08acf50ff15d4fc9f0
-  data.tar.gz: b2fcbb63e5e9da86909dd7b7b2bcb054b29aea1b80ae59fb26555a7aa678a7f4ebe04ee36dda69413c7cc8fbe08c7beeeaa5ad333e09d4bac08bbe2b769e85e7
+  metadata.gz: dcd2f7d6aa7b11074c059296b61b0fa22b691184915afe227ecf41d865690b6a81c8ea431c3b83407f05520aedca8238be4636f8da62217a0caf090d63bd8488
+  data.tar.gz: 1614e96cd679ebab6dfe811e7818e77e9e512603b126223f6a870f7b6550a2101cb86fe79f9856fe6559cc926cecf212afe15eb8a50892863c208b32d00f6386

data/lib/openhab/dsl/rules/automation_rule.rb

@@ -45,7 +45,7 @@ module OpenHAB
           between = config.between&.yield_self { between(config.between) }
           @between = between || OpenHAB::DSL::Between::ALL_DAY
           # Convert between to correct range or nil if not set
-          @trigger_delays = config.trigger_delays
+          @trigger_conditions = config.trigger_conditions
           @attachments = config.attachments
         end
         # rubocop:enable Metrics/MethodLength
@@ -57,18 +57,12 @@ module OpenHAB
         # @param [Map] inputs map provided by OpenHAB rules engine containing event and other information
         #
         #
-        def execute(mod = nil, inputs = nil) # rubocop:disable Metrics/MethodLength
+        def execute(mod = nil, inputs = nil)
           thread_local(RULE_NAME: name) do
             logger.trace { "Execute called with mod (#{mod&.to_string}) and inputs (#{inputs&.pretty_inspect})" }
             logger.trace { "Event details #{inputs['event'].pretty_inspect}" } if inputs&.key?('event')
-            if trigger_delay inputs
-              trigger_delay = trigger_delay(inputs)
-              process_trigger_delay(trigger_delay, mod, inputs)
-            else
-              # If guards are satisfied execute the run type blocks
-              # If they are not satisfied, execute the Othewise blocks
-              queue = create_queue(inputs)
-              process_queue(queue, mod, inputs)
+            trigger_conditions(inputs).process(mod: mod, inputs: inputs) do
+              process_queue(create_queue(inputs), mod, inputs)
             end
           end
         end
@@ -124,18 +118,18 @@ module OpenHAB
         end
 
         #
-        # Returns trigger delay from inputs if it exists
+        # Returns trigger conditions from inputs if it exists
         #
         # @param [Map] inputs map from OpenHAB containing UID
         #
-        # @return [Array] Array of trigger delays that match rule UID
+        # @return [Array] Array of trigger conditions that match rule UID
         #
-        def trigger_delay(inputs)
+        def trigger_conditions(inputs)
           # Parse this to get the trigger UID:
           # ["72698819-83cb-498a-8e61-5aab8b812623.event", "oldState", "module", \
           #  "72698819-83cb-498a-8e61-5aab8b812623.oldState", "event", "newState",\
           #  "72698819-83cb-498a-8e61-5aab8b812623.newState"]
-          @trigger_delays[trigger_id(inputs)]
+          @trigger_conditions[trigger_id(inputs)]
         end
 
         # If an attachment exists for the trigger for this event add it to the event object
@@ -151,138 +145,6 @@ module OpenHAB
           event
         end
 
-        #
-        # Check if trigger guards prevent rule execution
-        #
-        # @param [Delay] trigger_delay rules delaying trigger because of
-        # @param [Map] inputs OpenHAB map object describing rule trigger
-        #
-        # @return [Boolean] True if the rule should execute, false if trigger guard prevents execution
-        #
-        def check_trigger_guards(trigger_delay, inputs)
-          new_state, old_state = retrieve_states(inputs)
-          if check_from(trigger_delay, old_state)
-            return true if check_to(trigger_delay, new_state)
-
-            logger.trace("Skipped execution of rule '#{name}' because to state #{new_state}"\
-                         " does not equal specified state(#{trigger_delay.to})")
-          else
-            logger.trace("Skipped execution of rule '#{name}' because old state #{old_state}"\
-                         " does not equal specified state(#{trigger_delay.from})")
-          end
-        end
-
-        #
-        # Rerieve the newState and oldState, alternatively newStatus and oldStatus
-        # from the input map
-        #
-        # @param [Map] inputs OpenHAB map object describing rule trigger
-        #
-        # @return [Array] An array of the values for [newState, oldState] or [newStatus, oldStatus]
-        #
-        def retrieve_states(inputs)
-          old_state = inputs['oldState'] || thing_status_to_sym(inputs['oldStatus'])
-          new_state = inputs['newState'] || thing_status_to_sym(inputs['newStatus'])
-
-          [new_state, old_state]
-        end
-
-        #
-        # Converts a ThingStatus object to a ruby Symbol
-        #
-        # @param [Java::OrgOpenhabCoreThing::ThingStatus] status A ThingStatus instance
-        #
-        # @return [Symbol] A corresponding symbol, in lower case
-        #
-        def thing_status_to_sym(status)
-          status&.to_s&.downcase&.to_sym
-        end
-
-        #
-        # Check the from state against the trigger delay
-        #
-        # @param [TriggerDelay] trigger_delay Information about the trigger delay
-        # @param [Item State] state from state to check
-        #
-        # @return [Boolean] true if no from state is defined or defined state equals supplied state
-        #
-        def check_from(trigger_delay, state)
-          trigger_delay.from.nil? || state == trigger_delay.from
-        end
-
-        #
-        # Check the to state against the trigger delay
-        #
-        # @param [TriggerDelay] trigger_delay Information about the trigger delay
-        # @param [Item State] state to-state to check
-        #
-        # @return [Boolean] true if no to state is defined or defined state equals supplied state
-        #
-        def check_to(trigger_delay, state)
-          trigger_delay.to.nil? || state == trigger_delay.to
-        end
-
-        #
-        # Process any matching trigger delays
-        #
-        # @param [Map] mod OpenHAB map object describing rule trigger
-        # @param [Map] inputs OpenHAB map object describing rule trigger
-        #
-        #
-        def process_trigger_delay(trigger_delay, mod, inputs)
-          if trigger_delay.timer_active?
-            process_active_timer(inputs, mod, trigger_delay)
-          elsif check_trigger_guards(trigger_delay, inputs)
-            logger.trace("Trigger Guards Matched for #{trigger_delay}, delaying rule execution")
-            # Add timer and attach timer to delay object, and also state being tracked to so timer can be cancelled if
-            #   state changes
-            # Also another timer should not be created if changed to same value again but instead rescheduled
-            create_trigger_delay_timer(inputs, mod, trigger_delay)
-          else
-            logger.trace("Trigger Guards did not match for #{trigger_delay}, ignoring trigger.")
-          end
-        end
-
-        #
-        # Creatas a timer for trigger delays
-        #
-        # @param [Hash] inputs rule trigger inputs
-        # @param [Hash] mod rule trigger mods
-        # @param [TriggerDelay] trigger_delay specifications
-        #
-        #
-        def create_trigger_delay_timer(inputs, mod, trigger_delay)
-          logger.trace("Creating timer for rule #{name} and trigger delay #{trigger_delay}")
-          trigger_delay.timer = after(trigger_delay.duration) do
-            logger.trace("Delay Complete for #{trigger_delay}, executing rule")
-            trigger_delay.timer = nil
-            queue = create_queue(inputs)
-            process_queue(queue, mod, inputs)
-          end
-          trigger_delay.tracking_to, = retrieve_states(inputs)
-        end
-
-        #
-        # Process an active trigger timer
-        #
-        # @param [Hash] inputs rule trigger inputs
-        # @param [Hash] mod rule trigger mods
-        # @param [TriggerDelay] trigger_delay specifications
-        #
-        #
-        def process_active_timer(inputs, mod, trigger_delay)
-          state, = retrieve_states(inputs)
-          if state == trigger_delay.tracking_to
-            logger.trace("Item changed to #{state} for #{trigger_delay}, rescheduling timer.")
-            trigger_delay.timer.reschedule(ZonedDateTime.now.plus(trigger_delay.duration))
-          else
-            logger.trace("Item changed to #{state} for #{trigger_delay}, canceling timer.")
-            trigger_delay.timer.cancel
-            # Reprocess trigger delay after canceling to track new state (if guards matched, etc)
-            process_trigger_delay(trigger_delay, mod, inputs)
-          end
-        end
-
         #
         # Check if any guards prevent execution
         #

data/lib/openhab/dsl/rules/rule_config.rb

@@ -10,6 +10,7 @@ require_relative 'triggers/command'
 require_relative 'triggers/updated'
 require_relative 'triggers/generic'
 require_relative 'triggers/watch'
+require_relative 'triggers/conditions/none'
 require_relative 'guard'
 require 'openhab/core/entity_lookup'
 require 'openhab/dsl/between'
@@ -37,7 +38,7 @@ module OpenHAB
         attr_accessor :triggers
 
         # @return [Array] Of trigger delays
-        attr_reader :trigger_delays
+        attr_reader :trigger_conditions
 
         # @return [Hash] Hash of trigger UIDs to attachments
         attr_reader :attachments
@@ -86,7 +87,7 @@ module OpenHAB
         #
         def initialize(rule_name, caller_binding)
           @triggers = []
-          @trigger_delays = {}
+          @trigger_conditions = Hash.new(OpenHAB::DSL::Rules::Triggers::Conditions::None.instance)
           @attachments = {}
           @caller = caller_binding.eval 'self'
           name(rule_name)
@@ -145,7 +146,7 @@ module OpenHAB
             "Triggers: (#{triggers}) " \
             "Run blocks: (#{run}) " \
             "on_start: (#{on_start?}) " \
-            "Trigger Waits: #{trigger_delays} " \
+            "Trigger Conditions: #{trigger_conditions} " \
             "Trigger UIDs: #{triggers.map(&:id).join(', ')}" \
             "Attachments: #{attachments} "
         end

data/lib/openhab/dsl/rules/triggers/changed.rb

@@ -1,25 +1,19 @@
 # frozen_string_literal: true
 
 require 'openhab/log/logger'
+require_relative 'conditions/duration'
+require_relative 'conditions/range'
 
 module OpenHAB
   module DSL
     module Rules
       #
-      # Module holds rule triggers
+      # Module for changed triggers
       #
+
       module Triggers
         include OpenHAB::Log
 
-        #
-        # Struct capturing data necessary for a conditional trigger
-        #
-        TriggerDelay = Struct.new(:to, :from, :duration, :timer, :tracking_to, keyword_init: true) do
-          def timer_active?
-            timer&.is_active
-          end
-        end
-
         #
         # Creates a trigger item, group and thing changed
         #
@@ -38,7 +32,7 @@ module OpenHAB
             wait_duration = binding.local_variable_get(:for)
 
             each_state(from, to) do |from_state, to_state|
-              changed_or_wait(item, from_state, to_state, wait_duration, attach)
+              changed_trigger(item, from_state, to_state, wait_duration, attach)
             end
           end.flatten
         end
@@ -65,7 +59,7 @@ module OpenHAB
         end
 
         #
-        # Create regular or delayed trigger based on duration
+        # Create the trigger
         #
         # @param [Object] item item to create trigger for
         # @param [Item State] from state to restrict trigger to
@@ -75,9 +69,11 @@ module OpenHAB
         #
         # @return [Trigger] OpenHAB triggers
         #
-        def changed_or_wait(item, from, to, duration, attach)
+        def changed_trigger(item, from, to, duration, attach)
           if duration
             changed_wait(item, from: from, to: to, duration: duration, attach: attach)
+          elsif [to, from].grep(Range).any?
+            create_changed_range_trigger(item, from: from, to: to, attach: attach)
           else
             create_changed_trigger(item, from, to, attach)
           end
@@ -90,13 +86,31 @@ module OpenHAB
         # @param [OpenHAB::Core::Duration] duration to delay trigger for until condition is met
         # @param [Item State] to OpenHAB Item State item or group needs to change to
         # @param [Item State] from OpenHAB Item State item or group needs to be coming from
+        # @param [Object] attach to trigger
         #
         # @return [Trigger] OpenHAB trigger
         #
         def changed_wait(item, duration:, to: nil, from: nil, attach: nil)
           trigger = create_changed_trigger(item, nil, nil, attach)
           logger.trace("Creating Changed Wait Change Trigger for #{item}")
-          @trigger_delays[trigger.id] = TriggerDelay.new(to: to, from: from, duration: duration)
+          @trigger_conditions[trigger.id] = Conditions::Duration.new(to: to, from: from, duration: duration)
+          trigger
+        end
+
+        #
+        # Creates a trigger with a range condition on either 'from' or 'to' field
+        # @param [Object] item to create changed trigger on
+        # @param [Object] from state to restrict trigger to
+        # @param [Object] to state restrict trigger to
+        # @param [Object] attach to trigger
+        # @return [Trigger] OpenHAB trigger
+        #
+        def create_changed_range_trigger(item, from:, to:, attach:)
+          # swap range w/ nil if from or to is a range
+          from_range, from = from, from_range if from.is_a? Range
+          to_range, to = to, to_range if to.is_a? Range
+          trigger = create_changed_trigger(item, from, to, attach)
+          @trigger_conditions[trigger.id] = Conditions::Range.new(to: to_range, from: from_range)
           trigger
         end
 
@@ -104,8 +118,8 @@ module OpenHAB
         # Create a changed trigger
         #
         # @param [Object] item to create changed trigger on
-        # @param [String] from state to restrict trigger to
-        # @param [String] to state restrict trigger to
+        # @param [Object] from state to restrict trigger to
+        # @param [Object] to state restrict trigger to
         #
         #
         def create_changed_trigger(item, from, to, attach)

data/lib/openhab/dsl/rules/triggers/conditions/duration.rb

@@ -0,0 +1,184 @@
+# frozen_string_literal: true
+
+require 'openhab/log/logger'
+
+module OpenHAB
+  module DSL
+    module Rules
+      module Triggers
+        #
+        # Module for conditions for triggers
+        #
+        module Conditions
+          include OpenHAB::Log
+          #
+          # this is a no-op condition which simply executes the provided block
+          #
+
+          #
+          # Struct capturing data necessary for a conditional trigger
+          #
+          # TriggerDelay = Struct.new(:to, :from, :duration, :timer, :tracking_to, keyword_init: true) do
+          #  def timer_active?
+          #    timer&.is_active
+          #  end
+          # end
+
+          class Duration
+            def initialize(to:, from:, duration:)
+              @to = to
+              @from = from
+              @duration = duration
+            end
+
+            # Process rule
+            # @param [Hash] inputs inputs from trigger
+            #
+            def process(mod:, inputs:, &block)
+              process_trigger_delay(mod, inputs, &block)
+            end
+
+            private
+
+            #
+            # Checks if there is an active timer
+            # @return [true, false] true if the timer exists and is active, false otherwise
+            def timer_active?
+              @timer&.is_active
+            end
+
+            #
+            # Check if trigger guards prevent rule execution
+            #
+            # @param [Map] inputs OpenHAB map object describing rule trigger
+            #
+            # @return [Boolean] True if the rule should execute, false if trigger guard prevents execution
+            #
+            def check_trigger_guards(inputs)
+              new_state, old_state = retrieve_states(inputs)
+              if check_from(old_state)
+                return true if check_to(new_state)
+
+                logger.trace("Skipped execution of rule because to state #{new_state}"\
+                             " does not equal specified state(#{@to})")
+              else
+                logger.trace("Skipped execution of rule because old state #{old_state}"\
+                             " does not equal specified state(#{@from})")
+              end
+            end
+
+            #
+            # Rerieve the newState and oldState, alternatively newStatus and oldStatus
+            # from the input map
+            #
+            # @param [Map] inputs OpenHAB map object describing rule trigger
+            #
+            # @return [Array] An array of the values for [newState, oldState] or [newStatus, oldStatus]
+            #
+            def retrieve_states(inputs)
+              old_state = inputs['oldState'] || thing_status_to_sym(inputs['oldStatus'])
+              new_state = inputs['newState'] || thing_status_to_sym(inputs['newStatus'])
+
+              [new_state, old_state]
+            end
+
+            #
+            # Converts a ThingStatus object to a ruby Symbol
+            #
+            # @param [Java::OrgOpenhabCoreThing::ThingStatus] status A ThingStatus instance
+            #
+            # @return [Symbol] A corresponding symbol, in lower case
+            #
+            def thing_status_to_sym(status)
+              status&.to_s&.downcase&.to_sym
+            end
+
+            #
+            # Check the from state against the trigger delay
+            #
+            # @param [Item State] state from state to check
+            #
+            # @return [Boolean] true if no from state is defined or defined state equals supplied state
+            #
+            def check_from(state)
+              return @from.include?(state) if @from.is_a?(::Range)
+
+              @from.nil? || @from == state
+            end
+
+            #
+            # Check the to state against the trigger delay
+            #
+            # @param [Item State] state to-state to check
+            #
+            # @return [Boolean] true if no to state is defined or defined state equals supplied state
+            #
+            def check_to(state)
+              return @to.include?(state) if @to.is_a?(::Range)
+
+              @to.nil? || @to == state
+            end
+
+            #
+            # Process any matching trigger delays
+            #
+            # @param [Map] mod OpenHAB map object describing rule trigger
+            # @param [Map] inputs OpenHAB map object describing rule trigger
+            #
+            #
+            def process_trigger_delay(mod, inputs, &block)
+              if timer_active?
+                process_active_timer(inputs, mod, &block)
+              elsif check_trigger_guards(inputs)
+                logger.trace("Trigger Guards Matched for #{self}, delaying rule execution")
+                # Add timer and attach timer to delay object, and also state being tracked to so
+                # timer can be cancelled if state changes
+                # Also another timer should not be created if changed to same value again but instead rescheduled
+                create_trigger_delay_timer(inputs, mod, &block)
+              else
+                logger.trace("Trigger Guards did not match for #{self}, ignoring trigger.")
+              end
+            end
+
+            #
+            # Creates a timer for trigger delays
+            #
+            # @param [Hash] inputs rule trigger inputs
+            # @param [Hash] mod rule trigger mods
+            #
+            #
+            def create_trigger_delay_timer(inputs, _mod)
+              logger.trace("Creating timer for trigger delay #{self}")
+              @timer = after(@duration) do
+                logger.trace("Delay Complete for #{self}, executing rule")
+                @timer = nil
+                yield
+              end
+              @tracking_to, = retrieve_states(inputs)
+            end
+
+            #
+            # Process an active trigger timer
+            #
+            # @param [Hash] inputs rule trigger inputs
+            # @param [Hash] mod rule trigger mods
+            #
+            #
+            def process_active_timer(inputs, mod, &block)
+              state, = retrieve_states(inputs)
+              if state == @tracking_to
+                logger.trace("Item changed to #{state} for #{self}, rescheduling timer.")
+                @timer.reschedule(ZonedDateTime.now.plus(@duration))
+              else
+                logger.trace("Item changed to #{state} for #{self}, canceling timer.")
+                @timer.cancel
+                # Reprocess trigger delay after canceling to track new state (if guards matched, etc)
+                process_trigger_delay(mod, inputs, &block)
+              end
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/openhab/dsl/rules/triggers/conditions/none.rb

@@ -0,0 +1,32 @@
+# frozen_string_literal: true
+
+require 'openhab/log/logger'
+require 'singleton'
+
+module OpenHAB
+  module DSL
+    module Rules
+      module Triggers
+        #
+        # Module for conditions for triggers
+        #
+        module Conditions
+          include OpenHAB::Log
+          #
+          # this is a no-op condition which simply executes the provided block
+          #
+          class None
+            include Singleton
+
+            # Process rule
+            # @param [Hash] inputs inputs from trigger
+            #
+            def process(*)
+              yield
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/openhab/dsl/rules/triggers/conditions/range.rb

@@ -0,0 +1,74 @@
+# frozen_string_literal: true
+
+require 'openhab/log/logger'
+
+module OpenHAB
+  module DSL
+    module Rules
+      module Triggers
+        #
+        # Module for conditions for triggers
+        #
+        module Conditions
+          include OpenHAB::Log
+
+          #
+          # This creates trigger conditions that work on ranges
+          # @param [Range:] From range
+          # @param [To:] To range
+          #
+          class Range
+            def initialize(from: nil, to: nil)
+              @from = from
+              @to = to
+            end
+
+            #
+            # Process rule
+            # @param [Hash] inputs inputs from trigger
+            #
+            def process(mod:, inputs:) # rubocop:disable Lint/UnusedMethodArgument - mod is unused here but required
+              logger.trace("Checking #{inputs} against condition trigger #{self}")
+              yield if check_from(inputs: inputs) && check_to(inputs: inputs)
+            end
+
+            #
+            # Check if from condition match the inputs
+            # @param [Hash] inputs inputs from trigger
+            # @return [true/false] depending on if from is set and matches supplied conditions
+            #
+            def check_from(inputs:)
+              old_state = inputs['oldState']
+              return true if @from.nil? || @from.include?(old_state)
+
+              logger.trace("Skipped execution of rule because old state #{old_state}"\
+                           " does not equal specified range(#{@from})")
+              false
+            end
+
+            #
+            # Check if to conditions match the inputs
+            # @param [Hash] inputs inputs from trigger
+            # @return [true/false] depending on if from is set and matches supplied conditions
+            #
+            def check_to(inputs:)
+              new_state = inputs['newState'] || inputs['state'] # Get state for changed or update
+              return true if @to.nil? || @to.include?(new_state)
+
+              logger.trace("Skipped execution of rule because new state #{new_state}"\
+                           " does not equal specified range(#{@to})")
+              false
+            end
+
+            # Describe the Range Condition as a string
+            # @return [String] string representation of range condition
+            #
+            def to_s
+              "From:(#{@from}) To:(#{@to})"
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/openhab/dsl/rules/triggers/updated.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require 'openhab/log/logger'
+require_relative 'trigger'
 
 module OpenHAB
   module DSL
@@ -23,29 +24,59 @@ module OpenHAB
           separate_groups(items).map do |item|
             logger.trace("Creating updated trigger for item(#{item}) to(#{to})")
             [to].flatten.map do |to_state|
-              trigger, config = create_update_trigger(item, to_state)
-              append_trigger(trigger, config, attach: attach)
+              update_trigger(item: item, to: to_state, attach: attach)
             end
           end.flatten
         end
 
         private
 
+        #
+        # Create the trigger
+        #
+        # @param [Object] item item to create trigger for
+        # @param [Item State] from state to restrict trigger to
+        # @param [Item State] to state to restrict trigger to
+        # @param attach attachment
+        #
+        # @return [Trigger] OpenHAB triggers
+        #
+        def update_trigger(item:, to:, attach:)
+          if to.is_a? Range
+            create_update_range_trigger(item: item, to: to, attach: attach)
+          else
+            create_update_trigger(item: item, to: to, attach: attach)
+          end
+        end
+
+        #
+        # Creates a trigger with a range condition on the 'to' field
+        # @param [Object] item to create changed trigger on
+        # @param [Object] to state restrict trigger to
+        # @param [Object] attach to trigger
+        # @return [Trigger] OpenHAB trigger
+        #
+        def create_update_range_trigger(item:, to:, attach:)
+          trigger = create_update_trigger(item: item, to: nil, attach: attach)
+          @trigger_conditions[trigger.id] = Conditions::Range.new(to: to, from: nil)
+          trigger
+        end
+
         #
         # Create a trigger for updates
         #
         # @param [Object] item Type of item [Group,Thing,Item] to create update trigger for
         # @param [State] to_state state restriction on trigger
         #
-        # @return [Array<Hash,String>] first element is a String specifying trigger type
-        #  second element is a Hash configuring trigger
+        # @return [Trigger] OpenHAB triggers
         #
-        def create_update_trigger(item, to_state)
-          case item
-          when OpenHAB::DSL::Items::GroupItem::GroupMembers then group_update(item, to_state)
-          when Thing then thing_update(item, to_state)
-          else item_update(item, to_state)
-          end
+        def create_update_trigger(item:, to:, attach:)
+          trigger, config = case item
+                            when OpenHAB::DSL::Items::GroupItem::GroupMembers then group_update(item: item, to: to)
+                            when Thing then thing_update(thing: item, to: to)
+                            else item_update(item: item, to: to)
+                            end
+          append_trigger(trigger, config, attach: attach)
         end
 
         #
@@ -57,9 +88,9 @@ module OpenHAB
         # @return [Array<Hash,String>] first element is a String specifying trigger type
         #  second element is a Hash configuring trigger
         #
-        def item_update(item, to_state)
+        def item_update(item:, to:)
           config = { 'itemName' => item.name }
-          config['state'] = to_state.to_s unless to_state.nil?
+          config['state'] = to.to_s unless to.nil?
           trigger = Trigger::ITEM_STATE_UPDATE
           [trigger, config]
         end
@@ -73,9 +104,9 @@ module OpenHAB
         # @return [Array<Hash,String>] first element is a String specifying trigger type
         #  second element is a Hash configuring trigger
         #
-        def group_update(item, to_state)
+        def group_update(item:, to:)
           config = { 'groupName' => item.group.name }
-          config['state'] = to_state.to_s unless to_state.nil?
+          config['state'] = to.to_s unless to.nil?
           trigger = Trigger::GROUP_STATE_UPDATE
           [trigger, config]
         end
@@ -89,8 +120,8 @@ module OpenHAB
         # @return [Array<Hash,String>] first element is a String specifying trigger type
         #  second element is a Hash configuring trigger
         #
-        def thing_update(thing, to_state)
-          trigger_for_thing(thing, Trigger::THING_UPDATE, to_state)
+        def thing_update(thing:, to:)
+          trigger_for_thing(thing, Trigger::THING_UPDATE, to)
         end
       end
     end

data/lib/openhab/version.rb

@@ -5,5 +5,5 @@
 #
 module OpenHAB
   # @return [String] Version of OpenHAB helper libraries
-  VERSION = '4.28.2'
+  VERSION = '4.29.0'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: openhab-scripting
 version: !ruby/object:Gem::Version
-  version: 4.28.2
+  version: 4.29.0
 platform: ruby
 authors:
 - Brian O'Connell
@@ -106,6 +106,9 @@ files:
 - lib/openhab/dsl/rules/triggers/changed.rb
 - lib/openhab/dsl/rules/triggers/channel.rb
 - lib/openhab/dsl/rules/triggers/command.rb
+- lib/openhab/dsl/rules/triggers/conditions/duration.rb
+- lib/openhab/dsl/rules/triggers/conditions/none.rb
+- lib/openhab/dsl/rules/triggers/conditions/range.rb
 - lib/openhab/dsl/rules/triggers/cron.rb
 - lib/openhab/dsl/rules/triggers/generic.rb
 - lib/openhab/dsl/rules/triggers/trigger.rb