summaryrefslogtreecommitdiffstats
path: root/extensions/BMO
diff options
context:
space:
mode:
authorByron Jones <bjones@mozilla.com>2012-07-03 09:10:31 +0200
committerByron Jones <bjones@mozilla.com>2012-07-03 09:10:31 +0200
commitaf2212de5605617e259b39768b31aac38c7acd82 (patch)
tree07bf29d0abf71a689b959def47c79aea7f28e4a8 /extensions/BMO
parenta0b5e75cab5854985c0bc5ea07666d5676857d15 (diff)
downloadbugzilla-af2212de5605617e259b39768b31aac38c7acd82.tar.gz
bugzilla-af2212de5605617e259b39768b31aac38c7acd82.tar.xz
Fix simple searching when excluding comments
Diffstat (limited to 'extensions/BMO')
-rw-r--r--extensions/BMO/Extension.pm31
1 files changed, 15 insertions, 16 deletions
diff --git a/extensions/BMO/Extension.pm b/extensions/BMO/Extension.pm
index beabb99e7..a37847167 100644
--- a/extensions/BMO/Extension.pm
+++ b/extensions/BMO/Extension.pm
@@ -787,34 +787,33 @@ sub search_operator_field_override {
}
sub _short_desc_matches {
- # copy of Bugzilla::Search::_content_matches
+ # copy of Bugzilla::Search::_content_matches with comment searching removed
- my $self = shift;
- my %func_args = @_;
- my ($chartid, $supptables, $term, $groupby, $fields, $t, $v) =
- @func_args{qw(chartid supptables term groupby fields t v)};
+ my ($self, $args) = @_;
+ my ($chart_id, $joins, $fields, $operator, $value) =
+ @$args{qw(chart_id joins fields operator value)};
my $dbh = Bugzilla->dbh;
# Add the fulltext table to the query so we can search on it.
- my $table = "bugs_fulltext_$$chartid";
- push(@$supptables, "LEFT JOIN bugs_fulltext AS $table " .
- "ON bugs.bug_id = $table.bug_id");
+ my $table = "bugs_fulltext_$chart_id";
+ push(@$joins, { table => 'bugs_fulltext', as => $table });
# Create search terms to add to the SELECT and WHERE clauses.
- my ($term1, $rterm1) = $dbh->sql_fulltext_search("$table.short_desc", $$v, 1);
- $rterm1 = $term1 if !$rterm1;
+ my ($term, $rterm) =
+ $dbh->sql_fulltext_search("$table.short_desc", $value, 2);
+ $rterm = $term if !$rterm;
# The term to use in the WHERE clause.
- $$term = $term1;
- if ($$t =~ /not/i) {
- $$term = "NOT($$term)";
+ if ($operator =~ /not/i) {
+ $term = "NOT($term)";
}
+ $args->{term} = $term;
- my $current = Bugzilla::Search::COLUMNS->{'relevance'}->{name};
+ my $current = $self->COLUMNS->{'relevance'}->{name};
$current = $current ? "$current + " : '';
# For NOT searches, we just add 0 to the relevance.
- my $select_term = $$t =~ /not/ ? 0 : "($current$rterm1)";
- Bugzilla::Search::COLUMNS->{'relevance'}->{name} = $select_term;
+ my $select_term = $operator =~ /not/ ? 0 : "($current$rterm)";
+ $self->COLUMNS->{'relevance'}->{name} = $select_term;
}
sub mailer_before_send {