From e76a00ca471102d1e536737073314e0f36cf086a Mon Sep 17 00:00:00 2001 From: Peter Amstutz Date: Mon, 23 Oct 2017 08:44:15 -0400 Subject: [PATCH] 12125: Don't offer to untrash item if it isn't trash Ensure that it behaves in a reasonable way if user has access to an item via a sideways permission link instead of the usual ownership hierarchy. Arvados-DCO-1.1-Signed-off-by: Peter Amstutz --- apps/workbench/app/views/application/404.html.erb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/workbench/app/views/application/404.html.erb b/apps/workbench/app/views/application/404.html.erb index c09e6349ea..283f5d59fd 100644 --- a/apps/workbench/app/views/application/404.html.erb +++ b/apps/workbench/app/views/application/404.html.erb @@ -53,7 +53,7 @@ SPDX-License-Identifier: AGPL-3.0 %> <% if untrash_object != object %> You must untrash the owner project to access this <%= class_name_h %>. <% end %> - <% if untrash_object.editable? %> + <% if untrash_object.is_trashed and untrash_object.editable? %> <% msg = "Untrash '#{untrash_name}'?" %> <%= link_to({action: 'untrash_items', selection: [untrash_object.uuid], controller: :trash_items}, remote: true, method: :post, title: "Untrash", style: 'cursor: pointer;') do %> -- 2.30.2