From 4a7551a0100d0351375856247a89ee9d7db5ef44 Mon Sep 17 00:00:00 2001 From: Rushabh Mehta Date: Fri, 7 Jul 2017 17:40:22 +0530 Subject: [PATCH 1/2] [hot] more js fixes --- frappe/public/js/frappe/form/control.js | 2 +- frappe/public/js/frappe/form/script_manager.js | 16 +++++++++++++--- 2 files changed, 14 insertions(+), 4 deletions(-) diff --git a/frappe/public/js/frappe/form/control.js b/frappe/public/js/frappe/form/control.js index b1d765aaf8..854f04369a 100755 --- a/frappe/public/js/frappe/form/control.js +++ b/frappe/public/js/frappe/form/control.js @@ -129,7 +129,7 @@ frappe.ui.form.Control = Class.extend({ }; value = this.validate(value); - if (value.then) { + if (value && value.then) { // got a promise return value.then((value) => set(value)); } else { diff --git a/frappe/public/js/frappe/form/script_manager.js b/frappe/public/js/frappe/form/script_manager.js index 97663eb0d7..fce960e0d9 100644 --- a/frappe/public/js/frappe/form/script_manager.js +++ b/frappe/public/js/frappe/form/script_manager.js @@ -101,11 +101,21 @@ frappe.ui.form.ScriptManager = Class.extend({ // make list of functions to be run serially handlers.new_style.forEach((_function) => { - tasks.push(() => runner(_function, false)); + if(event_name==='setup') { + // setup must be called immediately + runner(_function, false); + } else { + tasks.push(() => runner(_function, false)); + } }); handlers.old_style.forEach((_function) => { - tasks.push(() => runner(_function, true)); + if(event_name==='setup') { + // setup must be called immediately + runner(_function, false); + } else { + tasks.push(() => runner(_function, true)); + } }); // run them serially @@ -147,7 +157,7 @@ frappe.ui.form.ScriptManager = Class.extend({ if(doctype.__custom_js) { try { - eval(doctype.__custom_js) + eval(doctype.__custom_js); } catch(e) { frappe.msgprint({ title: __('Error in Custom Script'), From e1c4c2ae9dfd22eddea3375ea9d97725c9e964c7 Mon Sep 17 00:00:00 2001 From: mbauskar Date: Fri, 7 Jul 2017 18:34:18 +0600 Subject: [PATCH 2/2] bumped to version 8.3.4 --- frappe/__init__.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/frappe/__init__.py b/frappe/__init__.py index 1f4e37c3d0..cd2d985191 100644 --- a/frappe/__init__.py +++ b/frappe/__init__.py @@ -14,7 +14,7 @@ import os, sys, importlib, inspect, json from .exceptions import * from .utils.jinja import get_jenv, get_template, render_template -__version__ = '8.3.3' +__version__ = '8.3.4' __title__ = "Frappe Framework" local = Local()