[ARVADOS] updated: e79e5c258fe6ec542c3f2bb3c4230a5de2812b25

git at public.curoverse.com git at public.curoverse.com
Fri Oct 3 19:45:30 EDT 2014


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

       via  e79e5c258fe6ec542c3f2bb3c4230a5de2812b25 (commit)
      from  49c507b0bb60e35b951e2f4946c28d7c6729ce88 (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 e79e5c258fe6ec542c3f2bb3c4230a5de2812b25
Author: Tom Clegg <tom at curoverse.com>
Date:   Fri Oct 3 19:44:56 2014 -0400

    3687: OK here just means this loop is OK, not everything is OK!

diff --git a/services/api/app/models/user.rb b/services/api/app/models/user.rb
index 2738e60..f283ea3 100644
--- a/services/api/app/models/user.rb
+++ b/services/api/app/models/user.rb
@@ -88,7 +88,7 @@ class User < ArvadosModel
                       name: sufficient_perms,
                       tail_uuid: groups_i_can(action) + [self.uuid],
                       head_uuid: target_uuid).any?
-          return true
+          next
         end
       end
       return false

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list