[ui][node] ExpressionTextField : fix lossing focus reset

Author: Alxiice

meshroom/core/evaluation.py

@@ -13,14 +13,15 @@ class MathEvaluator:
         print(mev.evaluate("pow(2, 8)"))
         print(mev.evaluate("round(sin(pi), 3)"))
     """
-    
+
     # Allowed math symbols
     allowed_symbols = {
         "e": math.e, "pi": math.pi,
         "cos": math.cos, "sin": math.sin, "tan": math.tan, "exp": math.exp,
         "pow": pow, "round": round, "abs": abs, "min": min, "max": max,
         "sqrt": math.sqrt, "log": math.log
     }
+
     # Allowed AST node types
     allowed_nodes = (
         ast.Expression, ast.BinOp, ast.UnaryOp, ast.Call, ast.Name, ast.Load,
@@ -29,6 +30,7 @@ class MathEvaluator:
         ast.LShift, ast.RShift, ast.Invert,
         ast.Constant
     )
+
     def _validate_ast(self, node):
         for child in ast.walk(node):
             if not isinstance(child, self.allowed_nodes):
@@ -37,6 +39,7 @@ def _validate_ast(self, node):
             if isinstance(child, ast.Name):
                 if child.id not in self.allowed_symbols:
                     raise ValueError(f"Unknown symbol: {child.id}")
+
     def evaluate(self, expr: str):
         if any(bad in expr for bad in ('\n', '#')):
             raise ValueError(f"Invalid expression: {expr}")

meshroom/ui/qml/Utils/ExpressionTextField.qml

@@ -106,7 +106,9 @@ TextField {
         }
     }
 
-    Component.onCompleted: {
-        refreshStatus()
+    Component.onDestruction: {
+        if (textChanged) {
+            root.accepted()
+        }
     }
 }

meshroom/ui/reconstruction.py

@@ -1179,6 +1179,7 @@ def evaluateMathExpression(self, expr):
             res = mev.evaluate(expr)
             return [res, False]
         except Exception as err:
+            self.parent().showMessage(f"Invalid field expression : {expr}", "error")
             return [None, True]
 
     selectedViewIdChanged = Signal()