Redmine

    query.merge(members_list_params)

  # Returns the scope of the members of project matching the filters

  # set in the request params

  def members_scope(project)

    project.memberships.like(params[:member_name]).with_role(params[:member_role_id])

  end

  def paginate_members(scope)

    ordered_ids = scope.sorted.pluck("#{Member.table_name}.id").uniq

    members_by_id = scope.where(:id => page_ids).preload(:project, :principal, :roles).index_by(&:id)

  # Returns the params that define the members list currently displayed

  # (pagination and filters), so that they can be preserved across the member

  # creation, edition and deletion requests

  def members_list_params

    {

      :members_page => params[:members_page],

      :member_name => params[:member_name],

      :member_role_id => params[:member_role_id]

    }.select {|_, value| value.present?}

  end

         :partial => 'projects/settings/members', :label => :label_member_plural,

         :url => {:tab => 'members'}.merge(members_list_params)},

  # Members whose principal matches the given string

  scope :like, (lambda do |arg|

    if arg.present?

      where(:user_id => Principal.like(arg).select(:id))

    end

  end)

  # Members having the given role, inherited or not

  scope :with_role, (lambda do |arg|

    if arg.present?

      where(:id => MemberRole.where(:role_id => arg.to_i).select(:member_id))

    end

  end)

