projects
/
arvados.git
/ blobdiff
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Merge branch 'master' of git.curoverse.com:arvados into 11876-r-sdk
[arvados.git]
/
services
/
api
/
app
/
controllers
/
arvados
/
v1
/
collections_controller.rb
diff --git
a/services/api/app/controllers/arvados/v1/collections_controller.rb
b/services/api/app/controllers/arvados/v1/collections_controller.rb
index cda7d6b7584abc4f803bc9582a765db64ac16bbb..3803d37691132782e927e6e32cbb5fbee80657c7 100644
(file)
--- a/
services/api/app/controllers/arvados/v1/collections_controller.rb
+++ b/
services/api/app/controllers/arvados/v1/collections_controller.rb
@@
-1,8
+1,21
@@
+# Copyright (C) The Arvados Authors. All rights reserved.
+#
+# SPDX-License-Identifier: AGPL-3.0
+
require "arvados/keep"
require "arvados/keep"
+require "trashable"
class Arvados::V1::CollectionsController < ApplicationController
class Arvados::V1::CollectionsController < ApplicationController
- def self.limit_index_columns_read
- ["manifest_text"]
+ include DbCurrentTime
+ include TrashableController
+
+ def self._index_requires_parameters
+ (super rescue {}).
+ merge({
+ include_trash: {
+ type: 'boolean', required: false, description: "Include collections whose is_trashed attribute is true."
+ },
+ })
end
def create
end
def create
@@
-13,6
+26,13
@@
class Arvados::V1::CollectionsController < ApplicationController
super
end
super
end
+ def find_objects_for_index
+ if params[:include_trash] || ['destroy', 'trash', 'untrash'].include?(action_name)
+ @objects = Collection.readable_by(*@read_users, {include_trash: true, query_on: Collection.unscoped})
+ end
+ super
+ end
+
def find_object_by_uuid
if loc = Keep::Locator.parse(params[:id])
loc.strip_hints!
def find_object_by_uuid
if loc = Keep::Locator.parse(params[:id])
loc.strip_hints!
@@
-23,10
+43,10
@@
class Arvados::V1::CollectionsController < ApplicationController
manifest_text: c.signed_manifest_text,
}
end
manifest_text: c.signed_manifest_text,
}
end
+ true
else
super
end
else
super
end
- true
end
def show
end
def show
@@
-39,6
+59,7
@@
class Arvados::V1::CollectionsController < ApplicationController
end
end
end
end
+
def find_collections(visited, sp, &b)
case sp
when ArvadosModel
def find_collections(visited, sp, &b)
case sp
when ArvadosModel
@@
-127,9
+148,9
@@
class Arvados::V1::CollectionsController < ApplicationController
visited[uuid] = job.as_api_response
if direction == :search_up
# Follow upstream collections referenced in the script parameters
visited[uuid] = job.as_api_response
if direction == :search_up
# Follow upstream collections referenced in the script parameters
- find_collections(visited, job) do |hash, uuid|
+ find_collections(visited, job) do |hash,
col_
uuid|
search_edges(visited, hash, :search_up) if hash
search_edges(visited, hash, :search_up) if hash
- search_edges(visited,
uuid, :search_up) if
uuid
+ search_edges(visited,
col_uuid, :search_up) if col_
uuid
end
elsif direction == :search_down
# Follow downstream job output
end
elsif direction == :search_down
# Follow downstream job output
@@
-184,10
+205,10
@@
class Arvados::V1::CollectionsController < ApplicationController
protected
def load_limit_offset_order_params *args
protected
def load_limit_offset_order_params *args
+ super
if action_name == 'index'
# Omit manifest_text and unsigned_manifest_text from index results unless expressly selected.
@select ||= model_class.selectable_attributes - ["manifest_text", "unsigned_manifest_text"]
end
if action_name == 'index'
# Omit manifest_text and unsigned_manifest_text from index results unless expressly selected.
@select ||= model_class.selectable_attributes - ["manifest_text", "unsigned_manifest_text"]
end
- super
end
end
end
end