diff options
author | David Lawrence <dlawrence@mozilla.com> | 2011-02-18 19:13:59 +0100 |
---|---|---|
committer | David Lawrence <dlawrence@mozilla.com> | 2011-02-18 19:13:59 +0100 |
commit | 6aad3a0979417a4e131a2ac45ceabfe840aa4af6 (patch) | |
tree | 1232546a7ed43a73b4338365918ac49ec30b099a /Bugzilla | |
parent | 8cef0a14747ab7b9a1b9e2dfb98fe44eeace06d6 (diff) | |
download | bugzilla-6aad3a0979417a4e131a2ac45ceabfe840aa4af6.tar.gz bugzilla-6aad3a0979417a4e131a2ac45ceabfe840aa4af6.tar.xz |
Bug 580490 - Quicksearch should optionally not search comments
r/a=mkanat
Diffstat (limited to 'Bugzilla')
-rw-r--r-- | Bugzilla/Install.pm | 2 | ||||
-rw-r--r-- | Bugzilla/Search/Quicksearch.pm | 8 |
2 files changed, 7 insertions, 3 deletions
diff --git a/Bugzilla/Install.pm b/Bugzilla/Install.pm index 6fbaa0b75..808a50698 100644 --- a/Bugzilla/Install.pm +++ b/Bugzilla/Install.pm @@ -88,6 +88,8 @@ sub SETTINGS { default => 'before_comments' }, # 2008-08-27 LpSolit@gmail.com -- Bug 182238 timezone => { subclass => 'Timezone', default => 'local' }, + # 2011-02-07 dkl@mozilla.com -- Bug 580490 + quicksearch_fulltext => { options => ['on', 'off'], default => 'on' }, } }; diff --git a/Bugzilla/Search/Quicksearch.pm b/Bugzilla/Search/Quicksearch.pm index 340421bbb..25850a378 100644 --- a/Bugzilla/Search/Quicksearch.pm +++ b/Bugzilla/Search/Quicksearch.pm @@ -128,7 +128,7 @@ use constant COMPONENT_EXCEPTIONS => ( ); # Quicksearch-wide globals for boolean charts. -our ($chart, $and, $or); +our ($chart, $and, $or, $fulltext); sub quicksearch { my ($searchstring) = (@_); @@ -142,6 +142,8 @@ sub quicksearch { $searchstring =~ s/(^[\s,]+|[\s,]+$)//g; ThrowUserError('buglist_parameters_required') unless ($searchstring); + $fulltext = Bugzilla->user->settings->{'quicksearch_fulltext'}->{'value'} eq 'on' ? 1 : 0; + if ($searchstring =~ m/^[0-9,\s]*$/) { _bug_numbers_only($searchstring); } @@ -317,7 +319,7 @@ sub _handle_special_first_chars { if ($firstChar eq '#') { addChart('short_desc', 'substring', $baseWord, $negate); - addChart('content', 'matches', _matches_phrase($baseWord), $negate); + addChart('content', 'matches', _matches_phrase($baseWord), $negate) if $fulltext; return 1; } if ($firstChar eq ':') { @@ -489,7 +491,7 @@ sub _default_quicksearch_word { addChart('alias', 'substring', $word, $negate); addChart('short_desc', 'substring', $word, $negate); addChart('status_whiteboard', 'substring', $word, $negate); - addChart('content', 'matches', _matches_phrase($word), $negate); + addChart('content', 'matches', _matches_phrase($word), $negate) if $fulltext; } sub _handle_urls { |