Persistence editor: Remove save checks as not required anymore

openhab/openhab-core#3727 made core accept null as values for `relative` and `inverted` fields of persistence filters and set defaults for those. Therefore, UI does not need to ensure they are set anymore. Signed-off-by: Florian Hotze <[email protected]>
florian-h05 · Jul 25, 2023 · 4f29597 · 4f29597
1 parent 1a0ea55
commit 4f29597
Showing 1 changed file with 1 addition and 12 deletions.
diff --git a/bundles/org.openhab.ui/web/src/pages/settings/persistence/persistence-edit.vue b/bundles/org.openhab.ui/web/src/pages/settings/persistence/persistence-edit.vue
@@ -261,7 +261,7 @@ export default {
 
       predefinedStrategies: ['everyChange', 'everyUpdate', 'restoreOnStartup'],
       // Filter configuration is completely based on these definitions, when adding new filters, no code needs to be updated.
-      // However, please note that some validation and checks are in place for some filter types in save(), editFilter(), saveFilter() and filter-popup.vue
+      // However, please note that some validation and checks are in place for some filter types in editFilter(), saveFilter() and filter-popup.vue
       filterTypes: [
         {
           name: 'thresholdFilters',
@@ -470,17 +470,6 @@ export default {
       if (!this.isEditable) return
       if (this.currentTab === 'code') this.fromYaml()
 
-      // Ensure relative is set on threshold filter, otherwise the save request fails with a 500
-      this.persistence.thresholdFilters.forEach((f) => {
-        if (f.relative === undefined) f.relative = false
-      })
-      // Ensure inverted is set for equals and include filter, otherwise the save request fails with a 500
-      this.persistence.equalsFilters.forEach((f) => {
-        if (f.inverted === undefined) f.inverted = false
-      })
-      this.persistence.includeFilters.forEach((f) => {
-        if (f.inverted === undefined) f.inverted = false
-      })
       // Update the code tab
       if (this.persistenceYaml) this.toYaml()