openhab-scripting 4.26.1 → 4.27.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8febc3e32091a7b34dcf12ddd329eaa6ab490b36e1a229189d05bd3651628b76
-  data.tar.gz: 94b67193670f7b4752c4a9229325e3bfedc4809f14c2e3f9d7552fd2e681d14e
+  metadata.gz: 16eb15617a1a088c561a6d12e1f6e2ba39459491c9e831933ff235d3177ba225
+  data.tar.gz: 40f21e9e4bb53c04f1bd9e4ee51b834756783218584f05728e26860eb95d491c
 SHA512:
-  metadata.gz: 134a6533d7616dede1e3badde2eca2776c7f6af51eb4b58172a4c55e4906f9b6d3e4101303324448b65bc9ae8ca98fdeff8ff7b95b964e18aee42c6ccf6a295a
-  data.tar.gz: 26bc59a29615e39b90c37caeca36bc63b2a32c3a6693e8032be6ee68f4fd0d28059d0b879d3a6ad94877af85dfecfc45c9691e2364f9005acd966332933d36ba
+  metadata.gz: ac79f5e48d51ec69530c88fe05541e3c6051df205858371a5f450a226d680940058023cab20df0037c148293f60e9b85e20768669c48657ead01ee828097a16a
+  data.tar.gz: e8b218bdd0ecedbb3910c9859bb43e8f06c5372ffc7aa94a6fae0abe7f5defeff8d3a3f6fdee1af9337e8a2684ea2ce194d9e09344f5144650c38b1e91c9f9ea

data/lib/openhab/core/entity_lookup.rb

@@ -5,6 +5,7 @@ require 'java'
 require 'set'
 
 require 'openhab/log/logger'
+require_relative 'item_proxy'
 
 # Automation lookup and injection of OpenHab entities
 
@@ -91,7 +92,8 @@ module OpenHAB
       def self.lookup_item(name)
         logger.trace("Looking up item(#{name})")
         name = name.to_s if name.is_a? Symbol
-        $ir.get(name) # rubocop: disable Style/GlobalVars
+        item = $ir.get(name) # rubocop: disable Style/GlobalVars
+        ItemProxy.new(item) unless item.nil?
       end
     end
   end

data/lib/openhab/core/item_proxy.rb

@@ -0,0 +1,29 @@
+# frozen_string_literal: true
+
+require 'delegate'
+require 'forwardable'
+
+module OpenHAB
+  module Core
+    # Class is a proxy to underlying item
+    class ItemProxy < Delegator
+      extend Forwardable
+      def_delegators :__getobj__, :class, :is_a?, :kind_of?, :instance_of?
+
+      #
+      # Set the proxy item (called by super)
+      #
+      def __setobj__(item)
+        # Convert name to java version for faster lookups
+        @item_name = item.name.to_java
+      end
+
+      #
+      # Lookup item from item registry
+      #
+      def __getobj__
+        $ir.get(@item_name) # rubocop:disable Style/GlobalVars
+      end
+    end
+  end
+end

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

@@ -33,6 +33,13 @@ module OpenHAB
           def ACCEPTED_DATA_TYPES
             [org.openhab.core.types.UnDefType.java_class].freeze
           end
+
+          #
+          # Override to support ItemProxy
+          #
+          def ===(other)
+            other.instance_of?(self)
+          end
         end
         # rubocop:enable Naming/MethodName
 

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

@@ -6,11 +6,9 @@ require 'openhab/dsl/types/types'
 require_relative 'item_registry'
 
 require_relative 'generic_item'
-
 require_relative 'switch_item'
 require_relative 'date_time_item'
 require_relative 'dimmer_item'
-
 require_relative 'color_item'
 require_relative 'contact_item'
 require_relative 'group_item'

data/lib/openhab/dsl/states.rb

@@ -48,7 +48,9 @@ module OpenHAB
       # @return [StateStorage] item states
       #
       def store_states(*items)
-        items = items.flatten
+        items = items.flatten.map do |item|
+          item.respond_to?(:__getobj__) ? item.__getobj__ : item
+        end
         states = StateStorage.new(BusEvent.storeStates(*items).to_h)
         if block_given?
           yield

data/lib/openhab/dsl/time/month_day.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require 'java'
+
 module OpenHAB
   module DSL
     # Support for time related functions
@@ -8,40 +10,21 @@ module OpenHAB
       module MonthDayRange
         include OpenHAB::Log
 
