931192 - Add support for product aliases to search

Status: VERIFIED FIXED

(bugzilla.mozilla.org :: General, defect)

Description

[David Lawrence [:dkl]]

Smallish patch coming for review.

Comment 1

[David Lawrence [:dkl]]

Attachment: 931192_1.patch

If you are comfortable with the changes and r+ then feel free to checkin so that they can be part of the next push.

dkl

Comment 2

[:glob 🎈]

Comment on attachment 822532 [details] [diff] [review]
931192_1.patch

i feel extremely uncomfortable with the idea of doing this hack, however i understand the need for it :(

if we're to hack searching, it needs to happen in the Bugzilla::Search class not in buglist.cgi to ensure we catch all ways to search (such as REST api).

Comment 3

[:glob 🎈]

i have a reasonable idea on how to do this in a clean-ish way inside Search.pm, taking :)

Comment 4

[:glob 🎈]

adding product aliases across the whole system is complicated to do, even in a hacky way.  it's easy to catch some but not all instances, leading to confusion.


i think it's reasonable to limit this to just searching (by any mechanism/api) rather than trying to cover all places where a product rename touches (eg. enter_bug UI, bug.create api, reports).

Comment 5

[:glob 🎈]

Attachment: 931192_2.patch

non-hacky search-only product alias support

Comment 6

[David Lawrence [:dkl]]

Comment on attachment 823195 [details] [diff] [review]
931192_2.patch

Review of attachment 823195 [details] [diff] [review]:
-----------------------------------------------------------------

Crashes when using quicksearch such as product:Boot2Gecko

Can't use string ("Boot2Gecko") as an ARRAY ref while "strict refs" in use at Bugzilla/Search.pm line 2730. at Bugzilla/Search.pm line 2730.

::: Bugzilla/Search.pm
@@ +2733,5 @@
> +            $aliased = 1;
> +        }
> +    }
> +    if ($aliased) {
> +        my $old_value = $args->{value};

This doesn't seem to be used anywhere else at first glance.

Comment 7

[:glob 🎈]

Attachment: 931192_3.patch

Comment 8

[David Lawrence [:dkl]]

Comment on attachment 823353 [details] [diff] [review]
931192_3.patch

Review of attachment 823353 [details] [diff] [review]:
-----------------------------------------------------------------

Only minor issue I see is that if someone uses product=Book2Gecko&product=Firefox+OS then you end up with 'Firefox OS,Firefox OS' which is harmless and that will probably not happen anyway.

r=dkl

Comment 9

[Jason Smith [:jsmith]]

Btw - let me know if you need testing support when you land this before it gets pushed to production. I could run through a bunch of B2G queries to see how well this works.

Comment 10

[David Lawrence [:dkl]]

(In reply to Jason Smith [:jsmith] from comment #9)
> Btw - let me know if you need testing support when you land this before it
> gets pushed to production. I could run through a bunch of B2G queries to see
> how well this works.

Once it is committed you will be able to test on our staging server https://bugzilla.allizom.org which is LDAP
protected. There is also https://bugzilla-dev.allizom.org which is a sanitized version of the prod database.

dkl

Comment 11

[:glob 🎈]

(In reply to David Lawrence [:dkl] from comment #10)
> Once it is committed you will be able to test on our staging server
> https://bugzilla.allizom.org which is LDAP protected.

most people won't have access to that system..

> There is also https://bugzilla-dev.allizom.org which is a
> sanitized version of the prod database.

but everyone does have access to bugzilla-dev :)


Committing to: bzr+ssh://[e-mail]/bmo/4.2/
modified Bugzilla/Search.pm
Committed revision 9101.

Comment 12

[:glob 🎈]

this is now live.

Comment 13

[Jason Smith [:jsmith]]

Marking verified - some sanity tests with some of the preexisting bug queries work great with this patch, so we're seeing the correct bugs returning here if Boot2Gecko is used with the product.