Redmine

        redirect_back_or_default edit_group_path(@group, group_users_query)

          redirect_back_or_default edit_group_path(@group, group_users_query)

  # Keeps the users tab and its pagination when redirecting after add/remove

  def group_users_query

    query = {:tab => 'users'}

    query[:users_page] = params[:users_page] if params[:users_page].present?

    query[:per_page] = params[:per_page] if params[:per_page].present?

    query

  end

  # Returns the requested page of the group's users together with its paginator

  # and the total user count, so the users tab loads only one page at a time

  # instead of rendering every user of the group.

  def paginate_group_users(group)

    scope = group.users.sorted

    user_count = scope.count

    user_pages = Redmine::Pagination::Paginator.new(user_count, per_page_option, params['users_page'], 'users_page')

    users = scope.limit(user_pages.per_page).offset(user_pages.offset).to_a

    [users, user_pages, user_count]

  end

<%= form_for(@group, :url => group_users_path(@group, :users_page => params[:users_page]), :remote => true, :method => :post) do |f| %>

<%

  # A dedicated 'users_page' parameter is used so it does not clash with the

  # 'page' used by the add-user modal's principal list (render_principals_for_new_group_users).

  users, user_pages, user_count = paginate_group_users(@group)

%>

<p><%= link_to sprite_icon('add', l(:label_user_new)), new_group_users_path(@group, :users_page => params[:users_page]), :remote => true, :class => "icon icon-add" %></p>

<% if users.any? %>

    <% users.each do |user| %>

          <%= link_to sprite_icon('link-break', l(:button_remove)), group_users_path(@group, :user_id => user, :users_page => params[:users_page]), :method => :delete, :class => 'icon icon-link-break' %>

  <span class="pagination">

    <%= pagination_links_full(user_pages, user_count) do |text, parameters, options|

          link_to text, edit_group_path(@group, request.query_parameters.merge(parameters).merge(:tab => 'users')), options

        end %>

  </span>

  def test_edit_users_tab_should_be_paginated

    group = Group.generate!

    3.times { group.users << User.generate! }

    with_settings :per_page_options => '2,25,50' do

      get(:edit, :params => {:id => group.id, :tab => 'users'})

      assert_response :success

      assert_select 'div#tab-content-users table.users tbody tr', :count => 2

      assert_select 'div#tab-content-users span.pagination'

      # Pagination links keep the users tab and use the users_page parameter

      assert_select 'div#tab-content-users span.pagination a[href*=?]', 'users_page='

      get(:edit, :params => {:id => group.id, :tab => 'users', :users_page => 2})

      assert_response :success

      assert_select 'div#tab-content-users table.users tbody tr', :count => 1

    end

  end

  def test_new_users_xhr_should_keep_current_page_in_form_action

    get(:new_users, :params => {:id => 10, :users_page => 2}, :xhr => true)

    assert_response :success

    # The add-user form posts with the current page so the list stays on it after adding

    assert_match %r{/groups/10/users\?users_page=2}, response.body

  end

  def test_paginate_group_users_returns_only_the_requested_page

    # per_page_option is provided by ApplicationController in the running app

    stubs(:per_page_option).returns(2)

    group = Group.generate!

    3.times { group.users << User.generate! }

    users, user_pages, user_count = paginate_group_users(group)

    assert_equal 2, users.size

    assert_equal 2, user_pages.per_page

    assert_equal group.users.count, user_count

  end