Merge pull request openhab#3950 from watou/nest-fix-3944

[nest] Allow setting setpoints with decimals

bundles/binding/org.openhab.binding.nest/src/main/java/org/openhab/binding/nest/internal/messages/DataModel.java

@@ -10,6 +10,7 @@
 
 import java.io.UnsupportedEncodingException;
 import java.lang.reflect.InvocationTargetException;
+import java.math.BigDecimal;
 import java.net.URLDecoder;
 import java.util.Date;
 import java.util.HashMap;
@@ -120,6 +121,17 @@ public Object convert(Class type, Object value) {
                 }
             }
         }, ColorState.class);
+        convertUtils.register(new Converter() {
+            @SuppressWarnings("rawtypes")
+            @Override
+            public Object convert(Class type, Object value) {
+                if (value instanceof DecimalType) {
+                    return ((DecimalType) value).toBigDecimal();
+                } else {
+                    return null;
+                }
+            }
+        }, BigDecimal.class);
         convertUtils.register(new Converter() {
             @SuppressWarnings("rawtypes")
             @Override
@@ -195,7 +207,7 @@ public void setThermostats_by_id(Map<String, Thermostat> thermostats_by_id) {
 
         /**
          * Return the thermostats map, mapped by name.
-         * 
+         *
          * @return the thermostats_by_name;
          */
         @JsonIgnore
@@ -324,7 +336,7 @@ public String getKey(String expression) {
 
     /**
      * Return a JavaBean property by name.
-     * 
+     *
      * @param name
      *            the named property to return
      * @return the named property's value
@@ -345,7 +357,7 @@ public Object getProperty(String name)
     /**
      * Set the specified property value, performing type conversions as required to conform to the type of the
      * destination property.
-     * 
+     *
      * @param name
      *            property name (can be nested/indexed/mapped/combo)
      * @param value
@@ -379,7 +391,7 @@ public void setDevices(Devices devices) {
 
     /**
      * Convenience method so property specs don't have to include "devices." in each one.
-     * 
+     *
      * @return name-based map of thermostats
      */
     @JsonIgnore
@@ -389,7 +401,7 @@ public Map<String, Thermostat> getThermostats() {
 
     /**
      * Convenience method so property specs don't have to include "devices." in each one.
-     * 
+     *
      * @return name-based map of smoke_co_alarms
      */
     @JsonIgnore
@@ -399,7 +411,7 @@ public Map<String, SmokeCOAlarm> getSmoke_co_alarms() {
 
     /**
      * Convenience method so property specs don't have to include "devices." in each one.
-     * 
+     *
      * @return name-based map of cameras
      */
     @JsonIgnore
@@ -475,7 +487,7 @@ public void sync() {
      * This method returns a new data model containing only the affected Structure, Thermostat, SmokeCOAlarm or Camera,
      * and only the property of the bean that was changed. This new DataModel object can be sent to the Nest API in
      * order to perform an update via HTTP PUT.
-     * 
+     *
      * @param property
      *            the property to change
      * @param newState
@@ -562,7 +574,7 @@ public DataModel updateDataModel(String property, Object newState)
 
         /**
          * Lastly, set the property into the update data model
-         * 
+         *
          * TODO: cannot update a binding string of the form "=[structures(Name).thermostats(Name).X]" or
          * "=[structures(Name).smoke_co_alarms(Name).X]" because the name-based map of structures is not present in the
          * updateDataModel