[ARVADOS] updated: f842b72b9e2bb50a68ba388922c55f54ad0399ae

git at public.curoverse.com git at public.curoverse.com
Mon Feb 16 13:02:23 EST 2015


Summary of changes:
 apps/workbench/app/controllers/collections_controller.rb | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

       via  f842b72b9e2bb50a68ba388922c55f54ad0399ae (commit)
      from  23c1bc62ea3644636c0edce8ccee39f2094db190 (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 f842b72b9e2bb50a68ba388922c55f54ad0399ae
Author: Radhika Chippada <radhika at curoverse.com>
Date:   Mon Feb 16 12:42:17 2015 -0500

    5096: more efficient update method override.

diff --git a/apps/workbench/app/controllers/collections_controller.rb b/apps/workbench/app/controllers/collections_controller.rb
index d9bf3a9..e883017 100644
--- a/apps/workbench/app/controllers/collections_controller.rb
+++ b/apps/workbench/app/controllers/collections_controller.rb
@@ -271,14 +271,8 @@ class CollectionsController < ApplicationController
     if @updates && (@updates.keys - ["name", "description"]).empty?
       # exclude manifest_text since only name or description is being updated
       @object.manifest_text = nil
-      if @object.update_attributes @updates
-        show
-      else
-        self.render_error status: 422
-      end
-    else
-      super
     end
+    super
   end
 
   protected

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list