You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@allura.apache.org by je...@apache.org on 2015/01/06 15:41:31 UTC

allura git commit: [#7737] ticket:702 Fix possible KeyError

Repository: allura
Updated Branches:
  refs/heads/ib/7737 [created] fa279e04b


[#7737] ticket:702 Fix possible KeyError


Project: http://git-wip-us.apache.org/repos/asf/allura/repo
Commit: http://git-wip-us.apache.org/repos/asf/allura/commit/fa279e04
Tree: http://git-wip-us.apache.org/repos/asf/allura/tree/fa279e04
Diff: http://git-wip-us.apache.org/repos/asf/allura/diff/fa279e04

Branch: refs/heads/ib/7737
Commit: fa279e04b95607e3a9ffd758544dfe13b2b82e03
Parents: fc812f4
Author: Igor Bondarenko <je...@gmail.com>
Authored: Tue Jan 6 13:21:58 2015 +0000
Committer: Igor Bondarenko <je...@gmail.com>
Committed: Tue Jan 6 13:21:58 2015 +0000

----------------------------------------------------------------------
 Allura/allura/lib/security.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/allura/blob/fa279e04/Allura/allura/lib/security.py
----------------------------------------------------------------------
diff --git a/Allura/allura/lib/security.py b/Allura/allura/lib/security.py
index ad7fd2a..b26a23c 100644
--- a/Allura/allura/lib/security.py
+++ b/Allura/allura/lib/security.py
@@ -147,7 +147,7 @@ class Credentials(object):
 
     def users_with_named_role(self, project_id, name):
         """ returns in sorted order """
-        role = RoleCache(self, [r for r in self.project_roles(project_id) if r['name'] == name])
+        role = RoleCache(self, [r for r in self.project_roles(project_id) if r.get('name') == name])
         return sorted(role.users_that_reach, key=lambda u: u.username)
 
     def userids_with_named_role(self, project_id, name):