openhab-scripting 5.46.0 → 5.46.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e6351743424c867636542a49ecdb2ef2eca63824d2009ae517763d256f7fab20
-  data.tar.gz: 72d778a25df32f555763c38539c1bb7d34ae2f24c70120d069504d18e036eaac
+  metadata.gz: 185967fd947cf9a2dafa36a14ed1847c46b6c4bf579855d1c26c2c7813164fa1
+  data.tar.gz: 161f3f62083751c1dda257078aac9e229cf9666b47dde175499714a7721b71d0
 SHA512:
-  metadata.gz: a6d35c4a0489003faeabf001e3215bb8f4e9e805cdffb69ba0b10d9261127f08f07f1c0e61cc072ff849eab6dc70c88d9b11003c9f830c53e93ff92ce5ae03ce
-  data.tar.gz: 0be0d9f6149485518eabb0b4c08bb4a73ce7fdf7c9d231e9c9ee590f132a74f647dcbb4c3e3d1087dd2be10a1f487736e45214af2037e2093f019de3d3aeadda
+  metadata.gz: f754fbb6eb44e1a9e776e80af7dfbc317d06cfa8eb1eb79f1246e022bc027bfa7f163a210ef06c7c1e7920a46e3d6959a8bfee17c25c10c8bfbd9e40eb7318ea
+  data.tar.gz: a9d9095df6de53da7b4b69542b7c49807115a40692242c9abddb91b6a905e697c8c1ba765dd19d275e3a1ee6f744ee3f94407a8f79feb9ae9e9110a82b922781

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

@@ -54,13 +54,13 @@ module OpenHAB
           # @!visibility private
           def <(other)
             check_type(other)
-            uid != other.uid && uid.start_with?(other.uid)
+            uid.start_with?("#{other.uid}_")
           end
 
           # @!visibility private
           def <=(other)
             check_type(other)
-            uid.start_with?(other.uid)
+            uid == other.uid || uid.start_with?("#{other.uid}_")
           end
 
           # @!visibility private
@@ -72,13 +72,13 @@ module OpenHAB
           # @!visibility private
           def >=(other)
             check_type(other)
-            other.uid.start_with?(uid)
+            other <= self
           end
 
           # @!visibility private
           def >(other)
             check_type(other)
-            uid != other.uid && other.uid.start_with?(uid)
+            other < self
           end
 
           # @return [String]

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

@@ -381,7 +381,7 @@ module OpenHAB
         #   Property = SemanticTag
 
         # put ourself into the global namespace, replacing the action
-        Object.send(:remove_const, :Semantics)
+        Object.send(:remove_const, :Semantics) if Object.const_defined?(:Semantics)
         ::Semantics = self # rubocop:disable Naming/ConstantName
 
         #

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

@@ -132,18 +132,20 @@ module OpenHAB
 
           # @!visibility private
           def add(builder)
-            if DSL.items.key?(builder.name)
+            if (old_item = DSL.items[builder.name])
               raise ArgumentError, "Item #{builder.name} already exists" unless @update
 
               # Use provider.get because openHAB's ManagedProvider does not support the #[] method.
-              unless (old_item = provider.get(builder.name))
-                raise FrozenError, "Item #{builder.name} is managed by #{DSL.items[builder.name].provider}"
+              # Note that ManagedProvider stores PersistableItem objects, not the actual item,
+              # so we cannot use provider.get as old_item. We'll get it from the Item Registry instead.
+              unless provider.get(builder.name)
+                raise FrozenError, "Item #{builder.name} is managed by #{old_item.provider}"
               end
 
               item = builder.build
-              if item.config_eql?(old_item)
+              if item.config_eql?(old_item.__getobj__)
                 logger.debug { "Not replacing existing item #{item.uid} because it is identical" }
-                item = old_item
+                item = old_item.__getobj__
               else
                 logger.debug { "Replacing existing item #{item.uid} because it is not identical" }
                 provider.update(item)
