From 71d9ea52ae12dc80d7062657fc5c059a128106d7 Mon Sep 17 00:00:00 2001
From: Radhika Chippada <radhika@curoverse.com>
Date: Sat, 31 Jan 2015 13:06:25 -0500
Subject: [PATCH] 2659: when an anonymous user is accessing public data, offer
 "Log in" button instead of "Log out" button.

---
 apps/workbench/app/views/layouts/body.html.erb           | 6 +++++-
 apps/workbench/test/integration/anonymous_access_test.rb | 4 ++--
 2 files changed, 7 insertions(+), 3 deletions(-)

diff --git a/apps/workbench/app/views/layouts/body.html.erb b/apps/workbench/app/views/layouts/body.html.erb
index f4757ded6f..f4228edd5b 100644
--- a/apps/workbench/app/views/layouts/body.html.erb
+++ b/apps/workbench/app/views/layouts/body.html.erb
@@ -53,7 +53,11 @@
                   <li role="menuitem"><a href="/users/<%=current_user.uuid%>/profile" role="menuitem"><i class="fa fa-key fa-fw"></i> Manage profile</a></li>
                 <% end %>
                 <% end %>
-                <li role="menuitem"><a href="<%= logout_path %>" role="menuitem"><i class="fa fa-sign-out fa-fw"></i> Log out</a></li>
+                <% if !is_anonymous %>
+                  <li role="menuitem"><a href="<%= logout_path %>" role="menuitem"><i class="fa fa-sign-out fa-fw"></i> Log out</a></li>
+                <% else %>
+                  <li><a href="<%= arvados_api_client.arvados_login_url(return_to: root_url) %>">Log in</a></li>
+                <% end %>
                 <% if user_notifications.any? %>
                   <li role="presentation" class="divider"></li>
                   <% user_notifications.each_with_index do |n, i| %>
diff --git a/apps/workbench/test/integration/anonymous_access_test.rb b/apps/workbench/test/integration/anonymous_access_test.rb
index 87bbe84d49..f5753a5548 100644
--- a/apps/workbench/test/integration/anonymous_access_test.rb
+++ b/apps/workbench/test/integration/anonymous_access_test.rb
@@ -38,8 +38,8 @@ class AnonymousAccessTest < ActionDispatch::IntegrationTest
         assert_selector 'a', "#{anonymous_user['email']}"
         find('a', text: "#{anonymous_user['email']}").click
         within('.dropdown-menu') do
-          assert_selector 'a', text: 'Log out'
-          #assert_selector 'a', 'Log in'
+          assert_selector 'a', text: 'Log in'
+          assert_no_selector 'a', text: 'Log out'
         end
       end
     end
-- 
2.30.2