From cc05b9c87978d9068c2976419a03899b5d87d6b0 Mon Sep 17 00:00:00 2001
From: Mark Herwege <mark.herwege@telenet.be>
Date: Sun, 11 Jun 2023 16:02:00 +0200
Subject: [PATCH] review feedback

Signed-off-by: Mark Herwege <mark.herwege@telenet.be>
---
 .../core/ui/internal/items/ItemUIRegistryImpl.java   | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/bundles/org.openhab.core.ui/src/main/java/org/openhab/core/ui/internal/items/ItemUIRegistryImpl.java b/bundles/org.openhab.core.ui/src/main/java/org/openhab/core/ui/internal/items/ItemUIRegistryImpl.java
index 382f6d681a8..1fb84d10442 100644
--- a/bundles/org.openhab.core.ui/src/main/java/org/openhab/core/ui/internal/items/ItemUIRegistryImpl.java
+++ b/bundles/org.openhab.core.ui/src/main/java/org/openhab/core/ui/internal/items/ItemUIRegistryImpl.java
@@ -371,7 +371,7 @@ private Switch createPlayerButtons() {
                 }
             }
         } catch (ItemNotFoundException e) {
-            logger.error("Cannot retrieve item '{}' for widget {}", itemName, w.eClass().getInstanceTypeName());
+            logger.warn("Cannot retrieve item '{}' for widget {}", itemName, w.eClass().getInstanceTypeName());
         }
 
         boolean considerTransform = false;
@@ -457,7 +457,7 @@ private Switch createPlayerButtons() {
                 label = label.trim();
                 int index = label.indexOf("[");
                 if (index >= 0) {
-                    label = label.substring(0, label.indexOf("[") + 1) + formatPattern + "]";
+                    label = label.substring(0, index + 1) + formatPattern + "]";
                 }
             }
         }
@@ -519,7 +519,7 @@ private QuantityType<?> convertStateToWidgetUnit(QuantityType<?> quantityState,
             }
 
         } catch (ItemNotFoundException e) {
-            logger.error("Cannot retrieve item '{}' for widget {}", itemName, w.eClass().getInstanceTypeName());
+            logger.warn("Cannot retrieve item '{}' for widget {}", itemName, w.eClass().getInstanceTypeName());
         }
 
         return pattern;
@@ -662,7 +662,7 @@ private String transform(String label, boolean matchTransform, @Nullable String
                 Item item = getItem(itemName);
                 return convertState(w, item, item.getState());
             } catch (ItemNotFoundException e) {
-                logger.error("Cannot retrieve item '{}' for widget {}", itemName, w.eClass().getInstanceTypeName());
+                logger.warn("Cannot retrieve item '{}' for widget {}", itemName, w.eClass().getInstanceTypeName());
             }
         }
         return UnDefType.UNDEF;
@@ -1262,7 +1262,7 @@ public boolean getVisiblity(Widget w) {
             try {
                 item = itemRegistry.getItem(itemName);
             } catch (ItemNotFoundException e) {
-                logger.error("Cannot retrieve visibility item {} for widget {}", rule.getItem(),
+                logger.warn("Cannot retrieve visibility item {} for widget {}", rule.getItem(),
                         w.eClass().getInstanceTypeName());
 
                 // Default to visible!
@@ -1393,7 +1393,7 @@ public void removeRegistryHook(RegistryHook<Item> hook) {
                 return numberItem.getUnitSymbol();
             }
         } catch (ItemNotFoundException e) {
-            logger.debug("Failed to retrieve item during widget rendering: {}", e.getMessage());
+            logger.warn("Failed to retrieve item during widget rendering, item does not exist: {}", e.getMessage());
         }
 
         return "";