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

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

Repository: allura
Updated Branches:
  refs/heads/master d4eb6ddae -> 6205700f3


[#7815] 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/6205700f
Tree: http://git-wip-us.apache.org/repos/asf/allura/tree/6205700f
Diff: http://git-wip-us.apache.org/repos/asf/allura/diff/6205700f

Branch: refs/heads/master
Commit: 6205700f3e9c0c0618d490a5634c78b71f7102e9
Parents: d4eb6dd
Author: Igor Bondarenko <je...@gmail.com>
Authored: Tue Jan 6 13:21:58 2015 +0000
Committer: Dave Brondsema <db...@slashdotmedia.com>
Committed: Tue Jan 6 18:56:38 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/6205700f/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):