[ARVADOS] updated: f049df0d8d69b8f4e235869103be2c814382bcce

git at public.curoverse.com git at public.curoverse.com
Wed May 28 11:01:52 EDT 2014


Summary of changes:
 services/api/app/models/user.rb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

       via  f049df0d8d69b8f4e235869103be2c814382bcce (commit)
      from  122e392ac6d9b36e376937f4dc2f910118b112ac (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit f049df0d8d69b8f4e235869103be2c814382bcce
Author: Ward Vandewege <ward at curoverse.com>
Date:   Wed May 28 11:00:07 2014 -0400

    Do not use symbol keys in the properties of permission links.

diff --git a/services/api/app/models/user.rb b/services/api/app/models/user.rb
index 8743b92..d6dd9dc 100644
--- a/services/api/app/models/user.rb
+++ b/services/api/app/models/user.rb
@@ -338,7 +338,7 @@ class User < ArvadosModel
 
       perm_exists = false
       login_perms.each do |perm|
-        if perm.properties[:username] == repo_name
+        if perm.properties['username'] == repo_name
           perm_exists = true
           break
         end
@@ -349,7 +349,7 @@ class User < ArvadosModel
                                  head_uuid: vm[:uuid],
                                  link_class: 'permission',
                                  name: 'can_login',
-                                 properties: {username: repo_name})
+                                 properties: {'username' => repo_name})
         logger.info { "login permission: " + login_perm[:uuid] }
       else
         login_perm = login_perms.first

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list