X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/0b102fac0e8d2a7d46d088b1bd8f7b27b325dd2a..060d38d627bd1e51dd2b3c6e7de9af6aa7d7b6f3:/services/api/app/controllers/arvados/v1/groups_controller.rb diff --git a/services/api/app/controllers/arvados/v1/groups_controller.rb b/services/api/app/controllers/arvados/v1/groups_controller.rb index 2356f2eece..75ef095b71 100644 --- a/services/api/app/controllers/arvados/v1/groups_controller.rb +++ b/services/api/app/controllers/arvados/v1/groups_controller.rb @@ -1,3 +1,7 @@ +# Copyright (C) The Arvados Authors. All rights reserved. +# +# SPDX-License-Identifier: AGPL-3.0 + class Arvados::V1::GroupsController < ApplicationController def self._contents_requires_parameters @@ -6,6 +10,9 @@ class Arvados::V1::GroupsController < ApplicationController uuid: { type: 'string', required: false, default: nil }, + recursive: { + type: 'boolean', required: false, description: 'Include contents from child groups recursively.' + }, }) params.delete(:select) params @@ -61,35 +68,111 @@ class Arvados::V1::GroupsController < ApplicationController request_orders = @orders.clone @orders = [] - [Group, - Job, PipelineInstance, PipelineTemplate, + request_filters = @filters + + klasses = [Group, + Job, PipelineInstance, PipelineTemplate, ContainerRequest, Workflow, Collection, - Human, Specimen, Trait].each do |klass| + Human, Specimen, Trait] + + table_names = Hash[klasses.collect { |k| [k, k.table_name] }] + + disabled_methods = Rails.configuration.disable_api_methods + avail_klasses = table_names.select{|k, t| !disabled_methods.include?(t+'.index')} + klasses = avail_klasses.keys + + request_filters.each do |col, op, val| + if col.index('.') && !table_names.values.include?(col.split('.', 2)[0]) + raise ArgumentError.new("Invalid attribute '#{col}' in filter") + end + end + + wanted_klasses = [] + request_filters.each do |col,op,val| + if op == 'is_a' + (val.is_a?(Array) ? val : [val]).each do |type| + type = type.split('#')[-1] + type[0] = type[0].capitalize + wanted_klasses << type + end + end + end + + filter_by_owner = {} + if @object + if params['recursive'] + filter_by_owner[:owner_uuid] = [@object.uuid] + @object.descendant_project_uuids + else + filter_by_owner[:owner_uuid] = @object.uuid + end + end + + seen_last_class = false + klasses.each do |klass| + @offset = 0 if seen_last_class # reset offset for the new next type being processed + + # if current klass is same as params['last_object_class'], mark that fact + seen_last_class = true if((params['count'].andand.==('none')) and + (params['last_object_class'].nil? or + params['last_object_class'].empty? or + params['last_object_class'] == klass.to_s)) + + # if klasses are specified, skip all other klass types + next if wanted_klasses.any? and !wanted_klasses.include?(klass.to_s) + + # don't reprocess klass types that were already seen + next if params['count'] == 'none' and !seen_last_class + + # don't process rest of object types if we already have needed number of objects + break if params['count'] == 'none' and all_objects.size >= limit_all + # If the currently requested orders specifically match the # table_name for the current klass, apply that order. # Otherwise, order by recency. request_order = request_orders.andand.find { |r| r =~ /^#{klass.table_name}\./i } || - "created_at desc" + klass.default_orders.join(", ") @select = nil - where_conds = {} - where_conds[:owner_uuid] = @object.uuid if @object + where_conds = filter_by_owner if klass == Collection @select = klass.selectable_attributes - ["manifest_text"] elsif klass == Group - where_conds[:group_class] = "project" + where_conds = where_conds.merge(group_class: "project") end - @objects = klass.readable_by(*@read_users). - order(request_order).where(where_conds) - @limit = limit_all - all_objects.count + @filters = request_filters.map do |col, op, val| + if !col.index('.') + [col, op, val] + elsif (col = col.split('.', 2))[0] == klass.table_name + [col[1], op, val] + else + nil + end + end.compact + + if klass == Collection and params[:include_trash] + @objects = klass.unscoped.readable_by(*@read_users). + order(request_order).where(where_conds) + else + @objects = klass.readable_by(*@read_users). + order(request_order).where(where_conds) + end + klass_limit = limit_all - all_objects.count + @limit = klass_limit apply_where_limit_order_params klass - klass_object_list = object_list + klass_object_list = object_list(model_class: klass) klass_items_available = klass_object_list[:items_available] || 0 @items_available += klass_items_available @offset = [@offset - klass_items_available, 0].max all_objects += klass_object_list[:items] + + if klass_object_list[:limit] < klass_limit + # object_list() had to reduce @limit to comply with + # max_index_database_read. From now on, we'll do all queries + # with limit=0 and just accumulate items_available. + limit_all = all_objects.count + end end @objects = all_objects