[ARVADOS] updated: c0d251c444855e8268e2e7079e8b3115d96a1cff

Git user git at public.curoverse.com
Fri Mar 31 14:01:41 EDT 2017


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

       via  c0d251c444855e8268e2e7079e8b3115d96a1cff (commit)
      from  02a997da87252380eac0a653edce2442c2b8b518 (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 c0d251c444855e8268e2e7079e8b3115d96a1cff
Author: radhika <radhika at curoverse.com>
Date:   Fri Mar 31 14:00:35 2017 -0400

    11368: optimize coll.exist?(new_file_path) invocation

diff --git a/apps/workbench/app/controllers/collections_controller.rb b/apps/workbench/app/controllers/collections_controller.rb
index 75553b6..dc9ed43 100644
--- a/apps/workbench/app/controllers/collections_controller.rb
+++ b/apps/workbench/app/controllers/collections_controller.rb
@@ -327,8 +327,7 @@ class CollectionsController < ApplicationController
 
       arv_coll = Arv::Collection.new(@object.manifest_text)
 
-      dst_exists = arv_coll.exist? new_file_path
-      if dst_exists
+      if arv_coll.exist?(new_file_path)
         @errors = 'Duplicate file path. Please use a different name.'
         self.render_error status: 422
       else

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list