X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/b0cbc39917df391c0f0e7f31d9d46c5cc9070520..d047c1cb9ceecb6e324adf102e5e38e11fe698e1:/apps/workbench/app/controllers/application_controller.rb diff --git a/apps/workbench/app/controllers/application_controller.rb b/apps/workbench/app/controllers/application_controller.rb index ee3ac4d681..5acf9a7351 100644 --- a/apps/workbench/app/controllers/application_controller.rb +++ b/apps/workbench/app/controllers/application_controller.rb @@ -1,3 +1,7 @@ +# Copyright (C) The Arvados Authors. All rights reserved. +# +# SPDX-License-Identifier: AGPL-3.0 + class ApplicationController < ActionController::Base include ArvadosApiClientHelper include ApplicationHelper @@ -232,11 +236,18 @@ class ApplicationController < ActionController::Base objects = @objects end if objects.respond_to?(:result_offset) and - objects.respond_to?(:result_limit) and - objects.respond_to?(:items_available) + objects.respond_to?(:result_limit) next_offset = objects.result_offset + objects.result_limit - if next_offset < objects.items_available + if objects.respond_to?(:items_available) and (next_offset < objects.items_available) next_offset + elsif @objects.results.size > 0 and (params[:count] == 'none' or + (params[:controller] == 'search' and params[:action] == 'choose')) + last_object_class = @objects.last.class + if params['last_object_class'].nil? or params['last_object_class'] == last_object_class.to_s + next_offset + else + @objects.select{|obj| obj.class == last_object_class}.size + end else nil end @@ -1227,8 +1238,15 @@ class ApplicationController < ActionController::Base @objects_for[obj.name] = obj end else + key_prefix = "request_#{Thread.current.object_id}_#{dataclass.to_s}_" dataclass.where(uuid: uuids).each do |obj| @objects_for[obj.uuid] = obj + if dataclass == Collection + # The collecions#index defaults to "all attributes except manifest_text" + # Hence, this object is not suitable for preloading the find() cache. + else + Rails.cache.write(key_prefix + obj.uuid, obj.as_json) + end end end @objects_for @@ -1243,6 +1261,49 @@ class ApplicationController < ActionController::Base end end + # helper method to get the names of collection files selected + helper_method :selected_collection_files + def selected_collection_files params + link_uuids, coll_ids = params["selection"].partition do |sel_s| + ArvadosBase::resource_class_for_uuid(sel_s) == Link + end + + unless link_uuids.empty? + Link.select([:head_uuid]).where(uuid: link_uuids).each do |link| + if ArvadosBase::resource_class_for_uuid(link.head_uuid) == Collection + coll_ids << link.head_uuid + end + end + end + + uuids = [] + pdhs = [] + source_paths = Hash.new { |hash, key| hash[key] = [] } + coll_ids.each do |coll_id| + if m = CollectionsHelper.match(coll_id) + key = m[1] + m[2] + pdhs << key + source_paths[key] << m[4] + elsif m = CollectionsHelper.match_uuid_with_optional_filepath(coll_id) + key = m[1] + uuids << key + source_paths[key] << m[4] + end + end + + unless pdhs.empty? + Collection.where(portable_data_hash: pdhs.uniq). + select([:uuid, :portable_data_hash]).each do |coll| + unless source_paths[coll.portable_data_hash].empty? + uuids << coll.uuid + source_paths[coll.uuid] = source_paths.delete(coll.portable_data_hash) + end + end + end + + [uuids, source_paths] + end + def wiselinks_layout 'body' end