openhab-scripting 2.10.1 → 2.13.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 91d900e25ee98409efcccd0151e3ce109ee89236d47ed7259602776d6c789a12
-  data.tar.gz: 321e7d7a009e8f7453398ff293dc8e58fe4da6c161e7b5024c025ccb75408474
+  metadata.gz: '089e0d97f175f04c563675de122c80d0f5988be5ea38be8833736e4e73341a75'
+  data.tar.gz: 4b98810dd2e4f01772272fc10e457b57e81ade54c19b1dc59a72afd85c368b5a
 SHA512:
-  metadata.gz: e3725cc8a1b3091991437bb34fc7bf43d81e88a81e19b76bc60648128e1e7c19ff30ac0274acc5a41fa9900f753b867f8537c42157509c4684e1239f6f48aaf9
-  data.tar.gz: 6d99214333852ddcdae333c45e664f1fc83a2ee31ffaee6f61202caf603abcf52257ac86587ebad336b9e1fd98a983f1c6de7bee47a5a2ffae210a53675cd09d
+  metadata.gz: 67abc23128c151fdeb509f7a6c770696e0221b6a41a6e8af36628c891aa959c72e7ed7123d1e6d9711da3fe862eba2adaac9e3692e1026193cc7638e11de926a
+  data.tar.gz: 4348d10e590f9e8179c3559c05ae2417cd9a75150d069e98f1566316e60a5d5ed12b4057b0d67272e97d92c813cbf337c97dac8d69b1cc5538b06576aa3084f7

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

@@ -22,7 +22,17 @@ module OpenHAB
             item = $ir.getItem(name)
             # rubocop: enable Style/GlobalVars
             (item.is_a? GroupItem) ? nil : item
+          rescue Java::OrgOpenhabCoreItems::ItemNotFoundException
+            nil
           end
+
+          # Returns true if the given item name exists
+          # @param name [String] Item name to check
+          # @return [Boolean] true if the item exists, false otherwise
+          def include?(name)
+            !$ir.getItems(name).empty?
+          end
+          alias key? include?
         end
 
         java_import org.openhab.core.items.GroupItem

data/lib/openhab/core/dsl/monkey_patch/items/dimmer_item.rb

@@ -14,6 +14,7 @@ Dimmer = DimmerItem
 #
 # rubocop:disable Style/ClassAndModuleChildren
 class Java::OrgOpenhabCoreLibraryItems::DimmerItem
+  include Comparable
   # rubocop:enable Style/ClassAndModuleChildren
   java_import org.openhab.core.library.types.DecimalType
   java_import org.openhab.core.library.types.IncreaseDecreaseType
@@ -85,6 +86,38 @@ class Java::OrgOpenhabCoreLibraryItems::DimmerItem
     target
   end
 
+  #
+  # Compare DimmerItem to supplied object
+  #
+  # @param [Object] other object to compare to
+  #
+  # @return [Integer] -1,0,1 or nil depending on value supplied, nil comparison to supplied object is not possible.
+  #
+  def <=>(other)
+    logger.trace("Comparing #{self} to #{other}")
+    case other
+    when DimmerItem, NumberItem
+      state <=> other.state
+    when DecimalType
+      state <=> other
+    else
+      to_i <=> other.to_i
+    end
+  end
+
+  #
+  # Compare DimmerItem to supplied object.
+  # The == operator needs to be overridden because the parent java object
+  # has .equals which overrides the <=> operator above
+  #
+  # @param [Object] other object to compare to
+  #
+  # @return [Integer] true if the two objects contain the same value, false otherwise
+  #
+  def ==(other)
+    (self <=> other).zero?
+  end
+
   #
   # Check if dimmer has a state and state is not zero
   #
@@ -105,6 +138,15 @@ class Java::OrgOpenhabCoreLibraryItems::DimmerItem
 
   alias to_int to_i
 
+  #
+  # Return the string representation of the dimmer item
+  #
+  # @return [String] String version of the dimmer value
+  #
+  def to_s
+    to_i.to_s
+  end
+
   #
   # Check if dimmer is on
   #

data/lib/openhab/core/dsl/monkey_patch/ruby/number.rb

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require 'core/duration'
-
 module OpenHAB
   module Core
     module DSL
@@ -14,44 +12,18 @@ module OpenHAB
             include OpenHAB::Core
 
             #
