src/Builder4/PopoverProperty.bjs wip_alan_T7437_more_depricated origin/wip_alan_T7437_more_depricated
authorAlan Knowles <alan@roojs.com>
Mon, 17 Oct 2022 08:01:09 +0000 (16:01 +0800)
committerAlan Knowles <alan@roojs.com>
Mon, 17 Oct 2022 08:01:09 +0000 (16:01 +0800)
src/Builder4/PopoverProperty.vala

src/Builder4/PopoverProperty.bjs
src/Builder4/PopoverProperty.vala

index a2d95f3..f708309 100644 (file)
            "\t// check if text is not empty..",
            "\tif ( _this.kname.el.get_text().strip().length < 1) {",
            "\t\t// error should already be showing?",
-           "\t\treturn;",
+           "\t\treturn false;",
            "\t}",
            "\t_this.updateProp();",
            "\t",
            "\tvar prop = _this.prop;",
            "\tif (_this.node.props.has_key(prop.to_index_key())) {",
            "\t\t_this.error.setError(\"Property already exists\");",
-           "\t\treturn;\t",
+           "\t\treturn false;\t",
            "\t}",
            "\t",
            "\t",
index a36226c..cd1e583 100644 (file)
@@ -714,7 +714,7 @@ public class Xcls_PopoverProperty : Object
                // check if text is not empty..
                if ( _this.kname.el.get_text().strip().length < 1) {
                        // error should already be showing?
-                       return;
+                       return false;
                }
                _this.updateProp();
                
@@ -724,7 +724,7 @@ public class Xcls_PopoverProperty : Object
                var prop = _this.prop;
                if (_this.node.props.has_key(prop.to_index_key())) {
                        _this.error.setError("Property already exists");
-                       return; 
+                       return false;   
                }