diff options
| author | Byron Jones <bjones@mozilla.com> | 2013-05-20 19:54:06 +0200 |
|---|---|---|
| committer | Byron Jones <bjones@mozilla.com> | 2013-05-20 19:54:06 +0200 |
| commit | 0b1e410c81430711a602adc56e4fc7667d1c841e (patch) | |
| tree | a538bc9da4cef2440a16e5f64c634daa6e8c1305 /Bugzilla/Search/Quicksearch.pm | |
| parent | 1f7cfaeb0ce976e5f7ec4ec9b8b02c47cd01b45e (diff) | |
| download | bugzilla-0b1e410c81430711a602adc56e4fc7667d1c841e.tar.gz bugzilla-0b1e410c81430711a602adc56e4fc7667d1c841e.tar.xz | |
Bug 828344: "contains all of the words" no longer looks for all words within the same comment or flag
r=LpSolit, a=LpSolit
Diffstat (limited to 'Bugzilla/Search/Quicksearch.pm')
| -rw-r--r-- | Bugzilla/Search/Quicksearch.pm | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/Bugzilla/Search/Quicksearch.pm b/Bugzilla/Search/Quicksearch.pm index 7424f831f..fd9d796d1 100644 --- a/Bugzilla/Search/Quicksearch.pm +++ b/Bugzilla/Search/Quicksearch.pm @@ -377,9 +377,14 @@ sub _handle_field_names { # Flag and requestee shortcut if ($or_operand =~ /^(?:flag:)?([^\?]+\?)([^\?]*)$/) { - addChart('flagtypes.name', 'substring', $1, $negate); - $chart++; $and = $or = 0; # Next chart for boolean AND - addChart('requestees.login_name', 'substring', $2, $negate); + my ($flagtype, $requestee) = ($1, $2); + addChart('flagtypes.name', 'substring', $flagtype, $negate); + if ($requestee) { + # AND + $chart++; + $and = $or = 0; + addChart('requestees.login_name', 'substring', $requestee, $negate); + } return 1; } |