From ee99848a3ba200e3b93d632534c58a9bc822218f Mon Sep 17 00:00:00 2001
From: Julien Lancelot <julien.lancelot@gmail.com>
Date: Wed, 21 Nov 2012 14:23:11 +0100
Subject: [PATCH] SONAR-3936 Add a pagination mechanism to the Quality Profile
 page

---
 .../controllers/rules_configuration_controller.rb    |  3 +++
 .../app/views/rules_configuration/index.html.erb     | 12 +++++++-----
 2 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/sonar-server/src/main/webapp/WEB-INF/app/controllers/rules_configuration_controller.rb b/sonar-server/src/main/webapp/WEB-INF/app/controllers/rules_configuration_controller.rb
index e633cbc1ffe..0b6f838924c 100644
--- a/sonar-server/src/main/webapp/WEB-INF/app/controllers/rules_configuration_controller.rb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/controllers/rules_configuration_controller.rb
@@ -71,6 +71,9 @@ class RulesConfigurationController < ApplicationController
       end
     end
 
+    @pagination = Api::Pagination.new(params)
+    @pagination.count = @rules.size
+    @current_rules = @rules[@pagination.offset, @pagination.limit]
   end
 
 
diff --git a/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/index.html.erb b/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/index.html.erb
index 0ed9901a709..043a859d694 100644
--- a/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/index.html.erb
+++ b/sonar-server/src/main/webapp/WEB-INF/app/views/rules_configuration/index.html.erb
@@ -90,9 +90,9 @@
       </li>
     <% end %>
   </ul>
-  <h2><%= @rules.size %> <%= message('results').downcase -%>
-     <% if @hidden_actives && @hidden_actives>0 %><span class="small">[<a href="<%= url_for params.merge({:rule_status => ''}) -%>" id="active-rules-link">+<%= message('rules_configuration.x_found_in_active_rules', :params => @hidden_actives) -%></a>]</span><% end %>
-     <% if @hidden_inactives && @hidden_inactives>0 %><span class="small">[<a href="<%= url_for params.merge({:rule_status => ''}) -%>" id="inactive-rules-link">+<%= message('rules_configuration.x_found_in_inactive_rules', :params => @hidden_inactives) -%></a>]</span><% end %>
+  <h2>
+     <% if @hidden_actives && @hidden_actives>0 %><span class="small"><a href="<%= url_for params.merge({:rule_status => ''}) -%>" id="active-rules-link">+<%= message('rules_configuration.x_found_in_active_rules', :params => @hidden_actives) -%></a></span><% end %>
+     <% if @hidden_inactives && @hidden_inactives>0 %><span class="small"><a href="<%= url_for params.merge({:rule_status => ''}) -%>" id="inactive-rules-link">+<%= message('rules_configuration.x_found_in_inactive_rules', :params => @hidden_inactives) -%></a></span><% end %>
   </h2>
 </div>
 <table class="table table-bordered width100 spacer-bottom" id="result_table">
@@ -103,13 +103,13 @@
     </tr>
 </thead>
 <tbody>
-<% if @rules.empty? %>
+<% if @current_rules.empty? %>
 <tr><td colspan="3" class="even"><%= message('no_results') -%>.</td></tr>
 <% end %>
 <%
   # avoid executing several times the method is_admin?()
   is_admin=is_admin?()
-  @rules.each do |rule|
+  @current_rules.each do |rule|
     active_rule = @profile.active_by_rule_id(rule.id)
 %>
   <tr id="rule_<%= rule.id -%>">
@@ -118,6 +118,8 @@
 <% end %>
 </tbody>
 </table>
+<%= render :partial => 'utils/tfoot_pagination', :locals => {:pagination => @pagination, :colspan => 2} %>
+
 <script type="text/javascript">
     function replaceTextField(span_id, key) {
       var text_field_value = $F(key);
-- 
2.39.5