openhab-scripting 5.27.0 → 5.27.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 59deb6289921c0c6d35579783efbb35f75ceac9ff24922e0f74d6652ef575b59
-  data.tar.gz: 260ee0993a3ab89775d9b5a2e8f3e40f295dd16ad8f0adf3b003b521cb08a12e
+  metadata.gz: 55919e4e10099bb160e0f64578c398caef457c330e88df280a20ed20ffec6310
+  data.tar.gz: 2052d4bcd4dc3bcd7b2be5c4bb7b995fcaebc7101b93e24fd2db0e849b80ff36
 SHA512:
-  metadata.gz: 127b6e33961395b973b8aa543940502530101f30e15526aa598b574e8d3a05ec82c65d79fd0d0d6176a2aa57aed8f244fbd480c76457e3440f94503f92028530
-  data.tar.gz: 4949521fcc45fdb1dba17a09c960d4a5e7afd2c964328a9fd8b66b11ac2c773c7e413e6ec2d797f9e6d9f8e9130297c9700af4d46bf968ec064b0dfa2ea6eb32
+  metadata.gz: 1c39a9d29033bf7844d1f5666f57be9ae99ae595a8d5f85e4464b6077c0171e4f4e9e19d933f17a0e190c2081dae57dc29136bb4c3e295f2d83e8e1d88374ce1
+  data.tar.gz: 5a828c34365b2d8fcef74c46fe19e94fc30dbb0c4fd17fcef7bc717a95ccb2e0b51cfadf701447d9a97c58057baebe77d8186811886479112096b46e16749a28

data/lib/openhab/core/actions.rb

@@ -37,7 +37,7 @@ module OpenHAB
                  else
                    (java_import action_class.ruby_class).first
                  end
-        logger.trace("Loaded ACTION: #{action_class}")
+        logger.trace { "Loaded ACTION: #{action_class}" }
         Object.const_set(module_name, action)
       end
 

data/lib/openhab/core/items/generic_item.rb

@@ -276,7 +276,7 @@ module OpenHAB
               raise FrozenError, "Cannot modify item #{name} from provider #{provider.inspect}." unless force
 
               provider = nil
-              logger.debug("Forcing modifications to non-managed item #{name}")
+              logger.debug { "Forcing modifications to non-managed item #{name}" }
             end
             @modified = false
             @modifying = true

data/lib/openhab/core/items/image_item.rb

@@ -55,7 +55,7 @@ module OpenHAB
         #
         #
         def update_from_url(uri)
-          logger.trace("Downloading image from #{uri}")
+          logger.trace { "Downloading image from #{uri}" }
           response = Net::HTTP.get_response(URI(uri))
           mime_type = response["content-type"]
           bytes = response.body

data/lib/openhab/core/items/metadata/namespace_hash.rb

@@ -73,7 +73,7 @@ module OpenHAB
               raise ArgumentError, "Wrong number of arguments (given #{default_value.length + 1}, expected 1..2)"
             end
 
-            logger.trace("Getting metadata for item: #{@item_name}, namespace '#{key}'")
+            logger.trace { "Getting metadata for item: #{@item_name}, namespace '#{key}'" }
             if (m = Provider.registry.get(MetadataKey.new(key, @item_name)))
               Hash.new(m)
             elsif block

data/lib/openhab/core/items/semantics.rb

@@ -251,7 +251,7 @@ module OpenHAB
           #
           # @!visibility private
           def const_missing(sym)
-            logger.trace("const missing, performing Semantics Lookup for: #{sym}")
+            logger.trace { "const missing, performing Semantics Lookup for: #{sym}" }
             lookup(sym)&.tap { |tag| const_set(sym, tag) } || super
           end
         end

data/lib/openhab/core/items.rb

@@ -44,7 +44,7 @@ module OpenHAB
             _command_predicate, state_predicate = Types::PREDICATE_ALIASES[state.to_s]
             next if klass.instance_methods.include?(state_predicate)
 
