237176 - full-text results should show relevance column

Status: RESOLVED FIXED

(Bugzilla :: Query/Bug List, enhancement)

Description

[Asa Dotzler [:asa]]

It would be nice if results from the fulltext searching would display relevance
as a column.

Comment 1

[Myk Melez [:myk] [@mykmelez]]

Attachment: patch v1: implements feature

This patch makes it possible for a user to add the relevance column to both the
"order" and the "columnlist" parameters to sort by relevance and to add
relevance to the search results, respectively.	As a side effect, it removes
the restriction that full text searches be sorted by relevance, although they
remain sorted that way by default.

As with bug 234797, there's intentionally no UI here.  This is a power user
feature and should remain that way until we can figure out the right way to
make this available to the masses (in particular, the relevance values need to
be represented intelligibly).

Comment 2

[Myk Melez [:myk] [@mykmelez]]

Comment on attachment 143657 [details] [diff] [review]
patch v1: implements feature

Dave, can you peruse this patch?  Much of the changes are moving around the
code that handles the "order" parameter.  Currently it assumes a custom order
if the parameter has a dot in it, but "relevance" has no dot, so the current
code can't handle that.

My revision checks for the standard values (Bug Number, Last Changed, etc.)
first, then assumes/parses/validates a custom value.

You can check out what this does at the following URL:

http://bugzilla.mozilla.org/buglist-bm.cgi?query_format=specific&bug_status=__a
ll__&product=Bugzilla&content=foo+bar+baz&columnlist=relevance,product,componen
t,assigned_to,bug_priority,severity,short_desc&order=relevance%20desc

(Note that the version being tested in the URL doesn't contain the LIMIT
clause, although this patch does contain that clause.)

Comment 3

[Dave Miller [:justdave]]

Comment on attachment 143657 [details] [diff] [review]
patch v1: implements feature

I like this.  To heck with the %::FORM access, they haven't fixed this file yet
anyway, they can change it all when they convert this one. :)

Comment 4

[Myk Melez [:myk] [@mykmelez]]

Checking in buglist.cgi;
/cvsroot/mozilla/webtools/bugzilla/buglist.cgi,v  <--  buglist.cgi
new revision: 1.251; previous revision: 1.250
done
Checking in Bugzilla/Search.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/Search.pm,v  <--  Search.pm
new revision: 1.53; previous revision: 1.52
done
Checking in template/en/default/search/search-specific.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/search/search-specific.html.tmpl,v
 <--  search-specific.html.tmpl
new revision: 1.5; previous revision: 1.4
done
Checking in template/en/default/bug/create/create-guided.html.tmpl;
/cvsroot/mozilla/webtools/bugzilla/template/en/default/bug/create/create-guided.html.tmpl,v
 <--  create-guided.html.tmpl
new revision: 1.18; previous revision: 1.17
done

Comment 5

[Christian Reis]

/me sees comment 3 and wonders who exactly "they" should be

Comment 6

[Dave Miller [:justdave]]

I bet you do. :)