Fix thing reloading from things file (openhab#3526)

It was found that things from textual configuration are not properly updated if changes are only made in configuration or label of a channel. The reason is that for equality only uid and accepted item-type where checked. Signed-off-by: Jan N. Klug <github@klug.nrw> GitOrigin-RevId: 016828c
ConnectorIO · Jul 12, 2023 · a75116e · a75116e
1 parent 09f1043
commit a75116e
Show file tree

Hide file tree

Showing 3 changed files with 45 additions and 19 deletions.
diff --git a/bundles/org.opensmarthouse.core.thing/src/main/java/org/openhab/core/thing/Channel.java b/bundles/org.opensmarthouse.core.thing/src/main/java/org/openhab/core/thing/Channel.java
@@ -14,6 +14,7 @@
 
 import java.util.LinkedHashSet;
 import java.util.Map;
+import java.util.Objects;
 import java.util.Set;
 
 import org.eclipse.jdt.annotation.NonNullByDefault;
@@ -178,4 +179,27 @@ public Set<String> getDefaultTags() {
     public @Nullable AutoUpdatePolicy getAutoUpdatePolicy() {
         return autoUpdatePolicy;
     }
+
+    @Override
+    public boolean equals(@Nullable Object o) {
+        if (this == o) {
+            return true;
+        }
+        if (o == null || getClass() != o.getClass()) {
+            return false;
+        }
+        Channel channel = (Channel) o;
+        return Objects.equals(acceptedItemType, channel.acceptedItemType) && kind == channel.kind
+                && Objects.equals(uid, channel.uid) && Objects.equals(channelTypeUID, channel.channelTypeUID)
+                && Objects.equals(label, channel.label) && Objects.equals(description, channel.description)
+                && Objects.equals(configuration, channel.configuration)
+                && Objects.equals(properties, channel.properties) && Objects.equals(defaultTags, channel.defaultTags)
+                && autoUpdatePolicy == channel.autoUpdatePolicy;
+    }
+
+    @Override
+    public int hashCode() {
+        return Objects.hash(acceptedItemType, kind, uid, channelTypeUID, label, description, configuration, properties,
+                defaultTags, autoUpdatePolicy);
+    }
 }
diff --git a/.../org.opensmarthouse.core.thing/src/main/java/org/openhab/core/thing/util/ThingHelper.java b/.../org.opensmarthouse.core.thing/src/main/java/org/openhab/core/thing/util/ThingHelper.java
@@ -12,10 +12,8 @@
  */
 package org.openhab.core.thing.util;
 
-import java.util.ArrayList;
 import java.util.Arrays;
 import java.util.Collection;
-import java.util.Collections;
 import java.util.HashSet;
 import java.util.Iterator;
 import java.util.List;
@@ -79,24 +77,10 @@ public static boolean equals(Thing a, Thing b) {
             return false;
         }
         // channels
-        List<Channel> channelsOfA = a.getChannels();
-        List<Channel> channelsOfB = b.getChannels();
-        if (channelsOfA.size() != channelsOfB.size()) {
-            return false;
-        }
-        if (!toString(channelsOfA).equals(toString(channelsOfB))) {
-            return false;
-        }
-        return true;
-    }
+        Set<Channel> channelsOfA = new HashSet<>(a.getChannels());
+        Set<Channel> channelsOfB = new HashSet<>(b.getChannels());
 
-    private static String toString(List<Channel> channels) {
-        List<String> strings = new ArrayList<>(channels.size());
-        for (Channel channel : channels) {
-            strings.add(channel.getUID().toString() + '#' + channel.getAcceptedItemType() + '#' + channel.getKind());
-        }
-        Collections.sort(strings);
-        return String.join(",", strings);
+        return channelsOfA.equals(channelsOfB);
     }
 
     public static void addChannelsToThing(Thing thing, Collection<Channel> channels) {

diff --git a/....opensmarthouse.core.thing/src/test/java/org/openhab/core/thing/util/ThingHelperTest.java b/....opensmarthouse.core.thing/src/test/java/org/openhab/core/thing/util/ThingHelperTest.java
@@ -16,6 +16,7 @@
 import static org.junit.jupiter.api.Assertions.*;
 
 import java.util.List;
+import java.util.Map;
 import java.util.stream.Stream;
 
 import org.eclipse.jdt.annotation.NonNullByDefault;
@@ -142,4 +143,21 @@ public void assertThatNoDuplicateChannelsCanBeAdded() {
                                 ChannelBuilder.create(new ChannelUID(thingUID, "channel3"), "").build())
                                 .collect(toList())));
     }
+
+    @Test
+    public void asserThatChannelsWithDifferentConfigurationAreDetectedAsDifferent() {
+        Thing thingA = ThingBuilder.create(THING_TYPE_UID, THING_UID)
+                .withChannels(ChannelBuilder.create(new ChannelUID("binding:type:thingId:channel1"), "itemType")
+                        .withConfiguration(new Configuration(Map.of("key", "v1"))).build())
+                .withConfiguration(new Configuration()).build();
+
+        assertTrue(ThingHelper.equals(thingA, thingA));
+
+        Thing thingB = ThingBuilder.create(THING_TYPE_UID, THING_UID)
+                .withChannels(ChannelBuilder.create(new ChannelUID("binding:type:thingId:channel1"), "itemType")
+                        .withConfiguration(new Configuration(Map.of("key", "v2"))).build())
+                .withConfiguration(new Configuration()).build();
+
+        assertFalse(ThingHelper.equals(thingA, thingB));
+    }
 }