projects
/
arvados.git
/ blobdiff
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
6057: if /projects/public is accessed when anonymous config is not enabled, show...
[arvados.git]
/
apps
/
workbench
/
app
/
controllers
/
projects_controller.rb
diff --git
a/apps/workbench/app/controllers/projects_controller.rb
b/apps/workbench/app/controllers/projects_controller.rb
index df454f72afcf3e24948d67ea00e7f47957a0a720..c4600f2c0f7627a88fa6e8235d04077678afafda 100644
(file)
--- a/
apps/workbench/app/controllers/projects_controller.rb
+++ b/
apps/workbench/app/controllers/projects_controller.rb
@@
-1,5
+1,9
@@
class ProjectsController < ApplicationController
class ProjectsController < ApplicationController
- before_filter :set_share_links, if: -> { defined? @object }
+ before_filter :set_share_links, if: -> { defined? @object and @object}
+ skip_around_filter :require_thread_api_token, if: proc { |ctrl|
+ Rails.configuration.anonymous_user_token and
+ %w(show tab_counts public).include? ctrl.action_name
+ }
def model_class
Group
def model_class
Group
@@
-38,27
+42,35
@@
class ProjectsController < ApplicationController
# us to tell the interface to get counts for each pane (using :filters).
# It also seems to me that something like these could be used to configure the contents of the panes.
def show_pane_list
# us to tell the interface to get counts for each pane (using :filters).
# It also seems to me that something like these could be used to configure the contents of the panes.
def show_pane_list
- pane_list = [
+ pane_list = []
+ if @object.uuid != current_user.andand.uuid
+ pane_list << 'Description'
+ end
+ pane_list <<
{
:name => 'Data_collections',
:filters => [%w(uuid is_a arvados#collection)]
{
:name => 'Data_collections',
:filters => [%w(uuid is_a arvados#collection)]
- },
+ }
+ pane_list <<
{
:name => 'Jobs_and_pipelines',
:filters => [%w(uuid is_a) + [%w(arvados#job arvados#pipelineInstance)]]
{
:name => 'Jobs_and_pipelines',
:filters => [%w(uuid is_a) + [%w(arvados#job arvados#pipelineInstance)]]
- },
+ }
+ pane_list <<
{
:name => 'Pipeline_templates',
:filters => [%w(uuid is_a arvados#pipelineTemplate)]
{
:name => 'Pipeline_templates',
:filters => [%w(uuid is_a arvados#pipelineTemplate)]
- },
+ }
+ pane_list <<
{
:name => 'Subprojects',
:filters => [%w(uuid is_a arvados#group)]
{
:name => 'Subprojects',
:filters => [%w(uuid is_a arvados#group)]
- },
- { :name => 'Other_objects',
- :filters => [%w(uuid is_a) + [%w(arvados#human arvados#specimen arvados#trait)]]
}
}
- ]
+ pane_list <<
+ {
+ :name => 'Other_objects',
+ :filters => [%w(uuid is_a) + [%w(arvados#human arvados#specimen arvados#trait)]]
+ } if current_user
pane_list << { :name => 'Sharing',
:count => @share_links.count } if @user_is_manager
pane_list << { :name => 'Advanced' }
pane_list << { :name => 'Sharing',
:count => @share_links.count } if @user_is_manager
pane_list << { :name => 'Advanced' }
@@
-124,7
+136,7
@@
class ProjectsController < ApplicationController
item.update_attributes owner_uuid: current_user.uuid
@removed_uuids << item.uuid
rescue ArvadosApiClient::ApiErrorResponseException => e
item.update_attributes owner_uuid: current_user.uuid
@removed_uuids << item.uuid
rescue ArvadosApiClient::ApiErrorResponseException => e
- if e.message.include? '
collection
_owner_uuid_name_unique'
+ if e.message.include? '_owner_uuid_name_unique'
rename_to = item.name + ' removed from ' +
(@object.name ? @object.name : @object.uuid) +
' at ' + Time.now.to_s
rename_to = item.name + ' removed from ' +
(@object.name ? @object.name : @object.uuid) +
' at ' + Time.now.to_s
@@
-148,7
+160,7
@@
class ProjectsController < ApplicationController
object.destroy
end
end
object.destroy
end
end
- while (objects = @object.contents
(include_linked: false)
).any?
+ while (objects = @object.contents).any?
objects.each do |object|
object.update_attributes! owner_uuid: current_user.uuid
end
objects.each do |object|
object.update_attributes! owner_uuid: current_user.uuid
end
@@
-189,7
+201,6
@@
class ProjectsController < ApplicationController
(val.is_a?(Array) ? val : [val]).each do |type|
objects = @object.contents(order: @order,
limit: @limit,
(val.is_a?(Array) ? val : [val]).each do |type|
objects = @object.contents(order: @order,
limit: @limit,
- include_linked: true,
filters: (@filters - kind_filters + [['uuid', 'is_a', type]]),
)
objects.each do |object|
filters: (@filters - kind_filters + [['uuid', 'is_a', type]]),
)
objects.each do |object|
@@
-227,7
+238,6
@@
class ProjectsController < ApplicationController
else
@objects = @object.contents(order: @order,
limit: @limit,
else
@objects = @object.contents(order: @order,
limit: @limit,
- include_linked: true,
filters: @filters,
offset: @offset)
@next_page_href = next_page_href(partial: :contents_rows,
filters: @filters,
offset: @offset)
@next_page_href = next_page_href(partial: :contents_rows,
@@
-297,4
+307,11
@@
class ProjectsController < ApplicationController
end
objects_and_names
end
end
objects_and_names
end
+
+ def public # Yes 'public' is the name of the action for public projects
+ return render_not_found if not Rails.configuration.anonymous_user_token
+ @objects = using_specific_api_token Rails.configuration.anonymous_user_token do
+ Group.where(group_class: 'project').order("updated_at DESC")
+ end
+ end
end
end