-        java_import java.time.Year
-
-        # Lambdas are used to calculate the year for the month day
-        # which must happen during evaluation time to support that rules
-        # creation and evaluation for execution are done in distinct phases
-        @current_year = -> { return Year.now }
-        @next_year = -> { return Year.now.plus_years(1) }
-
-        class << self
-          attr_reader :current_year, :next_year
-        end
-
         # Creates a range that can be compared against MonthDay objects, strings
         # or anything responding to 'to_date' to see if they are within the range
         # @return Range object representing a MonthDay Range
-        # rubocop:disable Metrics/AbcSize
-        # Range method cannot be broken up cleaner
         def self.range(range)
-          logger.trace "Creating MonthDay range from #{range}"
           raise ArgumentError, 'Supplied object must be a range' unless range.is_a? Range
 
           start = MonthDay.parse(range.begin)
           ending = MonthDay.parse(range.end)
 
-          logger.trace "Month Day Range Start(#{start}) - End (#{ending})"
-
-          # Wrap to next year if ending day of month is before starting day of month
-          ending_year = ending < start ? next_year : current_year
+          start_range = DayOfYear.new(month_day: start, range: start..ending)
+          ending_range = DayOfYear.new(month_day: ending, range: start..ending)
+          logger.trace "Month Day Range Start(#{start}) - End (#{ending}) - Created from (#{range})"
 
-          start_range = MonthDayRangeElement.new(month_day: start, year: current_year)
-          ending_range = MonthDayRangeElement.new(month_day: ending, year: ending_year)
           range.exclude_end? ? (start_range...ending_range) : (start_range..ending_range)
         end
-        # rubocop:enable Metrics/AbcSize
 
         # Checks if supplied range can be converted to a month day range
         # @param [Range] range to check begin and end values of
@@ -52,54 +35,80 @@ module OpenHAB
           MonthDay.day_of_month?(range.begin) && MonthDay.day_of_month?(range.end)
         end
 
-        # Represents a range element for a MonthDay object
-        # The LocalDate (MonthDay + Year) is dynamically calculated to allow for
-        # being used as a guard during rule evaluation
-        class MonthDayRangeElement
+        # Converts a MonthDay to a day of year
+        # which is represented as a number from 1 to 732 to support comparisions when the range overlaps a year boundary
+        class DayOfYear
           include Comparable
           include OpenHAB::Log
           java_import java.time.LocalDate
-          java_import java.time.Year
+
+          attr_accessor :month_day
+
+          # Number of days in a leap year
+          DAYS_IN_YEAR = 366
 
           # Create a new MonthDayRange element
           # @param [MonthDay] MonthDay element
-          # @param [Lambda] year lambda to calculate year to convert MonthDay to LocalDate
+          # @param [Range] Underlying MonthDay range
           #
-          def initialize(month_day:, year:)
+          def initialize(month_day:, range:)
             @month_day = month_day
-            @year = year
-          end
-
-          # Convert into a LocalDate using year lambda supplied in initializer
-          def to_local_date
-            @year.call.at_month_day(@month_day)
+            @range = range
           end
 
           # Returns the MonthDay advanced by 1 day
           # Required by Range class
           def succ
-            next_date = to_local_date.plus_days(1)
-            # Handle rollover to next year
-            year = -> { Year.from(next_date) }
-            MonthDayRangeElement.new(month_day: MonthDay.from(next_date), year: year)
+            next_day_of_month = @month_day.day_of_month + 1
+            next_month = @month_day.month_value
+
+            if next_day_of_month > @month_day.month.max_length
+              next_day_of_month = 1
+              next_month = @month_day.month.plus(1).value
+            end
+
+            DayOfYear.new(month_day: MonthDay.of(next_month, next_day_of_month), range: @range)
+          end
+
+          #
+          # Offset by 1 year if the range begin is greater than the range end
+          # and if the month day is less than the begining of the range
+          # @return [Number] 366 if the month_day should be offset by a year
+          def offset
+            @range.begin > @range.end && month_day < @range.begin ? DAYS_IN_YEAR : 0
+          end
+
+          #
+          # Calculate the day within the range for the underlying month day
+          # @return [Number] Representation of the MonthDay as a number from 1 to 732
+          def day_in_range
+            @day_in_range ||= month_day.max_day_of_year + offset
           end
 
           # Compare MonthDayRangeElement to other objects as required by Range class
