X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/1e2ace543b5614023e30f77d85c19da954cbf1cb..0561bd0c3c07257fd58ded6c7cfa5feeae97af57:/apps/workbench/app/controllers/collections_controller.rb diff --git a/apps/workbench/app/controllers/collections_controller.rb b/apps/workbench/app/controllers/collections_controller.rb index 0806a0a429..99065947b8 100644 --- a/apps/workbench/app/controllers/collections_controller.rb +++ b/apps/workbench/app/controllers/collections_controller.rb @@ -1,6 +1,10 @@ +# Copyright (C) The Arvados Authors. All rights reserved. +# +# SPDX-License-Identifier: AGPL-3.0 + require "arvados/keep" +require "arvados/collection" require "uri" -require "cgi" class CollectionsController < ApplicationController include ActionController::Live @@ -20,7 +24,7 @@ class CollectionsController < ApplicationController RELATION_LIMIT = 5 def show_pane_list - panes = %w(Files Upload Provenance_graph Used_by Advanced) + panes = %w(Files Upload Tags Provenance_graph Used_by Advanced) panes = panes - %w(Upload) unless (@object.editable? rescue false) panes end @@ -221,7 +225,7 @@ class CollectionsController < ApplicationController if params["tab_pane"] == "Provenance_graph" @prov_svg = ProvenanceHelper::create_provenance_graph(@object.provenance, "provenance_svg", {:request => request, - :direction => :bottom_up, + :direction => :top_down, :combine_jobs => :script_only}) rescue nil end @@ -240,12 +244,15 @@ class CollectionsController < ApplicationController render 'hash_matches' return else - jobs_with = lambda do |conds| - Job.limit(RELATION_LIMIT).where(conds) - .results.sort_by { |j| j.finished_at || j.created_at } + if Job.api_exists?(:index) + jobs_with = lambda do |conds| + Job.limit(RELATION_LIMIT).where(conds) + .results.sort_by { |j| j.finished_at || j.created_at } + end + @output_of = jobs_with.call(output: @object.portable_data_hash) + @log_of = jobs_with.call(log: @object.portable_data_hash) end - @output_of = jobs_with.call(output: @object.portable_data_hash) - @log_of = jobs_with.call(log: @object.portable_data_hash) + @project_links = Link.limit(RELATION_LIMIT).order("modified_at DESC") .where(head_uuid: @object.uuid, link_class: 'name').results project_hash = Group.where(uuid: @project_links.map(&:tail_uuid)).to_hash @@ -254,21 +261,14 @@ class CollectionsController < ApplicationController @permissions = Link.limit(RELATION_LIMIT).order("modified_at DESC") .where(head_uuid: @object.uuid, link_class: 'permission', name: 'can_read').results - @logs = Log.limit(RELATION_LIMIT).order("created_at DESC") - .select(%w(uuid event_type object_uuid event_at summary)) - .where(object_uuid: @object.uuid).results - @is_persistent = Link.limit(1) - .where(head_uuid: @object.uuid, tail_uuid: current_user.uuid, - link_class: 'resources', name: 'wants') - .results.any? @search_sharing = search_scopes if params["tab_pane"] == "Used_by" @used_by_svg = ProvenanceHelper::create_provenance_graph(@object.used_by, "used_by_svg", {:request => request, - :direction => :top_down, - :combine_jobs => :script_only, - :pdata_only => true}) rescue nil + :direction => :top_down, + :combine_jobs => :script_only, + :pdata_only => true}) rescue nil end end end @@ -298,6 +298,81 @@ class CollectionsController < ApplicationController sharing_popup end + def remove_selected_files + uuids, source_paths = selected_collection_files params + + arv_coll = Arv::Collection.new(@object.manifest_text) + source_paths[uuids[0]].each do |p| + arv_coll.rm "."+p + end + + if @object.update_attributes manifest_text: arv_coll.manifest_text + show + else + self.render_error status: 422 + end + end + + def update + updated_attr = params[:collection].each.select {|a| a[0].andand.start_with? 'rename-file-path:'} + + if updated_attr.size > 0 + # Is it file rename? + file_path = updated_attr[0][0].split('rename-file-path:')[-1] + + new_file_path = updated_attr[0][1] + if new_file_path.start_with?('./') + # looks good + elsif new_file_path.start_with?('/') + new_file_path = '.' + new_file_path + else + new_file_path = './' + new_file_path + end + + arv_coll = Arv::Collection.new(@object.manifest_text) + + if arv_coll.exist?(new_file_path) + @errors = 'Duplicate file path. Please use a different name.' + self.render_error status: 422 + else + arv_coll.rename "./"+file_path, new_file_path + + if @object.update_attributes manifest_text: arv_coll.manifest_text + show + else + self.render_error status: 422 + end + end + else + # Not a file rename; use default + super + end + end + + def tags + render + end + + def save_tags + tags_param = params['tag_data'] + if tags_param + if tags_param.is_a?(String) && tags_param == "empty" + tags = {} + else + tags = tags_param + end + end + + if tags + if @object.update_attributes properties: tags + @saved_tags = true + render + else + self.render_error status: 422 + end + end + end + protected def find_usable_token(token_list) @@ -340,7 +415,7 @@ class CollectionsController < ApplicationController # Prefer the attachment-only-host when we want an attachment # (and when there is no preview link configured) tmpl = Rails.configuration.keep_web_download_url - else + elsif not Rails.configuration.trust_all_content check_uri = URI.parse(tmpl % fmt) if opts[:query_token] and not check_uri.host.start_with?(munged_id + "--") and @@ -369,9 +444,9 @@ class CollectionsController < ApplicationController uri.path += 't=' + opts[:path_token] + '/' end uri.path += '_/' - uri.path += CGI::escape(file) + uri.path += URI.escape(file) - query = CGI::parse(uri.query || '') + query = Hash[URI.decode_www_form(uri.query || '')] { query_token: 'api_token', disposition: 'disposition' }.each do |opt, param| if opts.include? opt @@ -379,7 +454,7 @@ class CollectionsController < ApplicationController end end unless query.empty? - uri.query = query.to_query + uri.query = URI.encode_www_form(query) end uri.to_s