Merge pull request #49 from ShellyUSA/DanielWinks

DanielWinks · web-flow · commit a733cc2e43c8 · 2025-01-24T20:57:42.000-05:00
Make sure cover is only set to 0-100 and not below or above
diff --git a/ShellyDriverLibrary/ShellyUSA.ShellyUSA_Driver_Library.groovy b/ShellyDriverLibrary/ShellyUSA.ShellyUSA_Driver_Library.groovy
@@ -1375,7 +1375,7 @@ void setCoverState(String value, Integer id = 0) {
   }
 }
 
-void setChildCoverPosition(Integer position, ChildDeviceWrapper child) {child?.setCoverPosition(position)}
+void setChildCoverPosition(Integer position, ChildDeviceWrapper child) {child?.setCoverPosition(boundedLevel(position))}
 
 @CompileStatic
 void setCoverPosition(Integer position, Integer id = 0) {
@@ -2261,7 +2261,7 @@ void processGen2JsonMessageBody(LinkedHashMap<String, Object> json, Integer id =
       id = update?.id as Integer
       if(update?.current_pos != null) {
         Integer current_pos = update?.current_pos as Integer
-        if(current_pos != null) { setCoverPosition(current_pos, id) }
+        if(current_pos != null) { setCoverPosition(boundedLevel(current_pos), id) }
       }
       if(update?.state != null) {
         String coverState = update?.state as String

Original file line number	Diff line number	Diff line change
`@@ -1375,7 +1375,7 @@ void setCoverState(String value, Integer id = 0) {`
`1375`	`1375`	`}`
`1376`	`1376`	`}`
`1377`	`1377`
`1378`		`-void setChildCoverPosition(Integer position, ChildDeviceWrapper child) {child?.setCoverPosition(position)}`
	`1378`	`+void setChildCoverPosition(Integer position, ChildDeviceWrapper child) {child?.setCoverPosition(boundedLevel(position))}`
`1379`	`1379`
`1380`	`1380`	`@CompileStatic`
`1381`	`1381`	`void setCoverPosition(Integer position, Integer id = 0) {`
`@@ -2261,7 +2261,7 @@ void processGen2JsonMessageBody(LinkedHashMap<String, Object> json, Integer id =`
`2261`	`2261`	`id = update?.id as Integer`
`2262`	`2262`	`if(update?.current_pos != null) {`
`2263`	`2263`	`Integer current_pos = update?.current_pos as Integer`
`2264`		`- if(current_pos != null) { setCoverPosition(current_pos, id) }`
	`2264`	`+ if(current_pos != null) { setCoverPosition(boundedLevel(current_pos), id) }`
`2265`	`2265`	`}`
`2266`	`2266`	`if(update?.state != null) {`
`2267`	`2267`	`String coverState = update?.state as String`