Ok, much better

in fact, switchrule is meanfull only for autoupdate ... if autoupdate is off, it is module's responsability to ensure switch behaviour i think it's logic ...
gehelem · Dec 6, 2024 · 1b65271 · 1b65271
1 parent 24c2e6b
commit 1b65271
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/libs/basemodule.cpp b/libs/basemodule.cpp
@@ -295,7 +295,9 @@ void Basemodule::OnExternalEvent(const QString &pEventType, const QString  &pEve
                     }
                     if (getEltBase(keyprop, keyelt)->getType() == "bool")
                     {
-                        getEltBool(keyprop, keyelt)->setValue(v.toBool(), true);
+                        //a boolean need to be updated on property level because of SwitchRule's flag behaviour
+                        getProperty(keyprop)->setElt(keyelt, v);
+
                     }
                     if (getEltBase(keyprop, keyelt)->getType() == "date")
                     {

diff --git a/libs/model/property/propertymulti.cpp b/libs/model/property/propertymulti.cpp
@@ -174,7 +174,7 @@ bool  PropertyMulti::setElt(QString key, QVariant val)
                     return true;
 
                 }
-                sendError("PropertyMulti::setValue - " + key + " - OneOfMany - can't just unset");
+                sendWarning("PropertyMulti::setValue - " + key + " - OneOfMany - can't just unset");
                 return false;
                 break;
             case AtMostOne:

diff --git a/src/modules/dummy/dummy.cpp b/src/modules/dummy/dummy.cpp
@@ -141,7 +141,7 @@ void Dummy::OnMyExternalEvent(const QString &eventType, const QString  &eventMod
                         // test max gridlimit
                         getProperty("secondtestgrid")->clearGrid();
                         getProperty("secondtestgrid")->setGridLimit(1010);
-                        for ( int i = 0; i < 1010; i++)getProperty("secondtestgrid")->push();
+                        //for ( int i = 0; i < 1010; i++)getProperty("secondtestgrid")->push();
                     }
                 }
                 if (keyprop == "devices")