5365: add back data-toggle along with link disabled for Log tab.

[arvados.git] / apps / workbench / app / helpers / application_helper.rb

diff --git a/apps/workbench/app/helpers/application_helper.rb b/apps/workbench/app/helpers/application_helper.rb
index 147e5a60bbf1b60ffbb3d17a19916a5b4d65b337..8922f331e099b1eb9b277d1cfd9a52bffe8df499 100644 (file)
--- a/apps/workbench/app/helpers/application_helper.rb
+++ b/apps/workbench/app/helpers/application_helper.rb
@@ -177,19 +177,24 @@ module ApplicationHelper
     end
   end
 
-  def link_to_arvados_object_if_readable(attrvalue, link_text, link_text_if_not_readable, use_friendly_name=false, resource_class=nil)
-    resource_class = resource_class_for_uuid(attrvalue) if !resource_class
+  def link_to_arvados_object_if_readable(attrvalue, link_text_if_not_readable, opts={})
+    resource_class = resource_class_for_uuid(attrvalue)
     if !resource_class
       return link_text_if_not_readable
     end
 
-    readable = resource_class.find?(attrvalue)
-    if readable
-      if use_friendly_name
-        link_to_if_arvados_object attrvalue, friendly_name: true
+    if resource_class.to_s == 'Collection'
+      if CollectionsHelper.match(attrvalue)
+        readable = collection_for_pdh(attrvalue).any?
       else
-        link_to_if_arvados_object attrvalue, link_text: link_text
+        readable = collections_for_object(attrvalue).any?
       end
+    else
+      readable = object_for_dataclass(resource_class, attrvalue)
+    end
+
+    if readable
+      link_to_if_arvados_object attrvalue, opts
     else
       link_text_if_not_readable
     end
@@ -289,7 +294,7 @@ module ApplicationHelper
     end
 
     if not object.andand.attribute_editable?(attr)
-      return link_to_if_arvados_object attrvalue
+      return link_to_arvados_object_if_readable(attrvalue, attrvalue, friendly_name: true)
     end
 
     if dataclass