From 65b646e59e2d85854b856ffe83802210f3f7e2a7 Mon Sep 17 00:00:00 2001 From: Rohit Waghchaure Date: Sat, 23 Feb 2019 07:17:26 +0530 Subject: [PATCH] fix: Removed duplicate page user permissions for page and report --- .../__init__.py | 0 .../user_permission_for_page_and_report.js | 55 ---- .../user_permission_for_page_and_report.json | 309 ------------------ .../user_permission_for_page_and_report.py | 49 --- frappe/patches.txt | 3 +- ...pe_user_permissions_for_page_and_report.py | 9 + 6 files changed, 11 insertions(+), 414 deletions(-) delete mode 100644 frappe/core/doctype/user_permission_for_page_and_report/__init__.py delete mode 100644 frappe/core/doctype/user_permission_for_page_and_report/user_permission_for_page_and_report.js delete mode 100644 frappe/core/doctype/user_permission_for_page_and_report/user_permission_for_page_and_report.json delete mode 100644 frappe/core/doctype/user_permission_for_page_and_report/user_permission_for_page_and_report.py create mode 100644 frappe/patches/v11_0/remove_doctype_user_permissions_for_page_and_report.py diff --git a/frappe/core/doctype/user_permission_for_page_and_report/__init__.py b/frappe/core/doctype/user_permission_for_page_and_report/__init__.py deleted file mode 100644 index e69de29bb2..0000000000 diff --git a/frappe/core/doctype/user_permission_for_page_and_report/user_permission_for_page_and_report.js b/frappe/core/doctype/user_permission_for_page_and_report/user_permission_for_page_and_report.js deleted file mode 100644 index d5293ddfe1..0000000000 --- a/frappe/core/doctype/user_permission_for_page_and_report/user_permission_for_page_and_report.js +++ /dev/null @@ -1,55 +0,0 @@ -// Copyright (c) 2016, Frappe Technologies and contributors -// For license information, please see license.txt - -frappe.ui.form.on('User Permission for Page and Report', { - refresh: function(frm) { - frm.disable_save(); - frm.role_area.hide(); - }, - - onload: function(frm) { - if(!frm.roles_editor) { - frm.role_area = $('
') - .appendTo(frm.fields_dict.roles_html.wrapper); - frm.roles_editor = new frappe.RoleEditor(frm.role_area, frm); - } - }, - - page: function(frm) { - frm.trigger("get_roles"); - }, - - report: function(frm){ - frm.trigger("get_roles"); - }, - - get_roles: function(frm) { - frm.role_area.show(); - - return frappe.call({ - method:"get_custom_roles", - doc: frm.doc, - callback: function(r) { - refresh_field('roles'); - frm.roles_editor.show(); - } - }); - }, - - update: function(frm) { - if(frm.roles_editor) { - frm.roles_editor.set_roles_in_table(); - } - - return frappe.call({ - method:"set_custom_roles", - doc: frm.doc, - callback: function(r) { - refresh_field('roles'); - frm.roles_editor.show(); - frappe.msgprint(__("Successfully Updated")); - frm.reload_doc(); - } - }); - } -}); diff --git a/frappe/core/doctype/user_permission_for_page_and_report/user_permission_for_page_and_report.json b/frappe/core/doctype/user_permission_for_page_and_report/user_permission_for_page_and_report.json deleted file mode 100644 index 040a136347..0000000000 --- a/frappe/core/doctype/user_permission_for_page_and_report/user_permission_for_page_and_report.json +++ /dev/null @@ -1,309 +0,0 @@ -{ - "allow_copy": 1, - "allow_guest_to_view": 0, - "allow_import": 0, - "allow_rename": 0, - "beta": 0, - "creation": "2017-02-13 17:33:25.157332", - "custom": 0, - "docstatus": 0, - "doctype": "DocType", - "document_type": "", - "editable_grid": 1, - "engine": "InnoDB", - "fields": [ - { - "allow_bulk_edit": 0, - "allow_on_submit": 0, - "bold": 0, - "collapsible": 0, - "columns": 0, - "fieldname": "set_role_for", - "fieldtype": "Select", - "hidden": 0, - "ignore_user_permissions": 0, - "ignore_xss_filter": 0, - "in_filter": 0, - "in_global_search": 0, - "in_list_view": 1, - "in_standard_filter": 0, - "label": "Set Role For", - "length": 0, - "no_copy": 0, - "options": "\nPage\nReport", - "permlevel": 0, - "precision": "", - "print_hide": 0, - "print_hide_if_no_value": 0, - "read_only": 0, - "remember_last_selected_value": 0, - "report_hide": 0, - "reqd": 1, - "search_index": 0, - "set_only_once": 0, - "unique": 0 - }, - { - "allow_bulk_edit": 0, - "allow_on_submit": 0, - "bold": 0, - "collapsible": 0, - "columns": 0, - "depends_on": "eval:doc.set_role_for == 'Page'", - "fieldname": "page", - "fieldtype": "Link", - "hidden": 0, - "ignore_user_permissions": 0, - "ignore_xss_filter": 0, - "in_filter": 0, - "in_global_search": 0, - "in_list_view": 0, - "in_standard_filter": 0, - "label": "Page", - "length": 0, - "no_copy": 0, - "options": "Page", - "permlevel": 0, - "precision": "", - "print_hide": 0, - "print_hide_if_no_value": 0, - "read_only": 0, - "remember_last_selected_value": 0, - "report_hide": 0, - "reqd": 0, - "search_index": 0, - "set_only_once": 0, - "unique": 0 - }, - { - "allow_bulk_edit": 0, - "allow_on_submit": 0, - "bold": 0, - "collapsible": 0, - "columns": 0, - "depends_on": "eval:doc.set_role_for == 'Report'", - "fieldname": "report", - "fieldtype": "Link", - "hidden": 0, - "ignore_user_permissions": 0, - "ignore_xss_filter": 0, - "in_filter": 0, - "in_global_search": 0, - "in_list_view": 0, - "in_standard_filter": 0, - "label": "Report", - "length": 0, - "no_copy": 0, - "options": "Report", - "permlevel": 0, - "precision": "", - "print_hide": 0, - "print_hide_if_no_value": 0, - "read_only": 0, - "remember_last_selected_value": 0, - "report_hide": 0, - "reqd": 0, - "search_index": 0, - "set_only_once": 0, - "unique": 0 - }, - { - "allow_bulk_edit": 0, - "allow_on_submit": 0, - "bold": 0, - "collapsible": 0, - "columns": 0, - "fieldname": "roles_permission", - "fieldtype": "Section Break", - "hidden": 0, - "ignore_user_permissions": 0, - "ignore_xss_filter": 0, - "in_filter": 0, - "in_global_search": 0, - "in_list_view": 0, - "in_standard_filter": 0, - "label": "Roles Permission", - "length": 0, - "no_copy": 0, - "permlevel": 0, - "precision": "", - "print_hide": 0, - "print_hide_if_no_value": 0, - "read_only": 0, - "remember_last_selected_value": 0, - "report_hide": 0, - "reqd": 0, - "search_index": 0, - "set_only_once": 0, - "unique": 0 - }, - { - "allow_bulk_edit": 0, - "allow_on_submit": 0, - "bold": 0, - "collapsible": 0, - "columns": 0, - "depends_on": "", - "fieldname": "roles_html", - "fieldtype": "HTML", - "hidden": 0, - "ignore_user_permissions": 0, - "ignore_xss_filter": 0, - "in_filter": 0, - "in_global_search": 0, - "in_list_view": 0, - "in_standard_filter": 0, - "label": "Roles Html", - "length": 0, - "no_copy": 0, - "permlevel": 0, - "precision": "", - "print_hide": 0, - "print_hide_if_no_value": 0, - "read_only": 0, - "remember_last_selected_value": 0, - "report_hide": 0, - "reqd": 0, - "search_index": 0, - "set_only_once": 0, - "unique": 0 - }, - { - "allow_bulk_edit": 0, - "allow_on_submit": 0, - "bold": 0, - "collapsible": 0, - "columns": 0, - "fieldname": "roles", - "fieldtype": "Table", - "hidden": 1, - "ignore_user_permissions": 0, - "ignore_xss_filter": 0, - "in_filter": 0, - "in_global_search": 0, - "in_list_view": 0, - "in_standard_filter": 0, - "label": "Roles", - "length": 0, - "no_copy": 0, - "options": "Has Role", - "permlevel": 0, - "precision": "", - "print_hide": 0, - "print_hide_if_no_value": 0, - "read_only": 1, - "remember_last_selected_value": 0, - "report_hide": 0, - "reqd": 0, - "search_index": 0, - "set_only_once": 0, - "unique": 0 - }, - { - "allow_bulk_edit": 0, - "allow_on_submit": 0, - "bold": 0, - "collapsible": 0, - "columns": 0, - "fieldname": "section_break_8", - "fieldtype": "Section Break", - "hidden": 0, - "ignore_user_permissions": 0, - "ignore_xss_filter": 0, - "in_filter": 0, - "in_global_search": 0, - "in_list_view": 0, - "in_standard_filter": 0, - "length": 0, - "no_copy": 0, - "permlevel": 0, - "precision": "", - "print_hide": 0, - "print_hide_if_no_value": 0, - "read_only": 0, - "remember_last_selected_value": 0, - "report_hide": 0, - "reqd": 0, - "search_index": 0, - "set_only_once": 0, - "unique": 0 - }, - { - "allow_bulk_edit": 0, - "allow_on_submit": 0, - "bold": 0, - "collapsible": 0, - "columns": 0, - "fieldname": "update", - "fieldtype": "Button", - "hidden": 0, - "ignore_user_permissions": 0, - "ignore_xss_filter": 0, - "in_filter": 0, - "in_global_search": 0, - "in_list_view": 0, - "in_standard_filter": 0, - "label": "Update", - "length": 0, - "no_copy": 0, - "options": "", - "permlevel": 0, - "precision": "", - "print_hide": 0, - "print_hide_if_no_value": 0, - "read_only": 0, - "remember_last_selected_value": 0, - "report_hide": 0, - "reqd": 0, - "search_index": 0, - "set_only_once": 0, - "unique": 0 - } - ], - "has_web_view": 0, - "hide_heading": 0, - "hide_toolbar": 1, - "idx": 0, - "image_view": 0, - "in_create": 0, - "is_submittable": 0, - "issingle": 1, - "istable": 0, - "max_attachments": 0, - "modified": "2017-12-21 04:24:24.963988", - "modified_by": "Administrator", - "module": "Core", - "name": "User Permission for Page and Report", - "name_case": "", - "owner": "Administrator", - "permissions": [ - { - "amend": 0, - "apply_user_permissions": 0, - "cancel": 0, - "create": 1, - "delete": 1, - "email": 1, - "export": 0, - "if_owner": 0, - "import": 0, - "permlevel": 0, - "print": 1, - "read": 1, - "report": 0, - "role": "System Manager", - "set_user_permissions": 0, - "share": 1, - "submit": 0, - "write": 1 - } - ], - "quick_entry": 1, - "read_only": 0, - "read_only_onload": 0, - "show_name_in_global_search": 0, - "sort_field": "modified", - "sort_order": "DESC", - "track_changes": 0, - "track_seen": 0 -} \ No newline at end of file diff --git a/frappe/core/doctype/user_permission_for_page_and_report/user_permission_for_page_and_report.py b/frappe/core/doctype/user_permission_for_page_and_report/user_permission_for_page_and_report.py deleted file mode 100644 index 7efe3397f6..0000000000 --- a/frappe/core/doctype/user_permission_for_page_and_report/user_permission_for_page_and_report.py +++ /dev/null @@ -1,49 +0,0 @@ -# -*- coding: utf-8 -*- -# Copyright (c) 2015, Frappe Technologies and contributors -# For license information, please see license.txt - -from __future__ import unicode_literals -import frappe -from frappe.model.document import Document - -class UserPermissionforPageandReport(Document): - def get_custom_roles(self): - args = self.get_args() - self.set('roles', []) - - name = frappe.db.get_value('Custom Role', args, "name") - if name: - doc = frappe.get_doc('Custom Role', name) - else: - doctype = self.set_role_for - docname = self.page if self.set_role_for == 'Page' else self.report - doc = frappe.get_doc(doctype, docname) - - self.set('roles', doc.roles) - - def set_custom_roles(self): - args = self.get_args() - name = frappe.db.get_value('Custom Role', args, "name") - - args.update({ - 'doctype': 'Custom Role', - 'roles': self.roles - }) - - if name: - doc = frappe.get_doc("Custom Role", name) - doc.set('roles', self.roles) - doc.save() - else: - frappe.get_doc(args).insert() - - def get_args(self, row=None): - name = self.page if self.set_role_for == 'Page' else self.report - check_for_field = self.set_role_for.replace(" ","_").lower() - - return { - check_for_field: name - } - - def update_status(self): - return frappe.render_template diff --git a/frappe/patches.txt b/frappe/patches.txt index ff4079afa1..fdd032131d 100644 --- a/frappe/patches.txt +++ b/frappe/patches.txt @@ -234,4 +234,5 @@ frappe.patches.v11_0.migrate_report_settings_for_new_listview frappe.patches.v11_0.delete_all_prepared_reports frappe.patches.v11_0.fix_order_by_in_reports_json execute:frappe.delete_doc('Page', 'applications', ignore_missing=True) -frappe.patches.v11_0.set_missing_creation_and_modified_value_for_user_permissions \ No newline at end of file +frappe.patches.v11_0.set_missing_creation_and_modified_value_for_user_permissions +frappe.patches.v11_0.remove_doctype_user_permissions_for_page_and_report \ No newline at end of file diff --git a/frappe/patches/v11_0/remove_doctype_user_permissions_for_page_and_report.py b/frappe/patches/v11_0/remove_doctype_user_permissions_for_page_and_report.py new file mode 100644 index 0000000000..c1dc1b79be --- /dev/null +++ b/frappe/patches/v11_0/remove_doctype_user_permissions_for_page_and_report.py @@ -0,0 +1,9 @@ +# Copyright (c) 2015, Frappe Technologies Pvt. Ltd. and Contributors +# MIT License. See license.txt + +from __future__ import unicode_literals +import frappe + +def execute(): + if frappe.db.table_exists('User Permission for Page and Report'): + frappe.delete_doc("DocType", "User Permission for Page and Report") \ No newline at end of file