Provide a more meaningful warning if state transformation in item fai…

…ls (#5900) Because the TransformationHelper throws a TransformationException to the caller since PR #5679, it is now possible to provide a meaningful warning where the transformation is started, i.e. provide the item name. Fixes #5120 Signed-off-by: Stefan Triller <[email protected]>
eclipse-archived · Jul 12, 2018 · d712459 · d712459
1 parent 7bb5ac0
commit d712459
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/...est.core/src/main/java/org/eclipse/smarthome/io/rest/core/item/EnrichedItemDTOMapper.java b/...est.core/src/main/java/org/eclipse/smarthome/io/rest/core/item/EnrichedItemDTOMapper.java
@@ -59,7 +59,7 @@ public static EnrichedItemDTO map(Item item, boolean drillDown, Predicate<Item>
     private static EnrichedItemDTO map(Item item, ItemDTO itemDTO, URI uri, boolean drillDown,
             Predicate<Item> itemFilter, Locale locale) {
         String state = item.getState().toFullString();
-        String transformedState = considerTransformation(state, item.getStateDescription(locale));
+        String transformedState = considerTransformation(state, item, locale);
         if (transformedState != null && transformedState.equals(state)) {
             transformedState = null;
         }
@@ -106,7 +106,8 @@ private static StateDescription considerTransformation(StateDescription desc) {
         }
     }
 
-    private static String considerTransformation(String state, StateDescription stateDescription) {
+    private static String considerTransformation(String state, Item item, Locale locale) {
+        StateDescription stateDescription = item.getStateDescription(locale);
         if (stateDescription != null && stateDescription.getPattern() != null && state != null) {
             try {
                 return TransformationHelper.transform(RESTCoreActivator.getBundleContext(),
@@ -116,8 +117,8 @@ private static String considerTransformation(String state, StateDescription stat
                 // return state as it is without transformation
                 return state;
             } catch (TransformationException e) {
-                LOGGER.debug("Failed transforming the state '{}' with pattern '{}': {}", state,
-                        stateDescription.getPattern(), e.getMessage());
+                LOGGER.warn("Failed transforming the state '{}' on item '{}' with pattern '{}': {}", state,
+                        item.getName(), stateDescription.getPattern(), e.getMessage());
                 return state;
             }
         } else {