From e612a2d29a3a2df913e75df48a743c0d7324b5ce Mon Sep 17 00:00:00 2001
From: Jeff Ohrstrom <johrstrom@osc.edu>
Date: Wed, 4 Aug 2021 13:03:16 -0400
Subject: [PATCH] fix shell dropdown menu to correctly redirect to the shell
 app

---
 apps/dashboard/app/views/files/_shell_dropdown.html.erb | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/apps/dashboard/app/views/files/_shell_dropdown.html.erb b/apps/dashboard/app/views/files/_shell_dropdown.html.erb
index aa65ec8a98..1ca35ff02b 100644
--- a/apps/dashboard/app/views/files/_shell_dropdown.html.erb
+++ b/apps/dashboard/app/views/files/_shell_dropdown.html.erb
@@ -6,22 +6,19 @@
   <% if Configuration.login_clusters.count > 0 %>
     <button type="button" class="btn btn-sm btn-outline-dark dropdown-toggle dropdown-toggle-split" data-toggle="dropdown" aria-haspopup="true" aria-expanded="false"></button>
     <ul class="dropdown-menu" x-placement="right-start" id="shell-dropdown-items">
-      <% Configuration.login_clusters.each{ |cluster| %>
+      <% Configuration.login_clusters.each do |cluster| %>
         <%=
-          # <li>
-          #   <a href="" title="" target="_blank" class="dropdown-item"></a>
-          # </li>
           tag.li(
             tag.a(
               "#{cluster.metadata.title || cluster.id.to_s.titleize}",
-              href: OodAppkit::Urls::Shell.new(base_url: root_path).url(host: cluster.login.host, path: @path.to_s),
+              href: OodAppkit.shell.url(host: cluster.login.host, path: @path.to_s),
               title: "#{cluster.metadata.title || cluster.id.to_s.titleize}",
               target: '_blank',
               class: 'dropdown-item'
             )
           )
         %>
-      <% } %>
+      <% end %>
     </ul>
   <% end %>
 </div>