openhab-scripting 2.25.0 → 2.27.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7c6a3905160edfd4bf027cb1afbac0af180f0e79641b4558a875d0f89cc275f8
-  data.tar.gz: 97bc0f35a27b2ccd0fdcac64bfa0e5228b89a68085d5bf8213514e38f33a2add
+  metadata.gz: 2076584e55cc649fda227034943f6a412fb7a43a52479aec0ff3f33452266cc4
+  data.tar.gz: a7eb3154b9f436987883106bd39ab2a6574190abd2236d7c4898420624cbea87
 SHA512:
-  metadata.gz: 39c074134bcd470a537c57256c57203ad1b750daa7137feea17a18dbc457c990dea3b2a985d271a9d6f5d4f7ae9c215c2444cdd4745ca6ae51aee4eebba8ae0f
-  data.tar.gz: a8d60e555d4dc743bfdcda7e222281cb40e5fcd4bbea970e0d5473791e1f5e4d2cc590fa8256ba7d60a88a0c661ec7fb13ddc458d923e91166f3228333669926
+  metadata.gz: 5e1923dd15436484c0bd72c9aa83ab4a72d554e7cc17c7d9c06e4eda0b335a13b283405acdf8282e75bb1488c245c890b9b7151c0de03f8316d6367ad7e5be6f
+  data.tar.gz: f7e218fbf8b28b27ef154bcfa42dc167f0cf4bede89fe4772489a4aa8cec77d9827be83d32e748b663876a5286649217322d840b638ea73a37200ef82cfe34a9

data/lib/openhab/core/entity_lookup.rb

@@ -87,25 +87,25 @@ module OpenHAB
       # @return [Object] the ruby wrapper for the item
       #
       # rubocop: disable Metrics/MethodLength
-      # rubocop: disable Metrics/AbcSize
       # Disabled line length and branch size - case dispatch pattern
       private_class_method def self.decorate_item(item)
         case item
         when GroupItem
           decorate_group(item)
-        when Java::Org.openhab.core.library.items::NumberItem
+        when Java::OrgOpenhabCoreLibraryItems::NumberItem
           OpenHAB::DSL::Items::NumberItem.new(item)
-        when Java::Org.openhab.core.library.items::StringItem
+        when Java::OrgOpenhabCoreLibraryItems::StringItem
           OpenHAB::DSL::Items::StringItem.new(item)
-        when Java::Org.openhab.core.library.items::DateTimeItem
+        when Java::OrgOpenhabCoreLibraryItems::DateTimeItem
           OpenHAB::DSL::Items::DateTimeItem.new(item)
-        when Java::Org.openhab.core.library.items::RollershutterItem
+        when Java::OrgOpenhabCoreLibraryItems::RollershutterItem
           OpenHAB::DSL::Items::RollershutterItem.new(item)
+        when Java::OrgOpenhabCoreLibraryItems::PlayerItem
+          OpenHAB::DSL::Items::PlayerItem.new(item)
         else
           item
         end
       end
-      # rubocop: enable Metrics/AbcSize
       # rubocop: enable Metrics/MethodLength
 
       #

data/lib/openhab/dsl/dsl.rb

@@ -15,6 +15,7 @@ require 'openhab/dsl/things'
 require 'openhab/dsl/items/items'
 require 'openhab/dsl/items/datetime_item'
 require 'openhab/dsl/items/number_item'
+require 'openhab/dsl/items/player_item'
 require 'openhab/dsl/time_of_day'
 require 'openhab/dsl/gems'
 require 'openhab/dsl/persistence'

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

@@ -17,11 +17,13 @@ module OpenHAB
       class DateTimeItem
         extend Forwardable
         extend OpenHAB::DSL::Items::ItemDelegate
-
+        extend OpenHAB::DSL::Items::ItemCommand
         include Comparable
 
         def_item_delegator :@datetime_item
 
+        item_type Java::OrgOpenhabCoreLibraryItems::DateTimeItem
+
         #
         # Create a new DateTimeItem
         #

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

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
 require 'java'
+require 'openhab/log/logger'
 
 module OpenHAB
   module DSL
@@ -9,19 +10,25 @@ module OpenHAB
       # Holds methods to automatically generate commands and
       # accessors for items
       module ItemCommand