-            # Create Duration with unit of seconds
-            #
-            # @return [OpenHAB::Core::Duration] Duration with number of seconds from self
-            #
-            def seconds
-              Duration.new(temporal_unit: :SECONDS, amount: self)
-            end
-
-            #
-            # Create Duration with unit of milliseconds
-            #
-            # @return [OpenHAB::Core::Duration] Duration with number of milliseconds from self
-            #
-            def milliseconds
-              Duration.new(temporal_unit: :MILLISECONDS, amount: self)
-            end
-
-            #
-            # Create Duration with unit of minutes
-            #
-            # @return [OpenHAB::Core::Duration] Duration with number of minutes from self
-            #
-            def minutes
-              Duration.new(temporal_unit: :MINUTES, amount: self)
-            end
-
-            #
-            # Create Duration with unit of hours
+            # Create Duration with the specified unit
             #
-            # @return [OpenHAB::Core::Duration] Duration with number of hours from self
+            # @return [Java::JavaTime::Duration] Duration with number of units from self
             #
-            def hours
-              Duration.new(temporal_unit: :HOURS, amount: self)
+            %w[millis seconds minutes hours].each do |unit|
+              define_method(unit) { Java::JavaTime::Duration.public_send("of_#{unit}", self) }
             end
 
             alias second seconds
-            alias millisecond milliseconds
-            alias ms milliseconds
+            alias millisecond millis
+            alias milliseconds millis
+            alias ms millis
             alias minute minutes
             alias hour hours
           end

data/lib/openhab/core/dsl/rule/cron.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require 'java'
-require 'core/duration'
 require 'core/dsl/time_of_day'
 require 'core/cron'
 
@@ -64,8 +63,10 @@ module OpenHAB
               expression_map = EXPRESSION_MAP[value]
               expression_map = at_condition(expression_map, at) if at
               cron(map_to_cron(expression_map))
-            when Duration
-              cron(map_to_cron(value.cron_map))
+            when Java::JavaTime::Duration
+              raise ArgumentError, '"at" cannot be used with duration' if at
+
+              cron(map_to_cron(duration_to_map(value)))
             else
               raise ArgumentExpression, 'Unknown interval' unless expression_map
             end
@@ -93,6 +94,28 @@ module OpenHAB
             %i[second minute hour dom month dow].map { |field| map.fetch(field) }.join(' ')
           end
 
+          #
+          # Convert a Java Duration to a map for the map_to_cron method
+          #
+          # @param duration [Java::JavaTime::Duration] The duration object
+          #
+          # @return [Hash] a map suitable for map_to_cron
+          #
+          def duration_to_map(duration)
+            if duration.to_millis_part.zero? && duration.to_nanos_part.zero? && duration.to_days.zero?
+              %i[second minute hour].each do |unit|
+                to_unit_part = duration.public_send("to_#{unit}s_part")
+                next unless to_unit_part.positive?
+
+                to_unit = duration.public_send("to_#{unit}s")
+                break unless to_unit_part == to_unit
+
+                return EXPRESSION_MAP[unit].merge(unit => "*/#{to_unit}")
+              end
+            end
+            raise ArgumentError, "Cron Expression not supported for duration: #{duration}"
+          end
+
           #
           # If an at time is provided, parse that and merge the new fields into the expression.
           #

data/lib/openhab/core/dsl/rule/item.rb

@@ -97,6 +97,7 @@ module OpenHAB
           #
           def updated(*items, to: nil)
             items.flatten.each do |item|
+              item = item.group if item.is_a? Group
               logger.trace("Creating updated trigger for item(#{item}) to(#{to})")
               [to].flatten.each do |to_state|
                 case item

data/lib/openhab/core/dsl/rule/rule.rb

@@ -274,7 +274,7 @@ module OpenHAB
                   process_trigger_delay(mod, inputs)
                 else
                   logger.trace("Item changed to #{state} for #{trigger_delay}, rescheduling timer.")
-                  trigger_delay.timer.reschedule(ZonedDateTime.now.plus(Java::JavaTime::Duration.ofMillis(duration.to_ms)))
+                  trigger_delay.timer.reschedule(ZonedDateTime.now.plus(duration))
                 end
               end
             else
@@ -303,6 +303,23 @@ module OpenHAB
             false
           end
 
