diff --git a/frappe/core/doctype/server_script/test_server_script.py b/frappe/core/doctype/server_script/test_server_script.py index c39fcfa0d0..ed6b5539c5 100644 --- a/frappe/core/doctype/server_script/test_server_script.py +++ b/frappe/core/doctype/server_script/test_server_script.py @@ -109,3 +109,13 @@ class TestServerScript(unittest.TestCase): """Raise AttributeError if method not found in Namespace""" note = frappe.get_doc({"doctype": "Note", "title": "Test Note: Server Script"}) self.assertRaises(AttributeError, note.insert) + + def test_syntax_validation(self): + server_script = scripts[0] + server_script["script"] = "js || code.?" + + with self.assertRaises(frappe.ValidationError) as se: + frappe.get_doc(doctype="Server Script", **server_script).insert() + + self.assertTrue("invalid python code" in str(se.exception).lower(), + msg="Python code validation not working") diff --git a/frappe/handler.py b/frappe/handler.py index 8d0c18a00b..9d32b0acee 100755 --- a/frappe/handler.py +++ b/frappe/handler.py @@ -53,7 +53,7 @@ def execute_cmd(cmd, from_async=False): try: method = get_attr(cmd) except Exception as e: - frappe.throw(_('Invalid Method')) + frappe.throw(_('Failed to get method for command {0} with {1}').format(cmd, e)) if from_async: method = method.queue diff --git a/frappe/model/document.py b/frappe/model/document.py index 7443c92ab4..5c977bf0c6 100644 --- a/frappe/model/document.py +++ b/frappe/model/document.py @@ -507,6 +507,7 @@ class Document(BaseDocument): d._validate_selects() d._validate_non_negative() d._validate_length() + d._validate_code_fields() d._extract_images_from_text_editor() d._sanitize_content() d._save_passwords() @@ -1064,7 +1065,10 @@ class Document(BaseDocument): self.set("modified", now()) self.set("modified_by", frappe.session.user) - self.load_doc_before_save() + # load but do not reload doc_before_save because before_change or on_change might expect it + if not self.get_doc_before_save(): + self.load_doc_before_save() + # to trigger notification on value change self.run_method('before_change') diff --git a/frappe/public/js/frappe/form/controls/code.js b/frappe/public/js/frappe/form/controls/code.js index 9155333ee3..5af7cf2863 100644 --- a/frappe/public/js/frappe/form/controls/code.js +++ b/frappe/public/js/frappe/form/controls/code.js @@ -24,6 +24,7 @@ frappe.ui.form.ControlCode = class ControlCode extends frappe.ui.form.ControlTex this.editor = ace.edit(this.ace_editor_target.get(0)); this.editor.setTheme('ace/theme/tomorrow'); this.editor.setOption("showPrintMargin", false); + this.editor.setOption("wrap", this.df.wrap); this.set_language(); // events diff --git a/frappe/workflow/doctype/workflow/test_workflow.py b/frappe/workflow/doctype/workflow/test_workflow.py index 9bafd377fc..cd36fd2d0b 100644 --- a/frappe/workflow/doctype/workflow/test_workflow.py +++ b/frappe/workflow/doctype/workflow/test_workflow.py @@ -121,6 +121,16 @@ class TestWorkflow(unittest.TestCase): self.workflow.states[1].doc_status = 0 self.workflow.save() + def test_syntax_error_in_transition_rule(self): + self.workflow.transitions[0].condition = 'doc.status =! "Closed"' + + with self.assertRaises(frappe.ValidationError) as se: + self.workflow.save() + + self.assertTrue("invalid python code" in str(se.exception).lower(), + msg="Python code validation not working") + + def create_todo_workflow(): if frappe.db.exists('Workflow', 'Test ToDo'): frappe.delete_doc('Workflow', 'Test ToDo')