+        include OpenHAB::Log
+
         #
         # For every value in the supplied enumeration create a corresponding method mapped to the lowercase
         # string representation of the enum value For example, an enum with values of STOP and START
         # would create methods stop() and start() that send the corresponding STOP and START commands to the item
         #
         # @param [Java::JavaLang::Enum] command_enum Enumeration to create commands for
+        # @param [Hash] optional hash in which if a generated method name mactches a key, the value of that key
+        #   will be used as the method name instead, for example `:play? => :playing?`
         #
         #
-        def item_command(command_enum)
+        def item_command(command_enum, methods = {})
           # rubocop:disable Style/HashEachMethods
           # Disable rule because Java enum does not support each_value
           command_enum.values.each do |command|
             command_method = command.to_s.downcase
+            command_method = methods.transform_keys(&:to_sym).fetch(command_method.to_sym, command_method)
+            logger.trace("Creating command method (#{command_method}) for #{self.class}")
             define_method(command_method) do
               self.command(command)
             end
@@ -37,20 +44,42 @@ module OpenHAB
         #
         # @param [Java::JavaLang::Enum] command_enum Enumeration to create methods for each value
         #   to check if current state matches that enum
-        # @yield [state] Optional block that can be used to transform state prior to comparison
+        # @param [Hash] optional hash in which if a generated method name mactches a key, the value of that key
+        #   will be used as the method name instead, for example `:play? => :playing?`
         #
         #
-        def item_state(command_enum)
+        def item_state(command_enum, methods = {})
           # rubocop:disable Style/HashEachMethods
           # Disable rule because Java enum does not support each_value
           command_enum.values.each do |command|
             status_method = "#{command.to_s.downcase}?"
+            status_method = methods.transform_keys(&:to_sym).fetch(status_method.to_sym, status_method)
+            logger.trace("Creating status method (#{status_method}) for #{self.class}")
             define_method(status_method) do
               state? && state.as(command_enum) == command
             end
           end
           # rubocop:enable Style/HashEachMethods
         end
+
+        #
+        # Extract the accepted state and command types from the specified OpenHAB
+        # Item class and pass them to item_state/item_command
+        #
+        # @param [Java::JavaLang::Class] item_class a Class that implements Java::OrgOpenhabCoreItems::Item
+        # @param [Hash] optional hash in which if a generated method name mactches a key, the value of that key
+        #   will be used as the method name instead, for example `:play? => :playing?`
+        #
+        def item_type(item_class, methods = {})
+          item_class.field_reader(:ACCEPTED_DATA_TYPES)
+          item_class.field_reader(:ACCEPTED_COMMAND_TYPES)
+          item_class.ACCEPTED_DATA_TYPES.select(&:is_enum)
+                    .grep_v(UnDefType)
+                    .each { |type| item_state(type.ruby_class, methods) }
+          item_class.ACCEPTED_COMMAND_TYPES.select(&:is_enum)
+                    .grep_v(UnDefType)
+                    .each { |type| item_command(type.ruby_class, methods) }
+        end
       end
     end
   end

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

@@ -33,7 +33,7 @@ module OpenHAB
         alias key? include?
       end
 
-      java_import org.openhab.core.items.GroupItem
+      java_import Java::OrgOpenhabCoreItems::GroupItem
       # Fetches all non-group items from the item registry
       # @return [OpenHAB::DSL::Items::Items]
       def items

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

@@ -17,6 +17,7 @@ module OpenHAB
       # way of breaking it up into multiple classes
       class NumberItem < Numeric
         extend OpenHAB::DSL::Items::ItemDelegate
+        extend OpenHAB::DSL::Items::ItemCommand
 
         def_item_delegator :@number_item
 
@@ -25,6 +26,8 @@ module OpenHAB
         java_import 'tec.uom.se.format.SimpleUnitFormat'
         java_import 'tec.uom.se.AbstractUnit'
 
+        item_type Java::OrgOpenhabCoreLibraryItems::NumberItem
+
         #
         # Create a new NumberItem
         #

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

