From 4a70a319c5e6aba1779b18ee31a63f0961846373 Mon Sep 17 00:00:00 2001 From: Jean-Philippe Lang Date: Sun, 28 Jul 2013 15:29:31 +0000 Subject: [PATCH] Filtering issues on "related to" may ignore other filters (#14401). git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@12058 e93f8b46-1217-0410-a6f0-8f06a7374b81 --- app/models/issue_query.rb | 5 ++--- test/unit/query_test.rb | 15 +++++++++++++++ 2 files changed, 17 insertions(+), 3 deletions(-) diff --git a/app/models/issue_query.rb b/app/models/issue_query.rb index bc2ca7abe..56ef9f08e 100644 --- a/app/models/issue_query.rb +++ b/app/models/issue_query.rb @@ -467,10 +467,9 @@ class IssueQuery < Query if relation_options[:sym] == field && !options[:reverse] sqls = [sql, sql_for_relations(field, operator, value, :reverse => true)] - sqls.join(["!", "!*", "!p"].include?(operator) ? " AND " : " OR ") - else - sql + sql = sqls.join(["!", "!*", "!p"].include?(operator) ? " AND " : " OR ") end + "(#{sql})" end IssueRelation::TYPES.keys.each do |relation_type| diff --git a/test/unit/query_test.rb b/test/unit/query_test.rb index 4ec430cf7..12580bfd0 100644 --- a/test/unit/query_test.rb +++ b/test/unit/query_test.rb @@ -791,6 +791,21 @@ class QueryTest < ActiveSupport::TestCase assert_equal [1, 2, 3], find_issues_with_query(query).map(&:id).sort end + def test_filter_on_relations_should_not_ignore_other_filter + issue = Issue.generate! + issue1 = Issue.generate!(:status_id => 1) + issue2 = Issue.generate!(:status_id => 2) + IssueRelation.create!(:relation_type => "relates", :issue_from => issue, :issue_to => issue1) + IssueRelation.create!(:relation_type => "relates", :issue_from => issue, :issue_to => issue2) + + query = IssueQuery.new(:name => '_') + query.filters = { + "status_id" => {:operator => '=', :values => ['1']}, + "relates" => {:operator => '=', :values => [issue.id.to_s]} + } + assert_equal [issue1], find_issues_with_query(query) + end + def test_statement_should_be_nil_with_no_filters q = IssueQuery.new(:name => '_') q.filters = {} -- 2.39.5