If I try to QuickSearch for a phrase, I get 200 random bugs that don't even contain the second word. This almost prevents me from getting work done.

reed: I'm not entirely sure that this is an upstream bug. Can you reproduce it on landfill? I'm seeing a lot of bugs like "my search returns random results" reported against bmo.

Here's a patch that should work for the 3.6 branch. 4.0 and trunk need a better fix.

Comment on attachment 457245 [details] [diff] [review] patch - v1 (3.6 branch) This won't work on Pg, as mentioned on IRC: reed: The upstream problem is that we don't actually have a fulltext engine for Pg. reed: So we have to understand quotes inside of sql_fulltext or whatever it's called, in Bugzilla/DB.pm. reed: That could probably be accomplished by using splitwords from Text::ParseWords instead of just splitting the string.

Or shellwords, more specifically--that tends to work the most simply.

Comment on attachment 457245 [details] [diff] [review] patch - v1 (3.6 branch) Landed on bmo 3.6 branch as a temp hack until a better fix is made. Committing to: bzr+ssh://bzr.mozilla.org/bmo/3.6/ modified Bugzilla/Search/Quicksearch.pm Committed revision 7160.

Looks like a dupe of bug 556579.

(In reply to comment #6) > Looks like a dupe of bug 556579. No, that bug only affects 4.0, and this bug is against 3.6.

Before I go about this: I'm assuming that Oracle fulltext understands quotes mean "this is a phrase", yes?

This should fix it on 3.6, 4.0, and trunk, I'm pretty sure.

Comment on attachment 458553 [details] [diff] [review] v1 No time to get to this until minimum of next week.

Comment on attachment 458553 [details] [diff] [review] v1 >=== modified file 'Bugzilla/Search/Quicksearch.pm' >+sub _matches_phrase { >+ $phrase =~ s/"/\\"/g; Seems to work, but I'm not sure about to trigger this line as shellwords() splits phrases on quotes. r=LpSolit

I didn't check if it applies on 4.0 and 3.6.

Thanks! It should apply on 4.0 and 3.6 also--there haven't been any significant changes to that code since 3.6.

Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/ modified Bugzilla/DB.pm modified Bugzilla/Search/Quicksearch.pm Committed revision 7397. Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/4.0/ modified Bugzilla/DB.pm modified Bugzilla/Search/Quicksearch.pm Committed revision 7341. Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/3.6/ modified Bugzilla/DB.pm modified Bugzilla/Search/Quicksearch.pm Committed revision 7144.

Something about this fix is breaking the fulltext search, according to the xt/search tests on our tinderboxen.

Okay, this fixes the problem with Pg. The problem was that shellwords absolutely can't handle unbalanced quotes--it returns an empty result if you do something like: "don't try this" for content/matches (without the double-quotes). So instead, I'm doing a hack just to allow Quicksearch to work properly on Pg.

BTW, there's also currently some taint error on Pg for quicksearch, on HEAD. I have to find out what it is.

Comment on attachment 461961 [details] [diff] [review] Pg: No shellwords, v1 Seems to work fine. Let's hope we don't regress anything else. :) r=LpSolit

Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/ modified Bugzilla/DB.pm Committed revision 7415. Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/4.0/ modified Bugzilla/DB.pm Committed revision 7358. Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/3.6/ modified Bugzilla/DB.pm Committed revision 7151.

This is "fixed" (upstream), do we need another bug to get this patch deployed so it's actually fixed for BMO users?

(In reply to comment #21) > This is "fixed" (upstream), do we need another bug to get this patch deployed > so it's actually fixed for BMO users? Well, no, I think Mozilla just needs to employ some person who actually has time to work on bugzilla.mozilla.org. (Currently all maintenance is being done by volunteers, really.)

If it got committed to the 3.6 branch upstream, then we should have it on b.m.o already. I just merged it again yesterday when 3.6.2 released.