diff --git a/app/models/issue_query.rb b/app/models/issue_query.rb index dc27cd47b..deb06dc82 100644 --- a/app/models/issue_query.rb +++ b/app/models/issue_query.rb @@ -176,6 +176,7 @@ class IssueQuery < Query ) if project add_available_filter "subject", :type => :text add_available_filter "description", :type => :text + add_available_filter "notes", :type => :text add_available_filter "created_on", :type => :date_past add_available_filter "updated_on", :type => :date_past add_available_filter "closed_on", :type => :date_past @@ -447,6 +448,14 @@ class IssueQuery < Query raise StatementInvalid.new(e.message) end + def sql_for_notes_field(field, operator, value) + subquery = "SELECT 1 FROM #{Journal.table_name}" + + " WHERE #{Journal.table_name}.journalized_type='Issue' AND #{Journal.table_name}.journalized_id=#{Issue.table_name}.id" + + " AND (#{sql_for_field field, operator.ends_with?("~") ? "~" : "*", value, Journal.table_name, 'notes'})" + + " AND (#{Journal.visible_notes_condition(User.current, :skip_pre_condition => true)})" + "#{operator.starts_with?("!") ? "NOT EXISTS" : "EXISTS"} (#{subquery})" + end + def sql_for_updated_by_field(field, operator, value) neg = (operator == '!' ? 'NOT' : '') subquery = "SELECT 1 FROM #{Journal.table_name}" + diff --git a/test/unit/query_test.rb b/test/unit/query_test.rb index a6dff724c..a174e68d4 100644 --- a/test/unit/query_test.rb +++ b/test/unit/query_test.rb @@ -803,6 +803,43 @@ class QueryTest < ActiveSupport::TestCase end end + def test_filter_notes + user = User.generate! + Journal.create!(:user_id => user.id, :journalized => Issue.find(2), :notes => 'Notes') + Journal.create!(:user_id => user.id, :journalized => Issue.find(3), :notes => 'Notes') + + issue_journals = Issue.find(1).journals.sort + assert_equal ['Journal notes', 'Some notes with Redmine links: #2, r2.'], issue_journals.map(&:notes) + assert_equal [false, false], issue_journals.map(&:private_notes) + + query = IssueQuery.new(:name => '_') + filter_name = 'notes' + assert_include filter_name, query.available_filters.keys + + query.filters = {filter_name => {:operator => '~', :values => ['Notes']}} + assert_equal [1, 2, 3], find_issues_with_query(query).map(&:id).sort + + query.filters = {filter_name => {:operator => '!~', :values => ['Notes']}} + assert_equal (Issue.ids.sort - [1, 2, 3]), find_issues_with_query(query).map(&:id).sort + end + + def test_filter_notes_should_ignore_private_notes_that_are_not_visible + user = User.generate! + Journal.create!(:user_id => user.id, :journalized => Issue.find(2), :notes => 'Notes', :private_notes => true) + Journal.create!(:user_id => user.id, :journalized => Issue.find(3), :notes => 'Notes') + + issue_journals = Issue.find(1).journals.sort + assert_equal ['Journal notes', 'Some notes with Redmine links: #2, r2.'], issue_journals.map(&:notes) + assert_equal [false, false], issue_journals.map(&:private_notes) + + query = IssueQuery.new(:name => '_') + filter_name = 'notes' + assert_include filter_name, query.available_filters.keys + + query.filters = {filter_name => {:operator => '~', :values => ['Notes']}} + assert_equal [1, 3], find_issues_with_query(query).map(&:id).sort + end + def test_filter_updated_by user = User.generate! Journal.create!(:user_id => user.id, :journalized => Issue.find(2), :notes => 'Notes')