X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/b7ab1e8793ea06c5b778ce2249b29f4ab1607c58..35ade8a042094a27e2ca5cfd5e9754aa3513410c:/apps/workbench/app/controllers/actions_controller.rb diff --git a/apps/workbench/app/controllers/actions_controller.rb b/apps/workbench/app/controllers/actions_controller.rb index 898f6e0d02..59dcbb92bb 100644 --- a/apps/workbench/app/controllers/actions_controller.rb +++ b/apps/workbench/app/controllers/actions_controller.rb @@ -150,7 +150,7 @@ class ActionsController < ApplicationController files.each do |m| mt = chash[m[1]+m[2]].andand.manifest_text if not m[4].nil? and m[4].size > 1 - combined += arv_normalize mt, '--extract', m[4][1..-1] + combined += arv_normalize mt, '--extract', ".#{m[4]}" else combined += mt end @@ -161,12 +161,13 @@ class ActionsController < ApplicationController newc.name = newc.name || "Collection created at #{Time.now.localtime}" # set owner_uuid to current project, provided it is writable - current_project = nil + current_project_writable = false action_data = JSON.parse(params['action_data']) if params['action_data'] if action_data && action_data['current_project_uuid'] - current_project = Group.find(action_data['current_project_uuid']) - if (current_project.andand.writable_by.include?(current_user.uuid) rescue nil) + current_project = Group.find(action_data['current_project_uuid']) rescue nil + if (current_project && current_project.writable_by.andand.include?(current_user.uuid)) newc.owner_uuid = action_data['current_project_uuid'] + current_project_writable = true end end @@ -182,8 +183,9 @@ class ActionsController < ApplicationController l.save! end - msg = current_project ? "Created new collection in the project #{current_project.name}." : - "Created new collection in your Home project." + msg = current_project_writable ? + "Created new collection in the project #{current_project.name}." : + "Created new collection in your Home project." redirect_to newc, flash: {'message' => msg} end