<%= form_for(@member, :url => membership_path(@member, members_list_params),

<%= form_for @member, :as => :membership, :url => project_memberships_path(@project, members_list_params), :remote => true, :method => :post do |f| %>

<% members, member_pages, member_count = paginate_members(members_scope(@project)) %>

<p><%= link_to sprite_icon('add', l(:label_member_new)), new_project_membership_path(@project, members_list_params), :remote => true, :class => "icon icon-add" %></p>

<%= form_tag(settings_project_path(@project, :tab => 'members'), :method => :get, :id => 'members-filter-form') do %>

<fieldset><legend><%= l(:label_filter_plural) %></legend>

<label for='member_name'><%= l(:label_user) %> / <%= l(:label_group) %>:</label>

<%= text_field_tag 'member_name', params[:member_name], :size => 20 %>

<label for='member_role_id'><%= l(:label_role) %>:</label>

<%= select_tag 'member_role_id',

               options_for_select([[l(:label_all), '']] + Role.givable.pluck(:name, :id), params[:member_role_id]),

               :onchange => "this.form.submit(); return false;" %>

<%= submit_tag l(:button_apply), :class => "small", :name => nil %>

<%= link_to sprite_icon('reload', l(:button_clear)), settings_project_path(@project, :tab => 'members'), :class => 'icon icon-reload' %>

</fieldset>

<% end %>

 

                  edit_membership_path(member, members_list_params),

      <%= remove_link membership_path(member, members_list_params),

  def test_edit_xhr_should_keep_current_filters_in_form_action

    @request.session[:user_id] = 2

    get(:edit, :params => {:id => 2, :member_name => 'Smith', :member_role_id => 1}, :xhr => true)

    assert_response :success

    assert_match %r{/memberships/2\?member_name=Smith&member_role_id=1}, response.body

  end

  def test_new_xhr_should_keep_current_filters_in_form_action

    @request.session[:user_id] = 2

    get(:new, :params => {:project_id => 1, :member_name => 'Smith', :member_role_id => 1}, :xhr => true)

    assert_response :success

    assert_match %r{/projects/ecookbook/memberships\?member_name=Smith&member_role_id=1}, response.body

  end

  def test_update_xhr_should_keep_the_members_list_filtered

    @request.session[:user_id] = 2

    put(

      :update,

      :params => {

        :id => 2,

        :membership => {:role_ids => [2]},

        :member_role_id => 1

      },

      :xhr => true

    )

    assert_response :success

    # The updated member does not match the filter anymore, but the list

    # must still be filtered by role 1

    assert_match %r{/memberships/1\?member_role_id=1}, response.body

    assert_no_match %r{member-4}, response.body

  end

  def test_destroy_xhr_should_keep_the_members_list_filtered

    @request.session[:user_id] = 2

    assert_difference 'Member.count', -1 do

      delete(:destroy, :params => {:id => 2, :member_name => 'Smith'}, :xhr => true)

    end

    assert_response :success

    assert_match %r{/memberships/1\?member_name=Smith}, response.body

    assert_no_match %r{member-4}, response.body

  end

  def test_destroy_should_redirect_to_the_filtered_members_list

    @request.session[:user_id] = 2

    assert_difference 'Member.count', -1 do

      delete(:destroy, :params => {:id => 2, :member_name => 'Smith', :member_role_id => 1})

    end

    assert_redirected_to '/projects/ecookbook/settings/members?member_name=Smith&member_role_id=1'

  end

    assert_select 'div#tab-content-members form#members-filter-form'

  def test_settings_members_should_display_filter_form

    @request.session[:user_id] = 2

    get(:settings, :params => {:id => 'ecookbook', :tab => 'members'})

    assert_response :success

    assert_select 'div#tab-content-members form#members-filter-form[action=?]', '/projects/ecookbook/settings/members' do

      assert_select 'input[name=member_name]'

      assert_select 'select[name=member_role_id][onchange=?]', 'this.form.submit(); return false;'

      assert_select 'input[type=submit]'

      assert_select 'a[href=?]', '/projects/ecookbook/settings/members'

    end

  end

  def test_settings_members_should_filter_by_name

    @request.session[:user_id] = 2

    get(

      :settings,

      :params => {

        :id => 'ecookbook',

        :tab => 'members',

        :member_name => 'John'

      }

    )

    assert_response :success

    assert_select 'div#tab-content-members form#members-filter-form' do

      assert_select 'input[name=member_name][value=?]', 'John'

    end

    assert_select 'div#tab-content-members tr#member-1'

    assert_select 'div#tab-content-members tr#member-2', :count => 0

    assert_select 'a#tab-members[href*=?]', 'member_name=John'

  end

  def test_settings_members_should_filter_by_role

    @request.session[:user_id] = 2

    get(

      :settings,

      :params => {

        :id => 'ecookbook',

        :tab => 'members',

        :member_role_id => '2'

      }

    )

    assert_response :success

    assert_select 'div#tab-content-members form#members-filter-form' do

      assert_select 'select[name=member_role_id]' do

        assert_select 'option[value="2"][selected=selected]'

      end

    end

    assert_select 'div#tab-content-members tr#member-2'

    assert_select 'div#tab-content-members tr#member-1', :count => 0

    assert_select 'a#tab-members[href*=?]', 'member_role_id=2'

  end

  def test_settings_members_filter_with_no_matches_should_show_no_data

    @request.session[:user_id] = 2

    get(

      :settings,

      :params => {

        :id => 'ecookbook',

        :tab => 'members',

        :member_name => 'NonexistentPrincipal'

      }

    )

    assert_response :success

    assert_select 'div#tab-content-members form#members-filter-form'

    assert_select 'div#tab-content-members p.nodata'

    assert_select 'div#tab-content-members table.list.members', :count => 0

  end

  def test_settings_members_pagination_should_preserve_filters

    project = Project.find(1)

    per_page = 25

    (per_page + 5).times do

      user = User.generate!

      Member.create!(:project => project, :principal => user, :role_ids => [2])

    end

    @request.session[:user_id] = 2

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

      get(

        :settings,

        :params => {

          :id => 'ecookbook',

          :tab => 'members',

          :member_role_id => '2'

        }

      )

    end

    assert_response :success

    assert_select 'div#tab-content-members span.pagination a[href*=?]', 'member_role_id=2'

  end

    members, member_pages, member_count = paginate_members(project.memberships)

    members, _member_pages, member_count = paginate_members(project.memberships)

  def test_paginate_members_should_paginate_the_given_scope

    stubs(:per_page_option).returns(25)

    project = Project.find(1)

    members, _member_pages, member_count = paginate_members(project.memberships.with_role(2))

    assert_equal [2, 4], members.map(&:id).sort

    assert_equal 2, member_count

  end

  def test_like_scope_should_match_users_and_groups

    assert_equal [1], Project.find(1).memberships.like('Smith').ids

    assert_equal [9], Project.find(2).memberships.like('B Team').ids

  end

  def test_like_scope_with_blank_value_should_return_all_the_members

    project = Project.find(1)

    assert_equal project.memberships.ids.sort, project.memberships.like('').ids.sort

  end

  def test_with_role_scope_should_return_the_members_having_the_role

    assert_equal [2, 4], Project.find(1).memberships.with_role(2).ids.sort

  end

  def test_with_role_scope_should_return_the_members_having_the_role_inherited

    # Member 7 has the role 1 inherited from the member 6

    assert_include 7, Project.find(5).memberships.with_role(1).ids

  end

  def test_with_role_scope_with_blank_value_should_return_all_the_members

    project = Project.find(1)

    assert_equal project.memberships.ids.sort, project.memberships.with_role(nil).ids.sort

  end