Merge pull request #2939 from rohitwaghchaure/minor_fix_report_page_role

Cleanup the code
2017-03-27 13:21:54 +05:30 · 2017-03-27 13:21:54 +05:30 · e5f9b2f3ea
commit e5f9b2f3ea
parent c8e1001572 ab077e99e9
1 changed files with 6 additions and 2 deletions
--- a/frappe/core/doctype/role_permission_for_page_and_report/role_permission_for_page_and_report.js
+++ b/frappe/core/doctype/role_permission_for_page_and_report/role_permission_for_page_and_report.js
@ -30,11 +30,15 @@ frappe.ui.form.on('Role Permission for Page and Report', {
 	},

 	page: function(frm) {
-		frm.trigger("get_roles")
+		if(frm.doc.page) {
+			frm.trigger("get_roles")
+		}
 	},

 	report: function(frm){
-		frm.trigger("get_roles")
+		if(frm.doc.report) {
+			frm.trigger("get_roles")
+		}
 	},

 	get_roles: function(frm) {