@@ -169,6 +171,7 @@ module OpenHAB
               if !channel.include?(":") &&
                  (thing = builder.thing ||
                   thing = builder.groups.find { |g| g.is_a?(GroupItemBuilder) && g.thing }&.thing)
+                thing = thing.uid if thing.respond_to?(:uid)
                 channel = "#{thing}:#{channel}"
               end
 
@@ -248,7 +251,7 @@ module OpenHAB
         # Autoupdate setting
         # @return [true, false, nil]
         attr_accessor :autoupdate
-        # @return [String, Core::Things::Thing, Core::Things::ThingUID, nil]
+        # @return [String, Core::Things::Thing, Core::Things::ThingUID, DSL::Things::Builder::ThingBuilder, nil]
         #   {Core::Things::ThingUID Thing} from which to resolve relative channel ids
         attr_accessor :thing
         # @return [Core::Items::Metadata::NamespaceHash]
@@ -302,7 +305,7 @@ module OpenHAB
         # @param tags [String, Symbol, Semantics::Tag, Array<String, Symbol, Semantics::Tag>, nil]
         #        Fluent alias for `tag`.
         # @param autoupdate [true, false, nil] Autoupdate setting (see {ItemBuilder#autoupdate})
-        # @param thing [String, Core::Things::Thing, Core::Things::ThingUID, nil]
+        # @param thing [String, Core::Things::Thing, Core::Things::ThingUID, DSL::Things::Builder::ThingBuilder, nil]
         #   A Thing to be used as the base for the channel.
         # @param channel [String, Symbol, Core::Things::ChannelUID, Core::Things::Channel, nil]
         #   Channel to link the item to (see {ItemBuilder#channel}).
@@ -786,7 +789,7 @@ module OpenHAB
         # @return [String, nil]
         attr_accessor :function
         # A thing to be used as the base for the channel of any member items
-        # @return [Core::Things::ThingUID, Core::Things::Thing, String, nil]
+        # @return [Core::Things::ThingUID, Core::Things::Thing, DSL::Things::Builder::ThingBuilder, String, nil]
         attr_accessor :thing
         # A prefix to be added to the name of any member items
         # @return [String, nil]
@@ -800,7 +803,7 @@ module OpenHAB
 
         # @param type [Symbol, nil] The base type for the group
         # @param function [String, nil] The combiner function for this group
-        # @param thing [Core::Things::ThingUID, Core::Things::Thing, String, nil]
+        # @param thing [Core::Things::ThingUID, Core::Things::Thing, DSL::Things::Builder::ThingBuilder, String, nil]
         #        A Thing to be used as the base for the channel for any contained items.
         # @param (see ItemBuilder#initialize)
         def initialize(*, type: nil, function: nil, thing: nil, **)

data/lib/openhab/dsl/things/builder.rb

@@ -95,16 +95,13 @@ module OpenHAB
           builder.instance_eval(&block) if block
           thing = builder.build
 
-          if DSL.things.key?(thing.uid)
+          if (old_thing = DSL.things[thing.uid])
             raise ArgumentError, "Thing #{thing.uid} already exists" unless @update
+            raise FrozenError, "Thing #{thing.uid} is managed by #{thing.provider}" unless provider.get(thing.uid)
 
-            unless (old_thing = provider.get(thing.uid))
-              raise FrozenError, "Thing #{thing.uid} is managed by #{thing.provider}"
-            end
-
-            if thing.config_eql?(old_thing)
+            if thing.config_eql?(old_thing.__getobj__)
               logger.debug { "Not replacing existing thing #{thing.uid}" }
-              thing = old_thing
+              thing = old_thing.__getobj__
             else
               provider.update(thing)
             end

data/lib/openhab/dsl/version.rb

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

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: openhab-scripting
 version: !ruby/object:Gem::Version
-  version: 5.46.0
+  version: 5.46.1
 platform: ruby
 authors:
 - Brian O'Connell
@@ -354,7 +354,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.3
+rubygems_version: 4.0.4
 specification_version: 4
 summary: JRuby Helper Libraries for openHAB Scripting
 test_files: []