@@ -0,0 +1,47 @@
+# frozen_string_literal: true
+
+require 'forwardable'
+require 'java'
+require 'openhab/dsl/items/item_command'
+require 'openhab/dsl/items/item_delegate'
+
+module OpenHAB
+  module DSL
+    module Items
+      #
+      # Delegator to OpenHAB Player Item
+      #
+      class PlayerItem
+        extend OpenHAB::DSL::Items::ItemCommand
+        extend OpenHAB::DSL::Items::ItemDelegate
+        extend Forwardable
+
+        def_item_delegator :@player_item
+
+        item_type Java::OrgOpenhabCoreLibraryItems::PlayerItem, play?: :playing?, pause?: :paused?,
+                                                                rewind?: :rewinding?, fastforward?: :fastforwarding?
+
+        # rubocop: disable Style/Alias
+        # Disabled because 'alias' does not work with the dynamically defined methods
+        alias_method :fast_forward, :fastforward
+        alias_method :fast_forwarding?, :fastforwarding?
+        # rubocop: enable Style/Alias
+
+        #
+        # Creates a new PlayerItem
+        #
+        # @param [Java::OrgOpenhabCoreLibraryItems::PlayerItem] player_item
+        #   The OpenHAB PlayerItem to delegate to
+        #
+        def initialize(player_item)
+          logger.trace("Wrapping #{player_item}")
+          @player_item = player_item
+
+          item_missing_delegate { @player_item }
+
+          super()
+        end
+      end
+    end
+  end
+end

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

@@ -19,13 +19,7 @@ module OpenHAB
 
         def_item_delegator :@rollershutter_item
 
-        java_import Java::OrgOpenhabCoreLibraryTypes::PercentType
-        java_import Java::OrgOpenhabCoreLibraryTypes::UpDownType
-        java_import Java::OrgOpenhabCoreLibraryTypes::StopMoveType
-
-        item_command Java::OrgOpenhabCoreLibraryTypes::StopMoveType
-        item_command Java::OrgOpenhabCoreLibraryTypes::UpDownType
-        item_state   Java::OrgOpenhabCoreLibraryTypes::UpDownType
+        item_type Java::OrgOpenhabCoreLibraryItems::RollershutterItem
 
         #
         # Creates a new RollershutterItem

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

@@ -13,7 +13,7 @@ module OpenHAB
       #
       class StringItem
         extend Forwardable
-
+        extend OpenHAB::DSL::Items::ItemCommand
         include Comparable
         extend OpenHAB::DSL::Items::ItemDelegate
 
@@ -23,6 +23,8 @@ module OpenHAB
 
         def_item_delegator :@string_item
 
+        item_type Java::OrgOpenhabCoreLibraryItems::StringItem
+
         #
         # Create a new StringItem
         #

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

@@ -25,7 +25,7 @@ module OpenHAB
 
           java_import Java::OrgOpenhabCoreLibraryTypes::OpenClosedType
 
-          item_state Java::OrgOpenhabCoreLibraryTypes::OpenClosedType
+          item_type Java::OrgOpenhabCoreLibraryItems::ContactItem
 
           #
           # Compares contacts to OpenClosedTypes

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

@@ -30,8 +30,7 @@ module OpenHAB
           extend Forwardable
           extend OpenHAB::DSL::Items::ItemCommand
 
-          item_state Java::OrgOpenhabCoreLibraryTypes::OnOffType
-          item_command Java::OrgOpenhabCoreLibraryTypes::IncreaseDecreaseType
+          item_type Java::OrgOpenhabCoreLibraryItems::DimmerItem
 
           def_delegator :state, :to_s
 

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

@@ -103,8 +103,7 @@ module OpenHAB
             # @return [OpenHAB::DSL::MonkeyPatch::Items::MetadataItem], or nil if the namespace doesn't exist
             #
             def [](namespace)
-              logger.trace("Namespaces (#{NamespaceAccessor.registry.getAll})")
-              logger.trace("Namespace (#{NamespaceAccessor.registry.get(MetadataKey.new(namespace, @item_name))})")
+              logger.trace("Getting metadata for item: #{@item_name}, namespace '#{namespace}'")
               metadata = NamespaceAccessor.registry.get(MetadataKey.new(namespace, @item_name))
               MetadataItem.new(metadata: metadata) if metadata
             end

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

@@ -23,8 +23,7 @@ module OpenHAB
 
           java_import Java::OrgOpenhabCoreLibraryTypes::OnOffType
 
