]> git.ipfire.org Git - thirdparty/bugzilla.git/commitdiff
Bug 347864: Generally improve fulltext search
authormkanat%bugzilla.org <>
Sat, 26 Aug 2006 05:03:35 +0000 (05:03 +0000)
committermkanat%bugzilla.org <>
Sat, 26 Aug 2006 05:03:35 +0000 (05:03 +0000)
Patch By Max Kanat-Alexander <mkanat@bugzilla.org> r=myk, a=myk

Bugzilla/Search.pm

index 87c07129a2acd94bb6023ddb15e931c9de24a0ee..92ed27b80581641faa1c271d0918a591e34022da 100644 (file)
@@ -591,7 +591,7 @@ sub init {
 
              # short_desc searching for the WHERE clause
              my @words = _split_words_into_like('bugs.short_desc', $v);
-             my $term2_where = join(' AND ', @words);
+             my $term2_where = join(' OR ', @words);
 
              # short_desc relevance
              my $factor = SUMMARY_RELEVANCE_FACTOR;
@@ -605,18 +605,10 @@ sub init {
              # we SELECT the relevance value and give it an alias so we can
              # add it to the SORT BY clause when we build it in buglist.cgi.
              #
-             # Note: MySQL calculates relevance for each comment separately,
-             # so we need to do some additional calculations to get an overall
-             # relevance value, which we do by calculating the average (mean)
-             # comment relevance and then adding the summary relevance, if any.
-             # This weights summary relevance heavily, which makes sense
-             # since summaries are short and thus highly significant.
-             #
-             # Note: We should be calculating the average relevance of all
+             # Note: We should be calculating the relevance based on all
              # comments for a bug, not just matching comments, but that's hard
              # (see http://bugzilla.mozilla.org/show_bug.cgi?id=145588#c35).
-             my $select_term = "(SUM($term1)/COUNT(${table}.comment_id)"
-                             . " + $term2_select) AS relevance";
+             my $select_term = "(SUM($term1) + $term2_select) AS relevance";
 
              # add the column not used in aggregate function explicitly
              push(@groupby, 'bugs.short_desc');