From e40c874735b14cb444302337c6b561f3b27a22c2 Mon Sep 17 00:00:00 2001
From: Florian Hotze <florianh_dev@icloud.com>
Date: Mon, 30 Sep 2024 22:22:43 +0200
Subject: [PATCH] Item metadata: Improve dirty handling & Fix state description
 metadata always dirty

Signed-off-by: Florian Hotze <florianh_dev@icloud.com>
---
 .../item/metadata/item-metadata-widget.vue      |  2 +-
 .../items/metadata/item-metadata-edit.vue       | 17 ++++++++++-------
 2 files changed, 11 insertions(+), 8 deletions(-)

diff --git a/bundles/org.openhab.ui/web/src/components/item/metadata/item-metadata-widget.vue b/bundles/org.openhab.ui/web/src/components/item/metadata/item-metadata-widget.vue
index c591d25eb8..d516d01726 100644
--- a/bundles/org.openhab.ui/web/src/components/item/metadata/item-metadata-widget.vue
+++ b/bundles/org.openhab.ui/web/src/components/item/metadata/item-metadata-widget.vue
@@ -203,7 +203,7 @@ export default {
     },
     updateComponent () {
       const value = this.$refs.widgets.f7SmartSelect.getValue()
-      this.metadata.value = value
+      this.metadata.value = value || ' ' // ' ' is used to indicate the default widget
       this.setCurrentComponent()
       this.setConfigDescriptions()
       this.setPreviewContext()
diff --git a/bundles/org.openhab.ui/web/src/pages/settings/items/metadata/item-metadata-edit.vue b/bundles/org.openhab.ui/web/src/pages/settings/items/metadata/item-metadata-edit.vue
index 9eeec79a13..237a8dcf17 100644
--- a/bundles/org.openhab.ui/web/src/pages/settings/items/metadata/item-metadata-edit.vue
+++ b/bundles/org.openhab.ui/web/src/pages/settings/items/metadata/item-metadata-edit.vue
@@ -62,6 +62,8 @@
 
 <script>
 import YAML from 'yaml'
+import fastDeepEqual from 'fast-deep-equal/es6'
+import cloneDeep from 'lodash/cloneDeep'
 
 import MetadataNamespaces from '@/assets/definitions/metadata/namespaces.js'
 
@@ -93,6 +95,7 @@ export default {
       generic: false,
       item: {},
       metadata: { value: '', config: {} },
+      savedMetadata: {},
       yaml: null
     }
   },
@@ -100,7 +103,7 @@ export default {
     metadata: {
       handler: function () {
         if (this.ready) {
-          this.dirty = true
+          this.dirty = !fastDeepEqual(this.metadata, this.savedMetadata)
         }
       },
       deep: true
@@ -153,13 +156,12 @@ export default {
   methods: {
     onPageBeforeIn () {
       this.generic = MetadataNamespaces.map((n) => n.name).indexOf(this.namespace) < 0
-      this.ready = false
     },
     onPageAfterIn () {
-      this.$oh.api.get(`/rest/items/${this.itemName}?metadata=${this.namespace}`).then((data) => {
-        this.item = data
-        if (this.item.metadata) {
-          this.metadata = this.item.metadata[this.namespace]
+      this.$oh.api.get(`/rest/items/${this.itemName}?metadata=${this.namespace}`).then((item) => {
+        this.item = item
+        if (item.metadata) {
+          this.metadata = item.metadata[this.namespace]
           if (!this.metadata.config) this.$set(this.metadata, 'config', {})
           this.creationMode = false
         }
@@ -167,6 +169,7 @@ export default {
           this.currentTab = 'code'
           this.toYaml()
         }
+        this.savedMetadata = cloneDeep(this.metadata)
         this.$nextTick(() => {
           this.ready = true
         })
@@ -174,7 +177,6 @@ export default {
     },
     onEditorInput (value) {
       this.yaml = value
-      this.dirty = true
     },
     save () {
       if (this.currentTab === 'code' && !this.fromYaml()) return
@@ -193,6 +195,7 @@ export default {
             closeTimeout: 2000
           }).open()
         }
+        this.savedMetadata = cloneDeep(this.metadata)
         this.dirty = false
         this.$f7router.back()
       }).catch((err) => {