-            logger.trace("Defining #{klass}##{state_predicate} for #{state}")
+            logger.trace { "Defining #{klass}##{state_predicate} for #{state}" }
             klass.class_eval <<~RUBY, __FILE__, __LINE__ + 1
               def #{state_predicate}                                                  # def on?
                 raw_state.as(#{state.class.java_class.simple_name}).equal?(#{state})  #   raw_state.as(OnOffType) == ON
@@ -61,7 +61,7 @@ module OpenHAB
             next if klass.instance_methods.include?(command)
 
             if value.is_a?(Types::State)
-              logger.trace("Defining #{klass}/Enumerable##{command}/#{command}! for #{value}")
+              logger.trace { "Defining #{klass}/Enumerable##{command}/#{command}! for #{value}" }
 
               klass.class_eval <<~RUBY, __FILE__, __LINE__ + 1
                 ruby2_keywords def #{command}(*args, &block)   # ruby2_keywords def on(*args, &block)
@@ -83,7 +83,7 @@ module OpenHAB
                 end                    # end
               RUBY
             else
-              logger.trace("Defining #{klass}/Enumerable##{command} for #{value}")
+              logger.trace { "Defining #{klass}/Enumerable##{command} for #{value}" }
 
               klass.class_eval <<~RUBY, __FILE__, __LINE__ + 1
                 ruby2_keywords def #{command}(*args, &block)  # ruby2_keywords def refresh(*args, &block)
@@ -98,7 +98,7 @@ module OpenHAB
               RUBY
             end
 
-            logger.trace("Defining ItemCommandEvent##{command}? for #{value}")
+            logger.trace { "Defining ItemCommandEvent##{command}? for #{value}" }
             Events::ItemCommandEvent.class_eval <<~RUBY, __FILE__, __LINE__ + 1
               def #{command}?                                                       # def refresh?
                 command.as(#{value.class.java_class.simple_name}).equal?(#{value})  #   command.as(RefreshType).equal?(REFRESH)

data/lib/openhab/core/profile_factory.rb

@@ -86,7 +86,9 @@ module OpenHAB
         private
 
         def process_event(event, **params)
-          logger.trace("Handling event #{event.inspect} in profile #{@uid} with param #{params.values.first.inspect}.")
+          logger.trace do
+            "Handling event #{event.inspect} in profile #{@uid} with param #{params.values.first.inspect}."
+          end
 
           params[:callback] = @callback
           params[:context] = @context

data/lib/openhab/core/provider.rb

@@ -213,8 +213,18 @@ module OpenHAB
         old
       end
 
+      # @!visibility private
+      def clear
+        elements = @elements
+        @elements = java.util.concurrent.ConcurrentHashMap.new
+        elements.each_value do |v|
+          notify_listeners_about_removed_element(v)
+        end
+      end
+
       # @!visibility private
       def unregister
+        clear
         # @deprecated OH3.4 safe navigation only required for missing Semantics registry
         self.class.registry&.remove_provider(self)
       end

data/lib/openhab/core/script_handling.rb

@@ -119,7 +119,7 @@ module OpenHAB
       # Executed when openHAB loads a script file
       #
       def scriptLoaded(filename) # rubocop:disable Naming/MethodName method name dictated by openHAB
-        logger.trace("Script loaded: #{filename}")
+        logger.trace { "Script loaded: #{filename}" }
         ScriptHandlingCallbacks.script_loaded_hooks.each do |hook|
           hook.call
         rescue => e

data/lib/openhab/core/sitemaps/provider.rb

@@ -47,6 +47,7 @@ module OpenHAB
         # rubocop:enable Naming/MethodName
 
         # @!visibility private
+        # Override this because we don't have a registry
         def unregister
           clear
           @registration.unregister
@@ -140,16 +141,6 @@ module OpenHAB
           end
         end
 
-        # For use in specs
-        # @!visibility private
-        def clear
-          elements = @elements
-          @elements = java.util.concurrent.ConcurrentHashMap.new
-          elements.each_value do |v|
-            notify_listeners_about_removed_element(v)
-          end
-        end
-
         #
         # Notify listeners about updated sitemap
         #

data/lib/openhab/core/things/thing.rb

@@ -256,7 +256,7 @@ module OpenHAB
 
         # @!visibility private
         def respond_to_missing?(method_name, _include_private = false)
-          logger.trace("Checking if Thing #{uid} supports #{method_name} action")
+          logger.trace { "Checking if Thing #{uid} supports #{method_name} action" }
           return true if actions.respond_to?(method_name)
 
           super

data/lib/openhab/core/types/date_time_type.rb

@@ -26,36 +26,20 @@ module OpenHAB
           include Comparable
         end
 
-        #
-        # Regex expression to identify strings defining a time in hours, minutes and optionally seconds
-        #
-        TIME_ONLY_REGEX = /\A(?<hours>\d\d):(?<minutes>\d\d)(?<seconds>:\d\d)?\Z/.freeze
-
-        #
-        # Regex expression to identify strings defining a time in year, month, and day
-        #
-        DATE_ONLY_REGEX = /\A\d{4}-\d\d-\d\d\Z/.freeze
-        private_constant :TIME_ONLY_REGEX, :DATE_ONLY_REGEX
-
         class << self
           #
-          # Parses a String representing a time into a {DateTimeType}. First tries to parse it
-          #   using Java's parser, then falls back to the Ruby `Time.parse`.
+          # Parse a time string into a {DateTimeType}.
           #
-          # @param [String] time_string
+          # @note openHAB's DateTimeType.new(String) constructor will parse time-only strings and fill in `1970-01-01`
+          #   as the date, whereas this method will use the current date.
           #
+          # @param (see DSL#try_parse_time_like)
           # @return [DateTimeType]
           #
           def parse(time_string)
-            time_string = "#{time_string}Z" if TIME_ONLY_REGEX.match?(time_string)
-            DateTimeType.new(time_string)
-          rescue java.lang.StringIndexOutOfBoundsException, java.lang.IllegalArgumentException => e
-            # Try Ruby's Time.parse if DateTimeType parser fails
-            begin
-              DateTimeType.new(::Time.parse(time_string).to_zoned_date_time)
-            rescue ArgumentError
-              raise ArgumentError, e.message
-            end
+            DateTimeType.new(DSL.try_parse_time_like(time_string).to_zoned_date_time)
+          rescue ArgumentError
+            raise ArgumentError, e.message
           end
         end
 
@@ -129,7 +113,7 @@ module OpenHAB
         #   `nil` is returned if the two values are incomparable.
         #
         def <=>(other)
-          logger.trace("(#{self.class}) #{self} <=> #{other} (#{other.class})")
+          logger.trace { "(#{self.class}) #{self} <=> #{other} (#{other.class})" }
           if other.is_a?(self.class)
             zoned_date_time <=> other.zoned_date_time
           elsif other.respond_to?(:to_time)
@@ -151,7 +135,7 @@ module OpenHAB
         # @return [[DateTimeType, DateTimeType], nil]
         #
         def coerce(other)
-          logger.trace("Coercing #{self} as a request from #{other.class}")
+          logger.trace { "Coercing #{self} as a request from #{other.class}" }
           return [other, zoned_date_time] if other.respond_to?(:to_zoned_date_time)
 
           [DateTimeType.new(other), self] if other.respond_to?(:to_time)

data/lib/openhab/core/types/decimal_type.rb

@@ -91,7 +91,7 @@ module OpenHAB
         #   `nil` is returned if the two values are incomparable.
         #
         def <=>(other)
-          logger.trace("(#{self.class}) #{self} <=> #{other} (#{other.class})")
+          logger.trace { "(#{self.class}) #{self} <=> #{other} (#{other.class})" }
           if other.is_a?(QuantityType) || other.is_a?(HSBType)
             (other <=> self)&.-@
           elsif other.is_a?(self.class)
@@ -115,7 +115,7 @@ module OpenHAB
         # @return [Array<(DecimalType, DecimalType)>, nil]
         #
         def coerce(other)
-          logger.trace("Coercing #{self} as a request from #{other.class}")
+          logger.trace { "Coercing #{self} as a request from #{other.class}" }
           return unless other.respond_to?(:to_d)
 
           [self.class.new(other.to_d), self]

data/lib/openhab/core/types/hsb_type.rb

@@ -65,7 +65,7 @@ module OpenHAB
               # in this case, HTML hex format for rgb
               if (match = value.match(/^#(\h{2})(\h{2})(\h{2})$/))
                 rgb = match.to_a[1..3].map { |v| v.to_i(16) }
-                logger.trace("creating from rgb #{rgb.inspect}")
+                logger.trace { "creating from rgb #{rgb.inspect}" }
                 return from_rgb(*rgb)
               end
             end
@@ -110,7 +110,7 @@ module OpenHAB
         #   `nil` is returned if the two values are incomparable.
         #
         def <=>(other)
-          logger.trace("(#{self.class}) #{self} <=> #{other} (#{other.class})")
+          logger.trace { "(#{self.class}) #{self} <=> #{other} (#{other.class})" }
           if other.is_a?(HSBType)
             [brightness, hue, saturation] <=> [other.brightness, other.hue, other.saturation]
           else

data/lib/openhab/core/types/point_type.rb

@@ -103,7 +103,7 @@ module OpenHAB
         #
         # @return [QuantityType]
         def distance_from(other)
-          logger.trace("(#{self}).distance_from(#{other} (#{other.class})")
+          logger.trace { "(#{self}).distance_from(#{other} (#{other.class})" }
           raise TypeError, "#{other.class} can't be coerced into #{self.class}" unless other.is_a?(PointType)
 
           QuantityType.new(raw_distance_from(other), SIUnits::METRE)

data/lib/openhab/core/types/quantity_type.rb

@@ -132,7 +132,7 @@ module OpenHAB
         #   `nil` is returned if the two values are incomparable.
         #
         def <=>(other)
-          logger.trace("(#{self.class}) #{self} <=> #{other} (#{other.class})")
+          logger.trace { "(#{self.class}) #{self} <=> #{other} (#{other.class})" }
           case other
           when self.class
             return unitize(other.unit).compare_to(other) if unit == Units::ONE
@@ -161,7 +161,7 @@ module OpenHAB
         #
         # @return [Array<(QuantityType, QuantityType)>, nil]
         def coerce(other)
-          logger.trace("Coercing #{self} as a request from #{other.class}")
+          logger.trace { "Coercing #{self} as a request from #{other.class}" }
           return unless other.respond_to?(:to_d)
 
           [QuantityType.new(other.to_d.to_java, Units::ONE), self]
@@ -178,7 +178,7 @@ module OpenHAB
 
           class_eval( # rubocop:disable Style/DocumentDynamicEvalDefinition https://github.com/rubocop/rubocop/issues/10179
             # def +(other)
-            #   logger.trace("#{self} + #{other} (#{other.class})")
+            #   logger.trace { "#{self} + #{other} (#{other.class})" }
             #   other = other.state if other.is_a?(Core::Items::Persistence::PersistedState)
             #   if other.is_a?(QuantityType)
             #     add_quantity(other)
@@ -203,7 +203,7 @@ module OpenHAB
             # end
             <<~RUBY, __FILE__, __LINE__ + 1
               def #{ruby_op}(other)
-                logger.trace("\#{self} #{ruby_op} \#{other} (\#{other.class})")
+                logger.trace { "\#{self} #{ruby_op} \#{other} (\#{other.class})" }
                 other = other.state if other.is_a?(Core::Items::Persistence::PersistedState)
                 if other.is_a?(QuantityType)
                   #{java_op}_quantity(other)
@@ -236,7 +236,7 @@ module OpenHAB
         }.each do |java_op, ruby_op|
           class_eval( # rubocop:disable Style/DocumentDynamicEvalDefinition https://github.com/rubocop/rubocop/issues/10179
             # def *(other)
-            #   logger.trace("#{self} * #{other} (#{other.class})")
+            #   logger.trace { "#{self} * #{other} (#{other.class})" }
             #   other = other.state if other.is_a?(Core::Items::Persistence::PersistedState)
             #   if other.is_a?(QuantityType)
             #     multiply_quantity(other)
@@ -254,7 +254,7 @@ module OpenHAB
             # end
             <<~RUBY, __FILE__, __LINE__ + 1
               def #{ruby_op}(other)
-                logger.trace("\#{self} #{ruby_op} \#{other} (\#{other.class})")
+                logger.trace { "\#{self} #{ruby_op} \#{other} (\#{other.class})" }
                 other = other.state if other.is_a?(Core::Items::Persistence::PersistedState)
                 if other.is_a?(QuantityType)
                   #{java_op}_quantity(other).unitize
@@ -279,7 +279,7 @@ module OpenHAB
         def unitize(other_unit = unit, relative: false)
           # prefer converting to the thread-specified unit if there is one
           other_unit = DSL.unit(dimension) || other_unit
-          logger.trace("Converting #{self} to #{other_unit}")
+          logger.trace { "Converting #{self} to #{other_unit}" }
 
           case unit
           when Units::ONE

data/lib/openhab/core/types/string_type.rb

@@ -41,7 +41,7 @@ module OpenHAB
         #   `nil` is returned if the two values are incomparable.
         #
         def <=>(other)
-          logger.trace("(#{self.class}) #{self} <=> #{other} (#{other.class})")
+          logger.trace { "(#{self.class}) #{self} <=> #{other} (#{other.class})" }
           if other.respond_to?(:to_str)
             to_str <=> other.to_str
           elsif other.respond_to?(:coerce)
@@ -61,7 +61,7 @@ module OpenHAB
         # @return [[StringType, StringType], nil]
         #
         def coerce(other)
-          logger.trace("Coercing #{self} as a request from #{other.class}")
+          logger.trace { "Coercing #{self} as a request from #{other.class}" }
           [other.to_str, self] if other.respond_to?(:to_str)
         end
 

data/lib/openhab/core/types.rb

@@ -55,7 +55,7 @@ module OpenHAB
           states = Types::PREDICATE_ALIASES[value.to_s]
 
           ([command] | states).each do |method|
-            logger.trace("Defining #{klass}##{method} for #{value}")
+            logger.trace { "Defining #{klass}##{method} for #{value}" }
             klass.class_eval <<~RUBY, __FILE__, __LINE__ + 1
               def #{method}                                                 # def on?
                 as(#{value.class.java_class.simple_name}).equal?(#{value})  #   as(OnOffType).equal?(ON)

data/lib/openhab/core.rb

@@ -38,7 +38,7 @@ module OpenHAB
       def wait_till_openhab_ready
         logger.trace("Checking readiness of openHAB")
         until automation_manager
-          logger.trace("Automation manager not loaded, checking again in #{CHECK_DELAY} seconds.")
+          logger.trace { "Automation manager not loaded, checking again in #{CHECK_DELAY} seconds." }
           sleep CHECK_DELAY
         end
         logger.trace "Automation manager instantiated, openHAB ready for rule processing."

data/lib/openhab/core_ext/java/month_day.rb

@@ -20,7 +20,7 @@ module OpenHAB
           # @return [MonthDay]
           #
           def parse(string)
-            logger.trace("#{self.class}.parse #{string} (#{string.class})")
+            logger.trace { "#{self.class}.parse #{string} (#{string.class})" }
             java_send(:parse,
                       [java.lang.CharSequence, java.time.format.DateTimeFormatter],
                       string.to_s,

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

@@ -104,7 +104,7 @@ module OpenHAB
           @listener ||= org.openhab.core.common.registry.RegistryChangeListener.impl do |method, element|
             next unless method == :removed
 
-            logger.trace("Rule #{element.inspect} removed from registry")
+            logger.trace { "Rule #{element.inspect} removed from registry" }
             next unless element.uid == uid
 
             cleanup
@@ -225,7 +225,7 @@ module OpenHAB
 
           return true if @guard.should_run?(event)
 
-          logger.trace("Skipped execution of rule '#{name}' because of guard #{@guard}")
+          logger.trace { "Skipped execution of rule '#{name}' because of guard #{@guard}" }
           false
         end
 

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

@@ -37,7 +37,7 @@ module OpenHAB
         # @return [true,false] True if guard is satisfied, false otherwise
         #
         def should_run?(event)
-          logger.trace("Checking guards #{self}")
+          logger.trace { "Checking guards #{self}" }
           return false unless check_only_if(event)
           return false unless check_not_if(event)
 

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

@@ -44,7 +44,7 @@ module OpenHAB
         def append_trigger(type:, config:, attach: nil, conditions: nil, label: nil)
           config.transform_keys!(&:to_s)
           RuleTriggers.trigger(type: type, config: config, label: label).tap do |trigger|
-            logger.trace("Appending trigger (#{trigger.inspect}) attach (#{attach}) conditions(#{conditions})")
+            logger.trace { "Appending trigger (#{trigger.inspect}) attach (#{attach}) conditions(#{conditions})" }
             @triggers << trigger
             @attachments[trigger.id] = attach if attach
             @trigger_conditions[trigger.id] = conditions if conditions
@@ -61,7 +61,7 @@ module OpenHAB
         # @return [org.openhab.core.automation.Trigger] configured by type and supplied config
         #
         def self.trigger(type:, config:, label: nil)
-          logger.trace("Creating trigger of type '#{type}' config: #{config}")
+          logger.trace { "Creating trigger of type '#{type}' config: #{config}" }
           org.openhab.core.automation.util.TriggerBuilder.create
              .with_id(uuid)
              .with_type_uid(type)

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

@@ -22,8 +22,10 @@ module OpenHAB
               @conditions = Generic.new(to: to, from: from)
               @duration = duration
               @timers = {}
-              logger.trace "Created Duration Condition To(#{to}) From(#{from}) " \
-                           "Conditions(#{@conditions}) Duration(#{@duration})"
+              logger.trace do
+                "Created Duration Condition To(#{to}) From(#{from}) " \
+                  "Conditions(#{@conditions}) Duration(#{@duration})"
+              end
             end
 
             # Process rule
@@ -34,13 +36,13 @@ module OpenHAB
               if timer&.active?
                 process_active_timer(timer, inputs, mod, &block)
               elsif @conditions.process(mod: mod, inputs: inputs)
-                logger.trace("Trigger Guards Matched for #{self}, delaying rule execution")
+                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.")
+                logger.trace { "Trigger Guards did not match for #{self}, ignoring trigger." }
               end
             end
 
@@ -61,10 +63,10 @@ module OpenHAB
             #
             #
             def create_trigger_delay_timer(inputs, _mod)
-              logger.trace("Creating timer for trigger delay #{self}")
+              logger.trace { "Creating timer for trigger delay #{self}" }
               item_name = inputs["triggeringItem"]&.name
               @timers[item_name] = DSL.after(@duration) do
-                logger.trace("Delay Complete for #{self}, executing rule")
+                logger.trace { "Delay Complete for #{self}, executing rule" }
                 @timers.delete(item_name)
                 yield
               end
@@ -83,9 +85,9 @@ module OpenHAB
               new_state = Conditions.new_state_from(inputs)
               if @conditions.from? && new_state != @tracking_from &&
                  @conditions.process(mod: nil, inputs: { "state" => new_state })
-                logger.trace("Item changed from #{old_state} to #{new_state} for #{self}, keep waiting.")
+                logger.trace { "Item changed from #{old_state} to #{new_state} for #{self}, keep waiting." }
               else
-                logger.trace("Item changed from #{old_state} to #{new_state} for #{self}, canceling timer.")
+                logger.trace { "Item changed from #{old_state} to #{new_state} for #{self}, canceling timer." }
                 timer.cancel
                 # Reprocess trigger delay after canceling to track new state (if guards matched, etc)
                 process(mod: mod, inputs: inputs, &block)

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

@@ -30,7 +30,7 @@ module OpenHAB
             # @return [true, false] if the conditions passed (and therefore the block was run)
             #
             def process(mod:, inputs:)
-              logger.trace("Checking #{inputs} against condition trigger #{self}")
+              logger.trace { "Checking #{inputs} against condition trigger #{self}" }
               unless check_value(Conditions.old_state_from(inputs), @from) &&
                      check_value(Conditions.new_state_from(inputs), @to) &&
                      check_value(inputs["command"], @command)

data/lib/openhab/dsl/rules/triggers/cron/cron_handler.rb

@@ -45,7 +45,7 @@ module OpenHAB
               synchronized do
                 super(callback)
                 @schedule = @scheduler.schedule(self, @expression)
-                logger.trace("Scheduled cron job '#{@expression}' for trigger '#{@trigger.id}'.")
+                logger.trace { "Scheduled cron job '#{@expression}' for trigger '#{@trigger.id}'." }
               end
             end
 
@@ -76,7 +76,7 @@ module OpenHAB
                 @schedule.cancel(true)
                 @schedule = nil
               end
-              logger.trace("cancelled job for trigger '#{@trigger.id}'.")
+              logger.trace { "cancelled job for trigger '#{@trigger.id}'." }
             end
           end
 

data/lib/openhab/dsl/rules/triggers/watch/watch_handler.rb

@@ -140,7 +140,7 @@ module OpenHAB
               # Invoked by java super class to check if sub directories should be watched
               # @return [false] false
               def watchSubDirectories
-                logger.trace("watchSubDirectories returning #{@subdirs}")
+                logger.trace { "watchSubDirectories returning #{@subdirs}" }
                 @subdirs
               end
 

data/lib/openhab/dsl/timer_manager.rb

@@ -45,7 +45,7 @@ module OpenHAB
       # Add a timer that is now active
       # @!visibility private
       def add(timer)
-        logger.trace("Adding #{timer} to timers")
+        logger.trace { "Adding #{timer} to timers" }
         @timers[timer] = 1
       end
 
@@ -54,7 +54,7 @@ module OpenHAB
       #
       # @!visibility private
       def delete(timer)
-        logger.trace("Removing #{timer} from timers")
+        logger.trace { "Removing #{timer} from timers" }
         return unless @timers.remove(timer) && timer.id
 
         @timers_by_id.remove(timer.id)
@@ -193,7 +193,7 @@ module OpenHAB
         logged = false
         # don't use #each, in case timers are scheduling more timers
         until @timers.empty?
-          logger.trace("Canceling #{@timers.length} timers") unless logged
+          logger.trace { "Canceling #{@timers.length} timers" } unless logged
           logged = true
           timer = @timers.keys.first
           timer.cancel

data/lib/openhab/dsl/version.rb

@@ -4,6 +4,6 @@ module OpenHAB
   module DSL
     # Version of openHAB helper libraries
     # @return [String]
-    VERSION = "5.27.0"
+    VERSION = "5.27.2"
   end
 end

data/lib/openhab/dsl.rb

@@ -1072,6 +1072,16 @@ module OpenHAB
       Thread.current[:openhab_holiday_file] = file
     end
 
+    #
+    # @param [String] string Date/time string to parse.
+    #   The string can be in any format recognized by:
+    #   - {OpenHAB::CoreExt::Java::LocalTime.parse LocalTime.parse}
+    #   - {OpenHAB::CoreExt::Java::LocalDate.parse LocalDate.parse}
+    #   - {OpenHAB::CoreExt::Java::MonthDay.parse MonthDay.parse}, e.g. "12-25"
+    #   - {OpenHAB::CoreExt::Java::ZonedDateTime.parse ZonedDateTime.parse}
+    #   - [Time.parse](https://rubyapi.org/o/time#method-c-parse), e.g. "2021-01-01 12:00:", or "3:30pm"
+    # @return [LocalTime, LocalDate, MonthDay, ZonedDateTime, Time] the parsed date/time
+    #
     # @!visibility private
     def try_parse_time_like(string)
       return string unless string.is_a?(String)
@@ -1096,10 +1106,10 @@ module OpenHAB
       return super unless args.empty? && !block_given?
 
       if (context = Thread.current[:openhab_context]) && context.key?(method)
-        logger.trace("DSL#method_missing found context variable: '#{method}'")
+        logger.trace { "DSL#method_missing found context variable: '#{method}'" }
         return context[method]
       elsif Core.ui_context&.key?(method)
-        logger.trace("DSL#method_missing found UI context variable: '#{method}'")
+        logger.trace { "DSL#method_missing found UI context variable: '#{method}'" }
         return Core.ui_context[method]
       end
       super

data/lib/openhab/rspec/mocks/event_admin.rb

@@ -35,7 +35,7 @@ module OpenHAB
         def handle_event_internal(type, payload, topic, source)
           event_factory = @typed_event_factories[type]
           unless event_factory
-            logger.debug("Could not find an Event Factory for the event type '#{type}'.")
+            logger.debug { "Could not find an Event Factory for the event type '#{type}'." }
             return
           end
 
@@ -78,6 +78,7 @@ module OpenHAB
                 )
               end
             else
+              # Changing this to block syntax will cause specs to fail, for some reason.
               logger.trace("Skip event subscriber (#{event_subscriber.class}) because of its filter.")
             end
           end

data/lib/openhab/rspec/openhab/core/actions.rb

@@ -20,7 +20,7 @@ module OpenHAB
             button2 = nil,
             button3 = nil
           )
-            logger.debug("send_notification: #{email}, #{msg}, #{icon}, #{tag}, #{title}, #{id}, #{on_click}, #{attachment}, #{button1}, #{button2}, #{button3}") # rubocop:disable Layout/LineLength
+            logger.debug { "send_notification: #{email}, #{msg}, #{icon}, #{tag}, #{title}, #{id}, #{on_click}, #{attachment}, #{button1}, #{button2}, #{button3}" } # rubocop:disable Layout/LineLength
           end
 
           def send_broadcast_notification(
@@ -35,23 +35,23 @@ module OpenHAB
             button2 = nil,
             button3 = nil
           )
-            logger.debug("send_broadcast_notification: #{msg}, #{icon}, #{tag}, #{title}, #{id}, #{on_click}, #{attachment}, #{button1}, #{button2}, #{button3}") # rubocop:disable Layout/LineLength
+            logger.debug { "send_broadcast_notification: #{msg}, #{icon}, #{tag}, #{title}, #{id}, #{on_click}, #{attachment}, #{button1}, #{button2}, #{button3}" } # rubocop:disable Layout/LineLength
           end
 
           def hide_notification_by_reference_id(email, id)
-            logger.debug("hide_notification_by_reference_id: #{email}, #{id}")
+            logger.debug { "hide_notification_by_reference_id: #{email}, #{id}" }
           end
 
           def hide_notification_by_tag(email, tag)
-            logger.debug("hide_notification_by_tag: #{email}, #{tag}")
+            logger.debug { "hide_notification_by_tag: #{email}, #{tag}" }
           end
 
           def hide_broadcast_notification_by_reference_id(id)
-            logger.debug("hide_broadcast_notification_by_reference_id: #{id}")
+            logger.debug { "hide_broadcast_notification_by_reference_id: #{id}" }
           end
 
           def hide_broadcast_notification_by_tag(tag)
-            logger.debug("hide_broadcast_notification_by_tag: #{tag}")
+            logger.debug { "hide_broadcast_notification_by_tag: #{tag}" }
           end
         end
       end
@@ -59,7 +59,7 @@ module OpenHAB
       class Voice
         class << self
           def say(text, voice: nil, sink: nil, volume: nil)
-            logger.debug("say: #{text}")
+            logger.debug { "say: #{text}" }
           end
         end
       end
@@ -67,11 +67,11 @@ module OpenHAB
       class Audio
         class << self
           def play_sound(filename, sink: nil, volume: nil)
-            logger.debug("play_sound: #{filename}")
+            logger.debug { "play_sound: #{filename}" }
           end
 
           def play_stream(url, sink: nil)
-            logger.debug("play_stream: #{url}")
+            logger.debug { "play_stream: #{url}" }
           end
         end
       end

data/lib/openhab/rspec/suspend_rules.rb

@@ -9,7 +9,7 @@ module OpenHAB
       class ::OpenHAB::DSL::Rules::AutomationRule # rubocop:disable Style/ClassAndModuleChildren
         def execute(mod = nil, inputs = nil)
           if SuspendRules.suspended?
-            logger.trace("Skipping execution of #{uid} because rules are suspended.")
+            logger.trace { "Skipping execution of #{uid} because rules are suspended." }
             return
           end
           execute!(mod, inputs)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: openhab-scripting
 version: !ruby/object:Gem::Version
-  version: 5.27.0
+  version: 5.27.2
 platform: ruby
 authors:
 - Brian O'Connell
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-09-08 00:00:00.000000000 Z
+date: 2024-09-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler