From 8a2c83918be454f247c35fb53a858e61becbc8a8 Mon Sep 17 00:00:00 2001 From: Rohit Waghchaure Date: Mon, 20 Feb 2017 14:38:42 +0530 Subject: [PATCH] changed user_roles to roles --- .../custom_role_manager.js | 2 +- .../custom_role_manager.py | 8 ++--- frappe/core/doctype/user/test_user.py | 6 ++-- frappe/core/doctype/user/user.js | 6 ++-- frappe/core/doctype/user/user.py | 32 +++++++++---------- .../page/user_permissions/user_permissions.js | 2 +- frappe/desk/page/setup_wizard/setup_wizard.py | 2 +- .../doctype/email_queue/email_queue_list.js | 2 +- .../doctype/ldap_settings/ldap_settings.py | 2 +- frappe/model/document.py | 6 ++-- frappe/model/sync.py | 2 +- .../v6_20x/remove_roles_from_website_user.py | 4 +-- .../v7_3/copy_user_roles_to_has_roles.py | 15 +++++++++ .../v7_3/set_ref_doctype_roles_to_report.py | 2 +- frappe/permissions.py | 4 +-- frappe/public/js/frappe/desk.js | 8 ++--- frappe/public/js/frappe/form/toolbar.js | 2 +- frappe/public/js/frappe/form/workflow.js | 2 +- frappe/public/js/frappe/misc/tools.js | 2 +- frappe/public/js/frappe/model/model.js | 6 ++-- frappe/public/js/frappe/model/perm.js | 4 +-- frappe/public/js/frappe/model/workflow.js | 2 +- frappe/public/js/frappe/roles_editor.js | 6 ++-- frappe/public/js/legacy/globals.js | 2 +- frappe/sessions.py | 2 +- frappe/tests/test_data_import.py | 6 ++-- frappe/tests/test_document.py | 8 ++--- frappe/utils/install.py | 4 +-- 28 files changed, 82 insertions(+), 67 deletions(-) create mode 100644 frappe/patches/v7_3/copy_user_roles_to_has_roles.py diff --git a/frappe/core/doctype/custom_role_manager/custom_role_manager.js b/frappe/core/doctype/custom_role_manager/custom_role_manager.js index 814aa12ccb..c020191526 100644 --- a/frappe/core/doctype/custom_role_manager/custom_role_manager.js +++ b/frappe/core/doctype/custom_role_manager/custom_role_manager.js @@ -19,7 +19,7 @@ frappe.ui.form.on('Custom Role Manager', { method:"get_custom_roles", doc: frm.doc, callback: function(r) { - refresh_field('has_roles') + refresh_field('roles') frm.roles_editor.show() } }) diff --git a/frappe/core/doctype/custom_role_manager/custom_role_manager.py b/frappe/core/doctype/custom_role_manager/custom_role_manager.py index fe86e1dba1..bb21c2af9b 100644 --- a/frappe/core/doctype/custom_role_manager/custom_role_manager.py +++ b/frappe/core/doctype/custom_role_manager/custom_role_manager.py @@ -12,11 +12,11 @@ class CustomRoleManager(Document): args = self.get_args() name = frappe.db.get_value('Custom Role', args, "name") if not name: - self.set('has_roles', []) + self.set('roles', []) return doc = frappe.get_doc('Custom Role', name) - self.set('has_roles', doc.roles) + self.set('roles', doc.roles) def set_custom_roles(self): args = self.get_args() @@ -24,12 +24,12 @@ class CustomRoleManager(Document): args.update({ 'doctype': 'Custom Role', - 'has_roles': self.has_roles + 'roles': self.roles }) if name: doc = frappe.get_doc("Custom Role", name) - doc.set('has_roles', self.has_roles) + doc.set('roles', self.roles) doc.save() else: frappe.get_doc(args).insert() diff --git a/frappe/core/doctype/user/test_user.py b/frappe/core/doctype/user/test_user.py index 6f3be650b4..e616c3f1fa 100644 --- a/frappe/core/doctype/user/test_user.py +++ b/frappe/core/doctype/user/test_user.py @@ -27,7 +27,7 @@ class TestUser(unittest.TestCase): self.assertEquals(new_user.user_type, 'System User') # clear role - new_user.user_roles = [] + new_user.roles = [] new_user.save() self.assertEquals(new_user.user_type, 'Website User') @@ -86,7 +86,7 @@ class TestUser(unittest.TestCase): def test_high_permlevel_validations(self): user = frappe.get_meta("User") - self.assertTrue("user_roles" in [d.fieldname for d in user.get_high_permlevel_fields()]) + self.assertTrue("roles" in [d.fieldname for d in user.get_high_permlevel_fields()]) me = frappe.get_doc("User", "testperm@example.com") me.remove_roles("System Manager") @@ -101,7 +101,7 @@ class TestUser(unittest.TestCase): me = frappe.get_doc("User", "testperm@example.com") me.add_roles("System Manager") - self.assertTrue("System Manager" in [d.role for d in me.get("user_roles")]) + self.assertTrue("System Manager" in [d.role for d in me.get("roles")]) def test_user_limit_for_site(self): update_limits({'users': get_total_users()}) diff --git a/frappe/core/doctype/user/user.js b/frappe/core/doctype/user/user.js index 8d963299bb..d7b5fad06f 100644 --- a/frappe/core/doctype/user/user.js +++ b/frappe/core/doctype/user/user.js @@ -18,7 +18,7 @@ frappe.ui.form.on('User', { }, onload: function(frm) { - if(has_common(has_roles, ["Administrator", "System Manager"]) && !frm.doc.__islocal) { + if(has_common(roles, ["Administrator", "System Manager"]) && !frm.doc.__islocal) { if(!frm.roles_editor) { var role_area = $('
') .appendTo(frm.fields_dict.roles_html.wrapper); @@ -53,7 +53,7 @@ frappe.ui.form.on('User', { frappe.set_route("modules_setup"); }, null, "btn-default") - if(has_common(has_roles, ["Administrator", "System Manager"])) { + if(has_common(roles, ["Administrator", "System Manager"])) { frm.add_custom_button(__("Set User Permissions"), function() { frappe.route_options = { @@ -104,7 +104,7 @@ frappe.ui.form.on('User', { }, enabled: function(frm) { var doc = frm.doc; - if(!doc.__islocal && has_common(has_roles, ["Administrator", "System Manager"])) { + if(!doc.__islocal && has_common(roles, ["Administrator", "System Manager"])) { frm.toggle_display(['sb1', 'sb3', 'modules_access'], doc.enabled); frm.set_df_property('enabled', 'read_only', 0); } diff --git a/frappe/core/doctype/user/user.py b/frappe/core/doctype/user/user.py index ff7a3baba1..0a69b1d3f3 100644 --- a/frappe/core/doctype/user/user.py +++ b/frappe/core/doctype/user/user.py @@ -102,19 +102,19 @@ class User(Document): def add_system_manager_role(self): # if adding system manager, do nothing if not cint(self.enabled) or ("System Manager" in [user_role.role for user_role in - self.get("user_roles")]): + self.get("roles")]): return if self.name not in STANDARD_USERS and self.user_type == "System User" and not self.get_other_system_managers(): msgprint(_("Adding System Manager to this User as there must be atleast one System Manager")) - self.append("user_roles", { + self.append("roles", { "doctype": "UserRole", "role": "System Manager" }) if self.name == 'Administrator': # Administrator should always have System Manager Role - self.extend("user_roles", [ + self.extend("roles", [ { "doctype": "UserRole", "role": "System Manager" @@ -142,13 +142,13 @@ class User(Document): def has_desk_access(self): '''Return true if any of the set roles has desk access''' - if not self.user_roles: + if not self.roles: return False return len(frappe.db.sql("""select name from `tabRole` where desk_access=1 - and name in ({0}) limit 1""".format(', '.join(['%s'] * len(self.user_roles))), - [d.role for d in self.user_roles])) + and name in ({0}) limit 1""".format(', '.join(['%s'] * len(self.roles))), + [d.role for d in self.roles])) def share_with_self(self): @@ -338,11 +338,11 @@ class User(Document): def append_roles(self, *roles): """Add roles to user""" - current_roles = [d.role for d in self.get("user_roles")] + current_roles = [d.role for d in self.get("roles")] for role in roles: if role in current_roles: continue - self.append("user_roles", {"role": role}) + self.append("roles", {"role": role}) def add_roles(self, *roles): """Add roles to user and save""" @@ -350,28 +350,28 @@ class User(Document): self.save() def remove_roles(self, *roles): - existing_roles = dict((d.role, d) for d in self.get("user_roles")) + existing_roles = dict((d.role, d) for d in self.get("roles")) for role in roles: if role in existing_roles: - self.get("user_roles").remove(existing_roles[role]) + self.get("roles").remove(existing_roles[role]) self.save() def remove_all_roles_for_guest(self): if self.name == "Guest": - self.set("user_roles", list(set(d for d in self.get("user_roles") if d.role == "Guest"))) + self.set("roles", list(set(d for d in self.get("roles") if d.role == "Guest"))) def remove_disabled_roles(self): disabled_roles = [d.name for d in frappe.get_all("Role", filters={"disabled":1})] - for role in list(self.get('user_roles')): + for role in list(self.get('roles')): if role.role in disabled_roles: - self.get('user_roles').remove(role) + self.get('roles').remove(role) def ensure_unique_roles(self): exists = [] - for i, d in enumerate(self.get("user_roles")): + for i, d in enumerate(self.get("roles")): if (not d.role) or (d.role in exists): - self.get("user_roles").remove(d) + self.get("roles").remove(d) else: exists.append(d.role) @@ -465,7 +465,7 @@ def get_all_roles(arg=None): where name not in ('Administrator', 'Guest', 'All') and not disabled order by name""")] @frappe.whitelist() -def get_user_roles(arg=None): +def get_roles(arg=None): """get roles for a user""" return frappe.get_roles(frappe.form_dict['uid']) diff --git a/frappe/core/page/user_permissions/user_permissions.js b/frappe/core/page/user_permissions/user_permissions.js index 7dca0f2988..4922afe817 100644 --- a/frappe/core/page/user_permissions/user_permissions.js +++ b/frappe/core/page/user_permissions/user_permissions.js @@ -82,7 +82,7 @@ frappe.UserPermissions = Class.extend({ options: "[Select]" }); - if(has_roles.indexOf("System Manager")!==-1) { + if(roles.indexOf("System Manager")!==-1) { me.download = me.wrapper.page.add_field({ fieldname: "download", label: __("Download"), diff --git a/frappe/desk/page/setup_wizard/setup_wizard.py b/frappe/desk/page/setup_wizard/setup_wizard.py index 55ef8eb6b1..88f8ea82e5 100755 --- a/frappe/desk/page/setup_wizard/setup_wizard.py +++ b/frappe/desk/page/setup_wizard/setup_wizard.py @@ -134,7 +134,7 @@ def add_all_roles_to(name): user = frappe.get_doc("User", name) for role in frappe.db.sql("""select name from tabRole"""): if role[0] not in ["Administrator", "Guest", "All", "Customer", "Supplier", "Partner", "Employee"]: - d = user.append("user_roles") + d = user.append("roles") d.role = role[0] user.save() diff --git a/frappe/email/doctype/email_queue/email_queue_list.js b/frappe/email/doctype/email_queue/email_queue_list.js index b42b857def..d9d352f06d 100644 --- a/frappe/email/doctype/email_queue/email_queue_list.js +++ b/frappe/email/doctype/email_queue/email_queue_list.js @@ -4,7 +4,7 @@ frappe.listview_settings['Email Queue'] = { return [__(doc.status), colour[doc.status], "status,=," + doc.status]; }, refresh: function(doclist){ - if (has_common(has_roles, ["Administrator", "System Manager"])){ + if (has_common(roles, ["Administrator", "System Manager"])){ if (cint(frappe.defaults.get_default("hold_queue"))){ doclist.page.clear_inner_toolbar() doclist.page.add_inner_button(__("Resume Sending"), function() { diff --git a/frappe/integrations/doctype/ldap_settings/ldap_settings.py b/frappe/integrations/doctype/ldap_settings/ldap_settings.py index 81d420d18b..a9a553c094 100644 --- a/frappe/integrations/doctype/ldap_settings/ldap_settings.py +++ b/frappe/integrations/doctype/ldap_settings/ldap_settings.py @@ -158,7 +158,7 @@ def create_user(params): "send_welcome_email": 0, "language": "", "user_type": "System User", - "user_roles": [{ + "roles": [{ "role": _("Blogger") }] }) diff --git a/frappe/model/document.py b/frappe/model/document.py index aaee69a892..8f3025ba51 100644 --- a/frappe/model/document.py +++ b/frappe/model/document.py @@ -34,7 +34,7 @@ def get_doc(arg1, arg2=None): user = get_doc({ "doctype":"User" "email_id": "test@example.com", - "user_roles: [ + "roles: [ {"role": "System Manager"} ] }) @@ -459,10 +459,10 @@ class Document(BaseDocument): def get_permlevel_access(self, permission_type='write'): if not hasattr(self, "_has_access_to"): - user_roles = frappe.get_roles() + roles = frappe.get_roles() self._has_access_to = [] for perm in self.get_permissions(): - if perm.role in user_roles and perm.permlevel > 0 and perm.get(permission_type): + if perm.role in roles and perm.permlevel > 0 and perm.get(permission_type): if perm.permlevel not in self._has_access_to: self._has_access_to.append(perm.permlevel) diff --git a/frappe/model/sync.py b/frappe/model/sync.py index 283e6d81ab..b815265186 100644 --- a/frappe/model/sync.py +++ b/frappe/model/sync.py @@ -28,7 +28,7 @@ def sync_for(app_name, force=0, sync_everything = False, verbose=False, reset_pe if app_name == "frappe": # these need to go first at time of install for d in (("core", "docfield"), ("core", "docperm"), ("core", "doctype"), - ("core", "user"), ("core", "role"), ("core", "report_role"), ("custom", "custom_field"), + ("core", "user"), ("core", "role"), ("custom", "custom_field"), ("custom", "property_setter"), ("website", "web_form"), ("website", "web_form_field"), ("website", "portal_menu_item")): files.append(os.path.join(frappe.get_app_path("frappe"), d[0], diff --git a/frappe/patches/v6_20x/remove_roles_from_website_user.py b/frappe/patches/v6_20x/remove_roles_from_website_user.py index d5e2d37222..347491898f 100644 --- a/frappe/patches/v6_20x/remove_roles_from_website_user.py +++ b/frappe/patches/v6_20x/remove_roles_from_website_user.py @@ -3,6 +3,6 @@ import frappe def execute(): for user_name in frappe.get_all('User', filters={'user_type': 'Website User'}): user = frappe.get_doc('User', user_name) - if user.user_roles: - user.user_roles = [] + if user.roles: + user.roles = [] user.save() diff --git a/frappe/patches/v7_3/copy_user_roles_to_has_roles.py b/frappe/patches/v7_3/copy_user_roles_to_has_roles.py new file mode 100644 index 0000000000..1cb3bfbac0 --- /dev/null +++ b/frappe/patches/v7_3/copy_user_roles_to_has_roles.py @@ -0,0 +1,15 @@ +# Copyright (c) 2013, Web Notes Technologies Pvt. Ltd. and Contributors +# License: GNU General Public License v3. See license.txt + +from __future__ import unicode_literals +import frappe + +def execute(): + frappe.reload_doc("core", 'doctype', "user") + frappe.reload_doc("core", 'doctype', "has_role") + for data in frappe.get_all('User', fields = ["name"]): + doc = frappe.get_doc('User', data.name) + roles = [{'role': d.role} for d in doc.user_roles] + doc.set('roles', roles) + for role in doc.roles: + role.db_update() \ No newline at end of file diff --git a/frappe/patches/v7_3/set_ref_doctype_roles_to_report.py b/frappe/patches/v7_3/set_ref_doctype_roles_to_report.py index ff6dfdf64b..00cec9bf82 100644 --- a/frappe/patches/v7_3/set_ref_doctype_roles_to_report.py +++ b/frappe/patches/v7_3/set_ref_doctype_roles_to_report.py @@ -6,7 +6,7 @@ import frappe def execute(): frappe.reload_doc("core", 'doctype', "report") - frappe.reload_doc("core", 'doctype', "report_role") + frappe.reload_doc("core", 'doctype', "has_role") for data in frappe.get_all('Report', fields=["name"]): doc = frappe.get_doc('Report', data.name) doc.set_doctype_roles() diff --git a/frappe/permissions.py b/frappe/permissions.py index b4dee56bde..bfd82ed943 100644 --- a/frappe/permissions.py +++ b/frappe/permissions.py @@ -182,12 +182,12 @@ def get_role_permissions(meta, user=None, verbose=False): user_permission_doctypes={}, if_owner={} ) - user_roles = frappe.get_roles(user) + roles = frappe.get_roles(user) dont_match = [] has_a_role_with_apply_user_permissions = False for p in meta.permissions: - if cint(p.permlevel)==0 and (p.role in user_roles): + if cint(p.permlevel)==0 and (p.role in roles): # apply only for level 0 for ptype in rights: diff --git a/frappe/public/js/frappe/desk.js b/frappe/public/js/frappe/desk.js index 8e9bb657ef..0025895ef4 100644 --- a/frappe/public/js/frappe/desk.js +++ b/frappe/public/js/frappe/desk.js @@ -260,7 +260,7 @@ frappe.Application = Class.extend({ user = frappe.boot.user.name; user_fullname = frappe.user_info(user).fullname; user_defaults = frappe.boot.user.defaults; - has_roles = frappe.boot.user.roles; + roles = frappe.boot.user.roles; user_email = frappe.boot.user.email; sys_defaults = frappe.boot.sysdefaults; frappe.ui.py_date_format = frappe.boot.sysdefaults.date_format.replace('dd', '%d').replace('mm', '%m').replace('yyyy', '%Y'); @@ -288,7 +288,7 @@ frappe.Application = Class.extend({ user = 'Guest'; user_fullname = 'Guest'; user_defaults = {}; - has_roles = ['Guest']; + roles = ['Guest']; user_email = ''; sys_defaults = {}; }, @@ -489,12 +489,12 @@ frappe.get_desktop_icons = function(show_hidden, show_global) { } } - if(has_roles.indexOf('System Manager')!=-1) { + if(roles.indexOf('System Manager')!=-1) { var m = frappe.get_module('Setup'); if(show_module(m)) add_to_out(m) } - if(has_roles.indexOf('Administrator')!=-1) { + if(roles.indexOf('Administrator')!=-1) { var m = frappe.get_module('Core'); if(show_module(m)) add_to_out(m) } diff --git a/frappe/public/js/frappe/form/toolbar.js b/frappe/public/js/frappe/form/toolbar.js index 2c82926d50..35074a8a84 100644 --- a/frappe/public/js/frappe/form/toolbar.js +++ b/frappe/public/js/frappe/form/toolbar.js @@ -159,7 +159,7 @@ frappe.ui.form.Toolbar = Class.extend({ me.frm.savetrash();}, true); } - if(in_list(has_roles, "System Manager")) { + if(in_list(roles, "System Manager")) { this.page.add_menu_item(__("Customize"), function() { frappe.set_route("Form", "Customize Form", { doc_type: me.frm.doctype diff --git a/frappe/public/js/frappe/form/workflow.js b/frappe/public/js/frappe/form/workflow.js index 0ab3e1bd83..69bc1092d6 100644 --- a/frappe/public/js/frappe/form/workflow.js +++ b/frappe/public/js/frappe/form/workflow.js @@ -73,7 +73,7 @@ frappe.ui.form.States = Class.extend({ } $.each(frappe.workflow.get_transitions(this.frm.doctype, state), function(i, d) { - if(in_list(has_roles, d.allowed)) { + if(in_list(roles, d.allowed)) { added = true; me.frm.page.add_action_item(__(d.action), function() { var action = d.action; diff --git a/frappe/public/js/frappe/misc/tools.js b/frappe/public/js/frappe/misc/tools.js index 43a27fd989..622d0da507 100644 --- a/frappe/public/js/frappe/misc/tools.js +++ b/frappe/public/js/frappe/misc/tools.js @@ -4,7 +4,7 @@ frappe.provide("frappe.tools"); frappe.tools.downloadify = function(data, roles, title) { - if(roles && roles.length && !has_common(roles, has_roles)) { + if(roles && roles.length && !has_common(roles, roles)) { msgprint(__("Export not allowed. You need {0} role to export.", [frappe.utils.comma_or(roles)])); return; } diff --git a/frappe/public/js/frappe/model/model.js b/frappe/public/js/frappe/model/model.js index 8efae5558d..543a47db1f 100644 --- a/frappe/public/js/frappe/model/model.js +++ b/frappe/public/js/frappe/model/model.js @@ -230,7 +230,7 @@ $.extend(frappe.model, { can_import: function(doctype, frm) { // system manager can always import - if(has_roles.indexOf("System Manager")!==-1) return true; + if(roles.indexOf("System Manager")!==-1) return true; if(frm) return frm.perm[0].import===1; return frappe.boot.user.can_import.indexOf(doctype)!==-1; @@ -238,7 +238,7 @@ $.extend(frappe.model, { can_export: function(doctype, frm) { // system manager can always export - if(has_roles.indexOf("System Manager")!==-1) return true; + if(roles.indexOf("System Manager")!==-1) return true; if(frm) return frm.perm[0].export===1; return frappe.boot.user.can_export.indexOf(doctype)!==-1; @@ -263,7 +263,7 @@ $.extend(frappe.model, { can_set_user_permissions: function(doctype, frm) { // system manager can always set user permissions - if(has_roles.indexOf("System Manager")!==-1) return true; + if(roles.indexOf("System Manager")!==-1) return true; if(frm) return frm.perm[0].set_user_permissions===1; return frappe.boot.user.can_set_user_permissions.indexOf(doctype)!==-1; diff --git a/frappe/public/js/frappe/model/perm.js b/frappe/public/js/frappe/model/perm.js index 753966baa5..f660440c05 100644 --- a/frappe/public/js/frappe/model/perm.js +++ b/frappe/public/js/frappe/model/perm.js @@ -45,7 +45,7 @@ $.extend(frappe.perm, { return perm; } - if (user==="Administrator" || has_roles.indexOf("Administrator")!==-1) { + if (user==="Administrator" || roles.indexOf("Administrator")!==-1) { perm[0].read = 1; } @@ -109,7 +109,7 @@ $.extend(frappe.perm, { $.each(meta.permissions || [], function(i, p) { // if user has this role - if(has_roles.indexOf(p.role)!==-1) { + if(roles.indexOf(p.role)!==-1) { var permlevel = cint(p.permlevel); if(!perm[permlevel]) { perm[permlevel] = {}; diff --git a/frappe/public/js/frappe/model/workflow.js b/frappe/public/js/frappe/model/workflow.js index 1a3c0fec32..c20c687dae 100644 --- a/frappe/public/js/frappe/model/workflow.js +++ b/frappe/public/js/frappe/model/workflow.js @@ -58,7 +58,7 @@ frappe.workflow = { var allow_edit = state ? frappe.workflow.get_document_state(doctype, state).allow_edit : null; - if(has_roles.indexOf(allow_edit)==-1) { + if(roles.indexOf(allow_edit)==-1) { return true; } } diff --git a/frappe/public/js/frappe/roles_editor.js b/frappe/public/js/frappe/roles_editor.js index 5f92f95eae..06e88710a5 100644 --- a/frappe/public/js/frappe/roles_editor.js +++ b/frappe/public/js/frappe/roles_editor.js @@ -79,7 +79,7 @@ frappe.RoleEditor = Class.extend({ var existing_roles_map = {}; var existing_roles_list = []; - $.each((cur_frm.doc.has_roles || []), function(i, user_role) { + $.each((cur_frm.doc.roles || []), function(i, user_role) { existing_roles_map[user_role.role] = user_role.name; existing_roles_list.push(user_role.role); }); @@ -94,12 +94,12 @@ frappe.RoleEditor = Class.extend({ // add new roles that are checked $.each(opts.checked_roles, function(i, role) { if(existing_roles_list.indexOf(role)==-1) { - var user_role = frappe.model.add_child(cur_frm.doc, "UserRole", "has_roles"); + var user_role = frappe.model.add_child(cur_frm.doc, "UserRole", "roles"); user_role.role = role; } }); - refresh_field("has_roles"); + refresh_field("roles"); }, get_roles: function() { var checked_roles = []; diff --git a/frappe/public/js/legacy/globals.js b/frappe/public/js/legacy/globals.js index 0a893642ce..7946fc2cde 100644 --- a/frappe/public/js/legacy/globals.js +++ b/frappe/public/js/legacy/globals.js @@ -27,7 +27,7 @@ var DOWN_ARROW = 40; var user=null; var user=null; var user_defaults=null; -var has_roles=null; +var roles=null; var user_fullname=null; var user_email=null; var user_img = {}; diff --git a/frappe/sessions.py b/frappe/sessions.py index cdfa07b806..d4ae6370c1 100644 --- a/frappe/sessions.py +++ b/frappe/sessions.py @@ -30,7 +30,7 @@ def clear(user=None): def clear_cache(user=None): cache = frappe.cache() - groups = ("bootinfo", "user_recent", "user_roles", "user_doc", "lang", + groups = ("bootinfo", "user_recent", "roles", "user_doc", "lang", "defaults", "user_permissions", "roles", "home_page", "linked_with", "desktop_icons", 'portal_menu_items') diff --git a/frappe/tests/test_data_import.py b/frappe/tests/test_data_import.py index 0d33a8eeab..d7432da483 100644 --- a/frappe/tests/test_data_import.py +++ b/frappe/tests/test_data_import.py @@ -62,7 +62,7 @@ class TestDataImport(unittest.TestCase): user = frappe.get_doc("User", user_email) self.assertTrue(frappe.db.get_value("UserRole", filters={"role": "Blogger", "parent": user_email})) - self.assertTrue(user.get("user_roles")[0].role, "Blogger") + self.assertTrue(user.get("roles")[0].role, "Blogger") # overwrite exporter.get_template("UserRole", "User", all_doctypes="No", with_data="No") @@ -71,8 +71,8 @@ class TestDataImport(unittest.TestCase): importer.upload(content, overwrite=True) user = frappe.get_doc("User", user_email) - self.assertEquals(len(user.get("user_roles")), 1) - self.assertTrue(user.get("user_roles")[0].role, "Website Manager") + self.assertEquals(len(user.get("roles")), 1) + self.assertTrue(user.get("roles")[0].role, "Website Manager") def test_import_with_children(self): exporter.get_template("Event", all_doctypes="Yes", with_data="No") diff --git a/frappe/tests/test_document.py b/frappe/tests/test_document.py index 37cf0a3110..042e0b1ebf 100644 --- a/frappe/tests/test_document.py +++ b/frappe/tests/test_document.py @@ -7,7 +7,7 @@ import frappe, unittest class TestDocument(unittest.TestCase): def test_get_return_empty_list_for_table_field_if_none(self): d = frappe.get_doc({"doctype":"User"}) - self.assertEquals(d.get("user_roles"), []) + self.assertEquals(d.get("roles"), []) def test_load(self): d = frappe.get_doc("DocType", "User") @@ -114,7 +114,7 @@ class TestDocument(unittest.TestCase): "doctype": "User", "email": "test_link_validation@example.com", "first_name": "Link Validation", - "user_roles": [ + "roles": [ { "role": "ABC" } @@ -122,8 +122,8 @@ class TestDocument(unittest.TestCase): }) self.assertRaises(frappe.LinkValidationError, d.insert) - d.user_roles = [] - d.append("user_roles", { + d.roles = [] + d.append("roles", { "role": "System Manager" }) d.insert() diff --git a/frappe/utils/install.py b/frappe/utils/install.py index 4976243a03..9b8b348b92 100644 --- a/frappe/utils/install.py +++ b/frappe/utils/install.py @@ -46,11 +46,11 @@ def install_basic_docs(): install_docs = [ {'doctype':'User', 'name':'Administrator', 'first_name':'Administrator', 'email':'admin@example.com', 'enabled':1, "is_admin": 1, - 'user_roles': [{'role': 'Administrator'}] + 'roles': [{'role': 'Administrator'}] }, {'doctype':'User', 'name':'Guest', 'first_name':'Guest', 'email':'guest@example.com', 'enabled':1, "is_guest": 1, - 'user_roles': [{'role': 'Guest'}] + 'roles': [{'role': 'Guest'}] }, {'doctype': "Role", "role_name": "Report Manager"}, {'doctype': "Workflow State", "workflow_state_name": "Pending",