From 512b594cb4af6239b3b98da4bae8129b805cfdff Mon Sep 17 00:00:00 2001 From: Go MAEDA Date: Fri, 7 May 2021 04:47:24 +0000 Subject: [PATCH] Merged r20990 from trunk to 4.2-stable (#35201). git-svn-id: http://svn.redmine.org/redmine/branches/4.2-stable@20991 e93f8b46-1217-0410-a6f0-8f06a7374b81 --- app/models/query.rb | 2 +- test/unit/query_test.rb | 13 +++++++++++++ 2 files changed, 14 insertions(+), 1 deletion(-) diff --git a/app/models/query.rb b/app/models/query.rb index e8de4aa6d..fad3930aa 100644 --- a/app/models/query.rb +++ b/app/models/query.rb @@ -531,7 +531,7 @@ class Query < ActiveRecord::Base if has_filter?(field) || !filter.remote options[:values] = filter.values if options[:values] && values_for(field) - missing = Array(values_for(field)).select(&:present?) - options[:values].map(&:last) + missing = Array(values_for(field)).select(&:present?) - options[:values].map{|v| v[1]} if missing.any? && respond_to?(method = "find_#{field}_filter_values") options[:values] += send(method, missing) end diff --git a/test/unit/query_test.rb b/test/unit/query_test.rb index 75efaae8a..0c8f68c94 100644 --- a/test/unit/query_test.rb +++ b/test/unit/query_test.rb @@ -1519,6 +1519,19 @@ class QueryTest < ActiveSupport::TestCase end end + def test_available_filters_as_json_should_not_include_duplicate_assigned_to_id_values + set_language_if_valid 'en' + user = User.find_by_login 'dlopper' + with_current_user User.find(1) do + q = IssueQuery.new + q.filters = {"assigned_to_id" => {:operator => '=', :values => user.id.to_s}} + + filters = q.available_filters_as_json + assert_not_include [user.name, user.id.to_s], filters['assigned_to_id']['values'] + assert_include [user.name, user.id.to_s, 'active'], filters['assigned_to_id']['values'] + end + end + def test_available_filters_as_json_should_include_missing_author_id_values user = User.generate! with_current_user User.find(1) do -- 2.39.5