Merge pull request #4104 from FreakyJ/Fix-Bug-#3809

softhack007 · web-flow · commit 198aceee0757 · 2024-08-16T02:04:00.000+02:00
Fixes #3809 Loxone JSON parser doesn't handle lx=0 correctly
diff --git a/wled00/json.cpp b/wled00/json.cpp
@@ -197,11 +197,11 @@ bool deserializeSegment(JsonObject elem, byte it, byte presetId)
   // lx parser
   #ifdef WLED_ENABLE_LOXONE
   int lx = elem[F("lx")] | -1;
-  if (lx > 0) {
+  if (lx >= 0) {
     parseLxJson(lx, id, false);
   }
   int ly = elem[F("ly")] | -1;
-  if (ly > 0) {
+  if (ly >= 0) {
     parseLxJson(ly, id, true);
   }
   #endif

Original file line number	Diff line number	Diff line change
`@@ -197,11 +197,11 @@ bool deserializeSegment(JsonObject elem, byte it, byte presetId)`
`197`	`197`	`// lx parser`
`198`	`198`	`#ifdef WLED_ENABLE_LOXONE`
`199`	`199`	`int lx = elem[F("lx")] \| -1;`
`200`		`- if (lx > 0) {`
	`200`	`+ if (lx >= 0) {`
`201`	`201`	`parseLxJson(lx, id, false);`
`202`	`202`	`}`
`203`	`203`	`int ly = elem[F("ly")] \| -1;`
`204`		`- if (ly > 0) {`
	`204`	`+ if (ly >= 0) {`
`205`	`205`	`parseLxJson(ly, id, true);`
`206`	`206`	`}`
`207`	`207`	`#endif`