[ARVADOS] updated: 546687a7c95109a57efdf7cecbc3f795bc3e72f1

git at public.curoverse.com git at public.curoverse.com
Mon May 5 11:21:48 EDT 2014


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

       via  546687a7c95109a57efdf7cecbc3f795bc3e72f1 (commit)
      from  76e20e694fbe708e17e57a2e0b1c36aca6e8d7d0 (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 546687a7c95109a57efdf7cecbc3f795bc3e72f1
Author: Tom Clegg <tom at curoverse.com>
Date:   Mon May 5 06:08:30 2014 -0400

    Behave nicer if supplied owner_uuid is not a valid uuid at all.

diff --git a/services/api/app/models/arvados_model.rb b/services/api/app/models/arvados_model.rb
index ae0e490..e495f9f 100644
--- a/services/api/app/models/arvados_model.rb
+++ b/services/api/app/models/arvados_model.rb
@@ -144,6 +144,8 @@ class ArvadosModel < ActiveRecord::Base
           if x == uuid
             # Test for cycles with the new version, not the DB contents
             x = owner_uuid
+          elsif !owner_class.respond_to? :find_by_uuid
+            raise ActiveRecord::RecordNotFound.new
           else
             x = owner_class.find_by_uuid(x).owner_uuid
           end

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list