[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