-          # rubocop:disable Metrics/AbcSize
-          # Case statement needs to work against multiple types
-          def <=>(other)
+          def <=>(other) # rubocop:disable Metrics/AbcSize
             case other
-            when LocalDate then to_local_date.compare_to(other)
-            when Date then self.<=>(LocalDate.of(other.year, other.month, other.day))
-            when MonthDay then self.<=>(MonthDayRange.current_year.call.at_month_day(other))
+            when DayOfYear then day_in_range.<=>(other.day_in_range)
+            when MonthDay then self.<=>(DayOfYear.new(month_day: other, range: @range))
+            when LocalDate then self.<=>(MonthDay.of(other.month_value, other.day_of_month))
+            when Date then self.<=>(MonthDay.of(other.month, other.day))
             else
-              return self.<=>(other.to_local_date) if other.respond_to? :to_local_date
-              return self.<=>(other.to_date) if other.respond_to? :to_date
+              return self.<=>(other.to_local_date) if other.respond_to?(:to_local_date)
+              return self.<=>(other.to_date) if other.respond_to?(:to_date)
 
               raise "Unable to convert #{other.class} to compare to MonthDay"
             end
           end
-          # rubocop:enable Metrics/AbcSize
+        end
+      end
+
+      java_import java.time.Month
+      # Extend Month with helper method
+      class Month
+        # Calcalute and memoize the maximum number of days in a year before this month
+        # @return [Number] maximum nummber of days in a year before this month
+        def max_days_before
+          @max_days_before ||= Month.values.select { |month| month < self }.sum(&:max_length)
         end
       end
 
@@ -108,6 +117,7 @@ module OpenHAB
       class MonthDay
         include OpenHAB::Log
         java_import java.time.format.DateTimeFormatter
+        java_import java.time.Month
 
         #
         # Constructor
@@ -117,11 +127,9 @@ module OpenHAB
         #
         # @return [Object] MonthDay object
         #
-        # rubocop: disable Naming/MethodParameterName
-        def self.new(m:, d:)
+        def self.new(m:, d:) # rubocop:disable Naming/MethodParameterName
           MonthDay.of(m, d)
         end
-        # rubocop: enable Naming/MethodParameterName
 
         # Parse MonthDay string as defined with by Monthday class without leading double dash "--"
         def self.parse(string)
@@ -136,11 +144,23 @@ module OpenHAB
           /^-*[01][0-9]-[0-3]\d$/.match? obj.to_s
         end
 
+        # Get the maximum (supports leap years) day of the year this month day could be
+        def max_day_of_year
+          day_of_month + month.max_days_before
+        end
+
         # Remove -- from MonthDay string representation
         def to_s
           to_string.delete_prefix('--')
         end
 
+        # Checks if MonthDay is between the dates of the supplied range
+        # @param [Range] range to check against MonthDay
+        # @return [true,false] true if the MonthDay falls within supplied range, false otherwise
+        def between?(range)
+          MonthDayRange.range(range).cover? self
+        end
+
         # remove the inherited #== method to use our <=> below
         remove_method :==
 
@@ -151,6 +171,9 @@ module OpenHAB
           case other
           when String
             self.<=>(MonthDay.parse(other))
+          when OpenHAB::DSL::Between::MonthDayRange::DayOfYear
+            # Compare with DayOfYear and invert result
+            -other.<=>(self)
           else
             super
           end

data/lib/openhab/dsl/types/open_closed_type.rb

@@ -20,8 +20,8 @@ module OpenHAB
         # Invert the type
         # @return [OpenClosedType] +OPEN+ if +CLOSED+, +CLOSED+ if +OPEN+
         def !
-          return OPEN if open?
-          return CLOSED if closed?
+          return CLOSED if open?
+          return OPEN if closed?
         end
       end
     end

data/lib/openhab/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: openhab-scripting
 version: !ruby/object:Gem::Version
-  version: 4.26.1
+  version: 4.27.0
 platform: ruby
 authors:
 - Brian O'Connell
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-12-29 00:00:00.000000000 Z
+date: 2022-01-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -47,6 +47,7 @@ extra_rdoc_files: []
 files:
 - lib/openhab.rb
 - lib/openhab/core/entity_lookup.rb
+- lib/openhab/core/item_proxy.rb
 - lib/openhab/core/load_path.rb
 - lib/openhab/core/openhab_setup.rb
 - lib/openhab/core/osgi.rb