[ARVADOS] updated: 6b21bd450e8b8ed968a965290e641a75660cd695

git at public.curoverse.com git at public.curoverse.com
Mon May 5 16:55:40 EDT 2014


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

       via  6b21bd450e8b8ed968a965290e641a75660cd695 (commit)
      from  8370640cef5110b65ba6c3a2bf9d7f56541b4c7f (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 6b21bd450e8b8ed968a965290e641a75660cd695
Author: Tom Clegg <tom at curoverse.com>
Date:   Mon May 5 16:55:19 2014 -0400

    Raise PermissionDeniedError even more consistently

diff --git a/services/api/app/models/arvados_model.rb b/services/api/app/models/arvados_model.rb
index 875f1c0..1549291 100644
--- a/services/api/app/models/arvados_model.rb
+++ b/services/api/app/models/arvados_model.rb
@@ -168,7 +168,7 @@ class ArvadosModel < ActiveRecord::Base
   end
 
   def ensure_owner_uuid_is_permitted
-    return false if !current_user
+    raise PermissionDeniedError if !current_user
     self.owner_uuid ||= current_user.uuid
     if self.owner_uuid_changed?
       if current_user.uuid == self.owner_uuid or

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list