Merge "Remove admin role name 'admin' hardcode in User.is_superuser()"

This commit is contained in:
Jenkins
2014-11-03 13:10:58 +00:00
committed by Gerrit Code Review

View File

@@ -258,7 +258,12 @@ class User(models.AnonymousUser):
Returns ``True`` or ``False``.
"""
return 'admin' in [role['name'].lower() for role in self.roles]
admin_roles = [role.lower() for role in getattr(
settings,
'OPENSTACK_KEYSTONE_ADMIN_ROLES',
['admin'])]
user_roles = [role['name'].lower() for role in self.roles]
return True if set(admin_roles).intersection(user_roles) else False
@property
def authorized_tenants(self):