-          item_command Java::OrgOpenhabCoreLibraryTypes::OnOffType
-          item_state Java::OrgOpenhabCoreLibraryTypes::OnOffType
+          item_type Java::OrgOpenhabCoreLibraryItems::SwitchItem
 
           alias truthy? on?
 

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

@@ -255,13 +255,28 @@ module OpenHAB
           event = inputs&.dig('event')
 
           while (task = run_queue.shift)
-            case task
-            when RuleConfig::Run then process_run_task(event, task)
-            when RuleConfig::Trigger then process_trigger_task(event, task)
-            when RuleConfig::Delay then process_delay_task(inputs, mod, run_queue, task)
-            when RuleConfig::Otherwise then process_otherwise_task(event, task)
+            if task.is_a? RuleConfig::Delay
+              process_delay_task(inputs, mod, run_queue, task)
+            else
+              process_task(event, task)
             end
           end
+        rescue StandardError => e
+          print_backtrace(e)
+        end
+
+        #
+        # Dispatch execution block tasks to different methods
+        #
+        # @param [OpenHab Event] event that triggered the rule
+        # @param [Task] task task containing otherwise block to execute
+        #
+        def process_task(event, task)
+          case task
+          when RuleConfig::Run then process_run_task(event, task)
+          when RuleConfig::Trigger then process_trigger_task(event, task)
+          when RuleConfig::Otherwise then process_otherwise_task(event, task)
+          end
         end
 
         #
@@ -316,6 +331,16 @@ module OpenHAB
           task.block.call(event)
         end
 
+        #
+        # Print error and stack trace without calls to internal classes
+        #
+        # @param [Exception] error A rescued error
+        #
+        def print_backtrace(error)
+          error = logger.clean_backtrace(error)
+          logger.error { "#{error.message} (#{error.class})\nIn rule: #{name}\n#{error.backtrace&.join("\n")}" }
+        end
+
         #
         # Create a new hash in which all elements are converted to strings
         #

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

@@ -79,7 +79,7 @@ module OpenHAB
             procs, items = conditions.flatten.partition { |condition| condition.is_a? Proc }
             logger.trace("Procs: #{procs} Items: #{items}")
 
-            items.each { |item| logger.trace("#{item} truthy? #{item.truthy?}") }
+            items.each { |item| logger.trace { "#{item} truthy? #{item.truthy?}" } }
 
             process_check(check_type: check_type, event: event, items: items, procs: procs)
           end

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

@@ -27,6 +27,8 @@ module OpenHAB
         config.guard = Guard::Guard.new(only_if: config.only_if, not_if: config.not_if)
         logger.trace { config.inspect }
         process_rule_config(config)
+      rescue StandardError => e
+        re_raise_with_backtrace(e)
       end
 
       #
@@ -44,6 +46,16 @@ module OpenHAB
 
       private
 
+      #
+      # Re-raises a rescued error to OpenHAB with added rule name and stack trace
+      #
+      # @param [Exception] error A rescued error
+      #
+      def re_raise_with_backtrace(error)
+        error = logger.clean_backtrace(error)
+        raise error, "#{error.message}\nIn rule: #{@rule_name}\n#{error.backtrace.join("\n")}"
+      end
+
       #
       # Process a rule based on the supplied configuration
       #

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

@@ -22,11 +22,14 @@ module OpenHAB
 
         def_delegator :datetime, :to_s
         def_delegator :zoned_date_time, :month_value, :month
+        def_delegator :zoned_date_time, :day_of_month, :mday
+        def_delegator :zoned_date_time, :day_of_year, :yday
         def_delegator :zoned_date_time, :minute, :min
         def_delegator :zoned_date_time, :second, :sec
         def_delegator :zoned_date_time, :nano, :nsec
         def_delegator :zoned_date_time, :to_epoch_second, :to_i
         alias inspect to_s
+        alias day mday
 
         java_import Java::OrgOpenhabCoreLibraryTypes::DateTimeType
         java_import java.time.ZonedDateTime
@@ -181,6 +184,15 @@ module OpenHAB
           utc_offset.zero?
         end
 
+        #
+        # Returns an integer representing the day of the week, 0..6, with Sunday == 0.
+        #
+        # @return [Integer] The day of week
+        #
+        def wday
+          zoned_date_time.day_of_week.value % 7
+        end
+
         #
         # The timezone
         #

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

@@ -144,8 +144,10 @@ module OpenHAB
 
         OPERATIONS.each do |operation, method|
           define_method(operation) do |other|
-            logger.trace("Executing math operation '#{operation}' on quantity #{inspect} "\
-                         "with other type #{other.class} and value #{other.inspect}")
+            logger.trace do
+              "Executing math operation '#{operation}' on quantity #{inspect} "\
+                         "with other type #{other.class} and value #{other.inspect}"
+            end
 
             a, b = to_qt(coerce(other).reverse)
             logger.trace("Coerced a='#{a}' with b='#{b}'")

data/lib/openhab/log/logger.rb

@@ -16,7 +16,14 @@ module OpenHAB
       java_import org.slf4j.LoggerFactory
 
       # @return [Array] Supported logging levels
-      LEVELS = %i[TRACE DEBUG WARN INFO ERROR].freeze
+      LEVELS = %i[trace debug warn info error].freeze
+      private_constant :LEVELS
+
+      #
+      # Regex for matching internal calls in a stack trace
+      #
+      INTERNAL_CALL_REGEX = %r{(openhab-scripting-.*/lib)|(org/jruby/)}.freeze
+      private_constant :INTERNAL_CALL_REGEX
 
       #
       # Create a new logger
@@ -32,11 +39,30 @@ module OpenHAB
       # def <level>(msg=nil, &block)
       #   log(severity: <level>, msg: msg, &block)
       # end
+      #
+      # Also creates methods to check if the different logging levels are enabled
+      #
       LEVELS.each do |level|
-        method = level.to_s.downcase
-        define_method(method.to_s) do |msg = nil, &block|
+        define_method(level) do |msg = nil, &block|
           log(severity: level, msg: msg, &block)
         end
+        define_method("#{level}_enabled?") { @sl4fj_logger.send("is_#{level}_enabled") }
+      end
+
+      #
+      # Cleans the backtrace of an error to remove internal calls. If logging is set
+      # to debug or lower, the full backtrace is kept
+      #
+      # @param [Exception] error An exception to be cleaned
+      #
+      # @return [Exception] the exception, potentially with a cleaned backtrace.
+      #
+      def clean_backtrace(error)
+        return error if debug_enabled?
+
+        backtrace = error.backtrace_locations.reject { |line| INTERNAL_CALL_REGEX.match? line.to_s }
+        error.set_backtrace(backtrace.map(&:to_s))
+        error
       end
 
       private
@@ -54,7 +80,7 @@ module OpenHAB
         raise ArgumentError, "Unknown Severity #{severity}" unless LEVELS.include? severity
 
         # Dynamically check enablement of underlying logger, this expands to "is_<level>_enabled"
-        return unless @sl4fj_logger.send("is_#{severity.to_s.downcase}_enabled")
+        return unless send("#{severity}_enabled?")
 
         # Process block if no message provided
         msg = yield if msg.nil? && block_given?
@@ -62,7 +88,7 @@ module OpenHAB
         msg = message_to_string(msg: msg)
 
         # Dynamically invoke underlying logger, this expands to "<level>(message)"
-        @sl4fj_logger.send(severity.to_s.downcase, msg)
+        @sl4fj_logger.send(severity, msg)
       end
 
       #
@@ -121,7 +147,7 @@ module OpenHAB
         configure_logger_for(classname)
       end
 
-     private
+      private
 
       #
       # Configure a logger for the supplied classname
@@ -153,7 +179,7 @@ module OpenHAB
                         .first
                         .yield_self { |caller| File.basename(caller, '.*') }
       end
-   end
+    end
 
     #
     # Add logger method to the object that includes this module

data/lib/openhab/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: openhab-scripting
 version: !ruby/object:Gem::Version
-  version: 2.25.0
+  version: 2.27.0
 platform: ruby
 authors:
 - Brian O'Connell
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-02-24 00:00:00.000000000 Z
+date: 2021-03-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -44,6 +44,7 @@ files:
 - lib/openhab/dsl/items/item_delegate.rb
 - lib/openhab/dsl/items/items.rb
 - lib/openhab/dsl/items/number_item.rb
+- lib/openhab/dsl/items/player_item.rb
 - lib/openhab/dsl/items/rollershutter_item.rb
 - lib/openhab/dsl/items/string_item.rb
 - lib/openhab/dsl/monkey_patch/actions/actions.rb