8784: Fix test for latest firefox.

[arvados.git] / apps / workbench / app / views / application / _selection_checkbox.html.erb

diff --git a/apps/workbench/app/views/application/_selection_checkbox.html.erb b/apps/workbench/app/views/application/_selection_checkbox.html.erb
index 732c343d269ea9f0efb9a015f99052101e4e1cbe..47e1dfdecff9454175ac6ead87e2766308e7140f 100644 (file)
--- a/apps/workbench/app/views/application/_selection_checkbox.html.erb
+++ b/apps/workbench/app/views/application/_selection_checkbox.html.erb
@@ -1,9 +1,19 @@
-<%if object %>
+<%if object and object.uuid and (object.class.goes_in_projects? or (object.is_a?(Link) and ArvadosBase::resource_class_for_uuid(object.head_uuid).to_s == 'Collection')) %>
+  <% fn = if defined? friendly_name and not friendly_name.nil?
+            friendly_name
+          else
+            link_to_if_arvados_object object, {no_link: true}
+          end
+     %>
+  <% # This 'fn' string may contain embedded HTML which is already marked html_safe.
+     # Since we are putting it into a tag attribute, we need to copy into an
+     # unsafe string so that rails will escape it for us.
+     fn = String.new fn %>
 <%= check_box_tag 'uuids[]', object.uuid, false, {
       :class => 'persistent-selection',
+      :id => object.uuid,
       :friendly_type => object.class.name,
-      :friendly_name => (if defined? friendly_name then friendly_name
-                         else link_to_if_arvados_object object, {:no_link => true} end),
+      :friendly_name => fn,
       :href => "#{url_for controller: object.class.name.tableize, action: 'show', id: object.uuid }",
       :title => "Click to add this item to your selection list"
 } %>