From: Go MAEDA <maeda@farend.jp>
Date: Sat, 29 Feb 2020 14:27:23 +0000 (+0000)
Subject: Merged r19538 from trunk to 4.0-stable (#33059).
X-Git-Tag: 4.0.7~15
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=3c733c84a6797e71dea7f0f4519a8012b1d381e6;p=redmine.git

Merged r19538 from trunk to 4.0-stable (#33059).


git-svn-id: http://svn.redmine.org/redmine/branches/4.0-stable@19544 e93f8b46-1217-0410-a6f0-8f06a7374b81
---

diff --git a/app/controllers/workflows_controller.rb b/app/controllers/workflows_controller.rb
index c831827cb..9f73e8006 100644
--- a/app/controllers/workflows_controller.rb
+++ b/app/controllers/workflows_controller.rb
@@ -118,7 +118,7 @@ class WorkflowsController < ApplicationController
   def find_roles
     ids = Array.wrap(params[:role_id])
     if ids == ['all']
-      @roles = Role.sorted.to_a
+      @roles = Role.sorted.select(&:consider_workflow?)
     elsif ids.present?
       @roles = Role.where(:id => ids).to_a
     end
diff --git a/test/functional/workflows_controller_test.rb b/test/functional/workflows_controller_test.rb
index d33f4558d..2a71b1a26 100644
--- a/test/functional/workflows_controller_test.rb
+++ b/test/functional/workflows_controller_test.rb
@@ -92,8 +92,8 @@ class WorkflowsControllerTest < Redmine::ControllerTest
     get :edit, :params => {:role_id => 'all', :tracker_id => 'all'}
     assert_response :success
 
-    assert_select 'select[name=?][multiple=multiple]', 'role_id[]' do
-      assert_select 'option[selected=selected]', Role.all.select(&:consider_workflow?).count
+    assert_select 'select[name=?]', 'role_id[]' do
+      assert_select 'option[selected=selected][value=all]'
     end
     assert_select 'select[name=?]', 'tracker_id[]' do
       assert_select 'option[selected=selected][value=all]'