Merge branch 'master' into 5675-project-subprojects-in-anonymous-view

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

diff --git a/apps/workbench/app/views/application/_content.html.erb b/apps/workbench/app/views/application/_content.html.erb
index 1f7f2d29c50d7e4538872fcdf28d6df4c2f484dd..5e6211f4623fbedfcfe72539fc1beb4c56b3540c 100644 (file)
--- a/apps/workbench/app/views/application/_content.html.erb
+++ b/apps/workbench/app/views/application/_content.html.erb
@@ -8,14 +8,16 @@
 
       <% data_toggle = "tab" %>
       <% tab_tooltip = "" %>
+      <% link_disabled = "" %>
 
-      <% if !current_user and pane_name == "Log"
+      <% if (pane_name == "Log") and !(ArvadosBase.find(@object.owner_uuid).writable_by.include?(current_user.andand.uuid) rescue nil)
           if controller.model_class.to_s == 'Job'
             if @object.log and !@object.log.empty?
               logCollection = Collection.find? @object.log
               if !logCollection
                 data_toggle = "disabled"
                 tab_tooltip = "Log data is not available"
+                link_disabled = "disabled"
               end
             end
           elsif controller.model_class.to_s == 'PipelineInstance'
@@ -23,12 +25,13 @@
             if stderr_log_history(log_uuids, 1).empty?
               data_toggle = "disabled"
               tab_tooltip = "Log data is not available"
+              link_disabled = "disabled"
             end
           end
         end
       %>
 
-      <li class="<%= 'active' if i==0 %>" data-toggle="tooltip" data-placement="top" title="<%=tab_tooltip%>">
+      <li class="<%= 'active' if i==0 %> <%= link_disabled %>" data-toggle="tooltip" data-placement="top" title="<%=tab_tooltip%>">
         <a href="#<%= pane_name %>"
            id="<%= pane_name %>-tab"
            data-toggle="<%= data_toggle %>"