[ModelicaSystem] simplify _set_method_helper()

Author: syntron

OMPython/ModelicaSystem.py

@@ -1200,24 +1200,24 @@ def _set_method_helper(
 
         inputdata_status: dict[str, bool] = {}
         for key, val in inputdata.items():
-            status = False
-            if key in classdata:
-                if datatype == "parameter" and not self.isParameterChangeable(key):
-                    logger.debug(f"It is not possible to set the parameter {repr(key)}. It seems to be "
-                                 "structural, final, protected, evaluated or has a non-constant binding. "
-                                 "Use sendExpression(...) and rebuild the model using buildModel() API; example: "
-                                 "sendExpression(\"setParameterValue("
-                                 f"{self.modelName}, {key}, {val if val is not None else '<?value?>'}"
-                                 ")\") ")
-                else:
-                    classdata[key] = val
-                    if overwritedata is not None:
-                        overwritedata[key] = val
-                    status = True
-            else:
+            if key not in classdata:
                 raise ModelicaSystemError("Unhandled case in setMethodHelper.apply_single() - "
                                           f"{repr(key)} is not a {repr(datatype)} variable")
 
+            status = False
+            if datatype == "parameter" and not self.isParameterChangeable(key):
+                logger.debug(f"It is not possible to set the parameter {repr(key)}. It seems to be "
+                             "structural, final, protected, evaluated or has a non-constant binding. "
+                             "Use sendExpression(...) and rebuild the model using buildModel() API; example: "
+                             "sendExpression(\"setParameterValue("
+                             f"{self.modelName}, {key}, {val if val is not None else '<?value?>'}"
+                             ")\") ")
+            else:
+                classdata[key] = val
+                if overwritedata is not None:
+                    overwritedata[key] = val
+                status = True
+
             inputdata_status[key] = status
 
         return all(inputdata_status.values())