1032323 - canonicalise_query() should omit parameters with empty values so generated URLs are shorter

Status: RESOLVED FIXED

(Bugzilla :: Bugzilla-General, defect)

Description

[Ed Morley [:emorley]]

Broken out from bug 594928 comments 4-5.

This will partly fix bug 594928, as well as reduce the cruftyness of URLs generated via other consumers of canonicalise_query():
https://github.com/bugzilla/bugzilla/search?q=canonicalise_query

Comment 1

[Ed Morley [:emorley]]

Attachment: Upstream Patch

Bug 594928 comment 5 says that for assignee etc, an empty value means "use default". So as long as there are no other cases for which the caveat in the last sentence of bug 594928 comment 5 applies, then this should work... I hope! :-)

Comment 2

[Byron Jones ‹:glob› 🎈 (PTO until 6th Jan)]

Comment on attachment 8448138 [details] [diff] [review]
Upstream Patch

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

this was a tricky one to review, but as far as i can tell this shouldn't cause any bad side-effects.

::: Bugzilla/CGI.pm
@@ +665,1 @@
>  Values in C<@exclude> are not included in the result.

nit: the blank line between paragraphs shouldn't be removed; will fix on commit

Comment 3

[Ed Morley [:emorley]]

(In reply to Byron Jones ‹:glob› from comment #2)
> this was a tricky one to review, but as far as i can tell this shouldn't
> cause any bad side-effects.

Thank you - yeah there were quite a few consumers! :-)

Comment 4

[Byron Jones ‹:glob› 🎈 (PTO until 6th Jan)]

thank you for the patch ed :)

To ssh://[e-mail]/bugzilla/bugzilla.git
   2f10cf7..515a56d  master -> master