FIX A newly followed dashboard should always come last in the list

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/controllers/dashboards_controller.rb b/sonar-server/src/main/webapp/WEB-INF/app/controllers/dashboards_controller.rb
index b2ee37ba280accada9963739de2750c58baf4c74..587c81fc80f5c427193451be8367daa2235b6b84 100644 (file)
--- a/sonar-server/src/main/webapp/WEB-INF/app/controllers/dashboards_controller.rb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/controllers/dashboards_controller.rb
@@ -123,7 +123,8 @@ class DashboardsController < ApplicationController
       if active_dashboard
         flash[:error]=Api::Utils.message('dashboard.error_follow_existing_name')
       else
-        current_user.active_dashboards.create(:dashboard => dashboard, :user => current_user, :order_index => current_user.active_dashboards.size+1)
+        last_active_dashboard=current_user.active_dashboards.max_by(&:order_index)
+        current_user.active_dashboards.create(:dashboard => dashboard, :user => current_user, :order_index => (last_active_dashboard ? last_active_dashboard.order_index+1 : 1))
       end
     else
       bad_request('Unknown dashboard')