Commit 7371b1c6 authored by Ken Manheimer's avatar Ken Manheimer

manage_addLocalRoles(): Ensure local_roles is a list, since we may be

doing a .append().  (XXX This is necessary since
manage_setLocalRoles() accepts whatever sequence the user passes in -
which should be fixed...)
parent 267d11d1
......@@ -84,7 +84,7 @@
##############################################################################
"""Access control support"""
__version__='$Revision: 1.40 $'[11:-2]
__version__='$Revision: 1.41 $'[11:-2]
from Globals import HTMLFile, MessageDialog, Dictionary
......@@ -375,7 +375,7 @@ class RoleManager(ExtensionClass.Base, PermissionMapping.RoleManager):
if not roles:
raise ValueError, 'One or more roles must be given!'
dict=self.__ac_local_roles__ or {}
local_roles = dict.get(userid, [])
local_roles = list(dict.get(userid, []))
for r in roles:
if r not in local_roles:
local_roles.append(r)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment