3960: Make editable? return false or true, never nil.
authorTom Clegg <tom@curoverse.com>
Tue, 23 Sep 2014 18:32:09 +0000 (14:32 -0400)
committerTom Clegg <tom@curoverse.com>
Tue, 23 Sep 2014 18:32:09 +0000 (14:32 -0400)
apps/workbench/app/models/arvados_base.rb

index 7bc043cc9e2ccc7fbc79a130bf7ee2d83940815b..b691e6d2bafefaf42c13b9e27711de76f7404cd9 100644 (file)
@@ -329,7 +329,7 @@ class ArvadosBase < ActiveRecord::Base
      (current_user.is_admin or
       current_user.uuid == self.owner_uuid or
       new_record? or
-      (writable_by.include? current_user.uuid rescue false)))
+      (writable_by.include? current_user.uuid rescue false))) or false
   end
 
   def attribute_editable?(attr, ever=nil)