From f04ee9ab0380cc5bbb09e7698f4e103cbcaae3d8 Mon Sep 17 00:00:00 2001
From: Fabrice Bellingard <fabrice.bellingard@sonarsource.com>
Date: Mon, 3 Dec 2012 16:21:33 +0100
Subject: [PATCH] SONAR-3972 Move the "Manage dashboards" link in the left side
 bar

---
 .../webapp/WEB-INF/app/views/dashboard/_header.html.erb  | 9 +++------
 .../webapp/WEB-INF/app/views/layouts/_layout.html.erb    | 6 ++++++
 2 files changed, 9 insertions(+), 6 deletions(-)

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_header.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_header.html.erb
index 682fa9162b0..474bf68a477 100644
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_header.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/dashboard/_header.html.erb
@@ -1,15 +1,12 @@
 <% if logged_in? || @snapshot %>
 <div class="line-block">
-  <% if logged_in? %>
+  <% if logged_in? && @dashboard.editable_by?(current_user) %>
     <ul class="operations">
       <% if back %>
-        <li><%= link_to message('dashboard.back_to_dashboard'), dashboard_action(:index) -%></li>
+        <li class="last"><%= link_to message('dashboard.back_to_dashboard'), dashboard_action(:index) -%></li>
       <% else %>
-        <% if @dashboard.editable_by?(current_user) %>
-          <li><%= link_to message('dashboard.configure_widgets'), dashboard_action(:configure) -%></li>
-        <% end %>
+        <li class="last"><%= link_to message('dashboard.configure_widgets'), dashboard_action(:configure) -%></li>
       <% end %>
-      <li class="last"><%= link_to message('dashboard.manage_dashboards'), {:controller => :dashboards, :action => :index, :resource => (@resource.id if @resource) } -%></li>
     </ul>
   <% end %>
 
diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/layouts/_layout.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/layouts/_layout.html.erb
index 7829ef5476e..f773bc5031e 100644
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/layouts/_layout.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/layouts/_layout.html.erb
@@ -51,6 +51,8 @@
                 <a href="<%= ApplicationController.root_context -%>/dashboard/?did=<%= active_dashboard.dashboard_id -%>"><%= active_dashboard.dashboard.name(true) -%></a>
               </li>
             <% end %>
+            <li class="<%= 'active' if request.request_uri.include?('/dashboards/index') -%>">
+              <a href="<%= ApplicationController.root_context -%>/dashboards/index"><%= message('dashboard.manage_dashboards') -%></a></li>
 
             <% controller.java_facade.getPages(Navigation::SECTION_HOME, nil, nil, nil, nil).each do |page|
               page_url = (page.isController() ? page.getId() : "/plugins/home/#{page.getId()}")
@@ -76,8 +78,12 @@
                 <a href="<%= ApplicationController.root_context -%>/dashboard/index/<%= @project.id -%>?did=<%= active_dashboard.dashboard_id -%><%= "&"+period_param if period_param -%>"><%= active_dashboard.dashboard.name(true) -%></a>
               </li>
             <% end %>
+            <li class="<%= 'active' if request.request_uri.include?('/dashboards/index') -%>">
+              <a href="<%= ApplicationController.root_context -%>/dashboards/index?resource=<%= @resource.id -%>"><%= message('dashboard.manage_dashboards') -%></a></li>            
+            
             <li class="spacer"></li>
             <li class="sidebar-title"><%= message('sidebar.tools') -%></li>
+            
             <li class="<%= 'active' if request.request_uri.include?('/components/index') -%>">
               <a href="<%= ApplicationController.root_context -%>/components/index/<%= @project.id -%>"><%= message('components.page') -%></a></li>
             <li class="<%= 'active' if request.request_uri.include?('/drilldown/violations') -%>">
-- 
2.39.5