diff --git a/app/controllers/repositories_controller.rb b/app/controllers/repositories_controller.rb index 6e968f9..364b834 100644 --- a/app/controllers/repositories_controller.rb +++ b/app/controllers/repositories_controller.rb @@ -92,7 +92,7 @@ class RepositoriesController < ApplicationController def committers @committers = @repository.committers - @users = @project.users + @users = @project.users.to_a additional_user_ids = @committers.collect(&:last).collect(&:to_i) - @users.collect(&:id) @users += User.where(:id => additional_user_ids).to_a unless additional_user_ids.empty? @users.compact! diff --git a/test/functional/repositories_controller_test.rb b/test/functional/repositories_controller_test.rb index 3e0fa59..056375c 100644 --- a/test/functional/repositories_controller_test.rb +++ b/test/functional/repositories_controller_test.rb @@ -279,6 +279,15 @@ class RepositoriesControllerTest < ActionController::TestCase assert_select 'input[value=foo] + select option[selected=selected]', 0 # no option selected end + def test_get_committers_without_changesets + Changeset.delete_all + @request.session[:user_id] = 2 + + get :committers, :id => 10 + assert_response :success + assert_template 'committers' + end + def test_post_committers @request.session[:user_id] = 2 # add a commit with an unknown user