Optional label in OpenHABOptions (#850)

* OpenHABOptions label as optional string Signed-off-by: Tim Bert <5411131+timbms@users.noreply.github.com>
openhab · Oct 31, 2024 · dd5553e · dd5553e
1 parent dfc0292
commit dd5553e
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/OpenHABCore/Sources/OpenHABCore/Model/OpenHABOptions.swift b/OpenHABCore/Sources/OpenHABCore/Model/OpenHABOptions.swift
@@ -13,5 +13,5 @@ import Foundation
 
 public class OpenHABOptions: Decodable {
     public var value = ""
-    public var label = ""
+    public var label: String?
 }
diff --git a/OpenHABCore/Sources/OpenHABCore/Model/OpenHABWidget.swift b/OpenHABCore/Sources/OpenHABCore/Model/OpenHABWidget.swift
@@ -119,7 +119,7 @@ public class OpenHABWidget: NSObject, MKAnnotation, Identifiable {
         if mappings.isEmpty, let commandOptions = item?.commandDescription?.commandOptions {
             commandOptions.map { OpenHABWidgetMapping(command: $0.command, label: $0.label ?? "") }
         } else if mappings.isEmpty, let stateOptions = item?.stateDescription?.options {
-            stateOptions.map { OpenHABWidgetMapping(command: $0.value, label: $0.label) }
+            stateOptions.map { OpenHABWidgetMapping(command: $0.value, label: $0.label ?? "") }
         } else {
             mappings
         }

diff --git a/openHABWatch/Model/ObservableOpenHABWidget.swift b/openHABWatch/Model/ObservableOpenHABWidget.swift
@@ -97,7 +97,7 @@ class ObservableOpenHABWidget: NSObject, MKAnnotation, Identifiable, ObservableO
         if mappings.isEmpty, let commandOptions = item?.commandDescription?.commandOptions {
             commandOptions.map { OpenHABWidgetMapping(command: $0.command, label: $0.label ?? "") }
         } else if mappings.isEmpty, let stateOptions = item?.stateDescription?.options {
-            stateOptions.map { OpenHABWidgetMapping(command: $0.value, label: $0.label) }
+            stateOptions.map { OpenHABWidgetMapping(command: $0.value, label: $0.label ?? "") }
         } else {
             mappings
         }