+          #
+          # Patch event to include event.item when it doesn't exist
+          # This is to patch a bug see https://github.com/boc-tothefuture/openhab-jruby/issues/75
+          # It may be fixed in the openhab core in the future, in which case, this patch will no longer be necessary
+          #
+          # @param [OpenHAB Event] event to check for item accessor
+          # @param [OpenHAB Event Inputs] inputs inputs to running rule
+          #
+          def add_event_item(event, inputs)
+            return if event.nil? || defined?(event.item)
+
+            class << event
+              attr_accessor :item
+            end
+            event.item = inputs&.dig('triggeringItem')
+          end
+
           #
           # Process the run queue
           #
@@ -317,7 +334,7 @@ module OpenHAB
               when RuleConfig::Run
 
                 event = inputs&.dig('event')
-
+                add_event_item(event, inputs)
                 logger.trace { "Executing rule '#{name}' run block with event(#{event})" }
                 task.block.call(event)
               when RuleConfig::Trigger

data/lib/openhab/core/dsl/timers.rb

@@ -4,8 +4,6 @@ require 'java'
 require 'delegate'
 require 'forwardable'
 
-require 'core/duration'
-
 module OpenHAB
   module Core
     module DSL
@@ -49,7 +47,7 @@ module OpenHAB
 
             semaphore.synchronize do
               @timer = ScriptExecution.createTimer(
-                ZonedDateTime.now.plus(Java::JavaTime::Duration.ofMillis(@duration.to_ms)), @block
+                ZonedDateTime.now.plus(@duration), @block
               )
               super(@timer)
             end
@@ -64,7 +62,7 @@ module OpenHAB
           #
           def reschedule(duration = nil)
             duration ||= @duration
-            @timer.reschedule(ZonedDateTime.now.plus(Java::JavaTime::Duration.ofMillis(duration.to_ms)))
+            @timer.reschedule(ZonedDateTime.now.plus(duration))
           end
         end
 

data/lib/openhab/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: openhab-scripting
 version: !ruby/object:Gem::Version
-  version: 2.10.1
+  version: 2.13.1
 platform: ruby
 authors:
 - Brian O'Connell
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-01-31 00:00:00.000000000 Z
+date: 2021-02-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -76,7 +76,6 @@ files:
 - lib/openhab/core/dsl/timers.rb
 - lib/openhab/core/dsl/types/quantity.rb
 - lib/openhab/core/dsl/units.rb
-- lib/openhab/core/duration.rb
 - lib/openhab/core/log.rb
 - lib/openhab/core/patch_load_path.rb
 - lib/openhab/core/startup_delay.rb

data/lib/openhab/core/duration.rb

@@ -1,69 +0,0 @@
-# frozen_string_literal: true
-
-require 'openhab/core/cron'
-
-module OpenHAB
-  module Core
-    #
-    # This class represents a duration of time
-    #
-    class Duration
-      include OpenHAB::Core::Cron
-
-      # @return [Array] of supported temperal units (milliseconds, seconds, minutes and hours)
-      TEMPORAL_UNITS = %i[MILLISECONDS SECONDS MINUTES HOURS].freeze
-
-      #
-      # Create a new Duration object
-      #
-      # @param [Symbol] temporal_unit Unit for duration
-      # @param [Integer] amount of that unit
-      #
-      def initialize(temporal_unit:, amount:)
-        unless TEMPORAL_UNITS.include? temporal_unit
-          raise ArgumentError,
-                "Unexpected Temporal Unit: #{temporal_unit}"
-        end
-
-        @temporal_unit = temporal_unit
-        @amount = amount
-      end
-
-      #
-      # Return a map
-      #
-      # @return [Map] Map with fields representing this duration @see OpenHAB::Core::Cron
-      #
-      def cron_map
-        case @temporal_unit
-        when :SECONDS
-          cron_expression_map.merge(second: "*/#{@amount}")
-        when :MINUTES
-          cron_expression_map.merge(minute: "*/#{@amount}")
-        when :HOURS
-          cron_expression_map.merge(hour: "*/#{@amount}")
-        else
-          raise ArgumentError, "Cron Expression not supported for temporal unit: #{temporal_unit}"
-        end
-      end
-
-      #
-      # Convert the duration to milliseconds
-      #
-      # @return [Integer] Duration in milliseconds
-      #
-      def to_ms
-        case @temporal_unit
-        when :MILLISECONDS
-          @amount
-        when :SECONDS
-          @amount * 1000
-        when :MINUTES
-          @amount * 1000 * 60
-        when :HOURS
-          @amount * 1000 * 60 * 60
-        end
-      end
-    end
-  end
-end