86908 - buglist.cgi should also accept spaces for separators in the id parameter

Status: NEW

(Bugzilla :: Query/Bug List, enhancement, P4)

Description

[bobj]

If you enter "bugs numbered:" separated by spaces as
   86637 80446 84491

Bugzilla generates the buglist using:
   &bug_id=86637+80446+84491

Here is the full query:
http://bugzilla.mozilla.org/buglist.cgi?email1=&emailtype1=substring&emailassign
ed_to1=1&email2=&emailtype2=substring&emailreporter2=1&bugidtype=include&bug_id=
86637+80446+84491&changedin=&votes=&chfieldfrom=&chfieldto=Now&chfieldvalue=&sho
rt_desc=&short_desc_type=allwordssubstr&long_desc=&long_desc_type=allwordssubstr
&bug_file_loc=&bug_file_loc_type=allwordssubstr&status_whiteboard=&status_whiteb
oard_type=allwordssubstr&keywords=&keywords_type=anywords&field0-0-0=noop&type0-
0-0=noop&value0-0-0=&cmdtype=doit&order=Reuse+same+sort+as+last+time

If you enter "bugs numbered:" separated by '|' as
   86637|80446|84491

Bugzilla generates the buglist using:
   &bug_id=86637%7C80446%7C84491

Here is the full query:
http://bugzilla.mozilla.org/buglist.cgi?email1=&emailtype1=substring&emailassign
ed_to1=1&email2=&emailtype2=substring&emailreporter2=1&bugidtype=include&bug_id=
86637%7C80446%7C84491&changedin=&votes=&chfieldfrom=&chfieldto=Now&chfieldvalue=
&short_desc=&short_desc_type=allwordssubstr&long_desc=&long_desc_type=allwordssu
bstr&bug_file_loc=&bug_file_loc_type=allwordssubstr&status_whiteboard=&status_wh
iteboard_type=allwordssubstr&keywords=&keywords_type=anywords&field0-0-0=noop&ty
pe0-0-0=noop&value0-0-0=&cmdtype=doit&order=Reuse+same+sort+as+last+time

The only way I can get the buglist.cgi to work is by using this for the list:
   &bug_id=86637&bug_id=80446&bug_id=86637&bug_id=84491

Here is the full query:
http://bugzilla.mozilla.org/buglist.cgi?email1=&emailtype1=substring&emailassign
ed_to1=1&email2=&emailtype2=substring&emailreporter2=1&bugidtype=include&bug_id=
86637&bug_id=80446&bug_id=86637&bug_id=84491&changedin=&votes=&chfieldfrom=&chfi
eldto=Now&chfieldvalue=&short_desc=&short_desc_type=allwordssubstr&long_desc=&lo
ng_desc_type=allwordssubstr&bug_file_loc=&bug_file_loc_type=allwordssubstr&statu
s_whiteboard=&status_whiteboard_type=allwordssubstr&keywords=&keywords_type=anyw
ords&field0-0-0=noop&type0-0-0=noop&value0-0-0=&cmdtype=doit&order=Reuse+same+so
rt+as+last+time

After I hand enter the query that works (above), and I click on 
"Edit this query", it sends me to the query.cgi page and fills "bugs numbered:" 
separated by '|' as
   86637|80446|84491

and if you do not edit anything and just click on submit, it will generate
the incorrect query as in the 2nd example above.

Comment 1

[Christian Reis]

Confirmed in current CVS bugzilla. Note that comma-spacing works.

Comment 2

[Matthew Tuck [:CodeMachine]]

OK, weird.

For keywords, we support space and comma, or a sequence of these as a
separator.  We should do the same for bug numbers.

I don't see why we would need to support '|' as a separator.  I think we should
either fail when receiving multiple parameters (if possible), or we should make
sure Edit Query uses ", " as a separator.

Note that two keywords parameters are interpreted as appended together (see bug
#87407, screen 2 attachment).  We should aim for consistency.

It is important that if we have an illegal character, we should fail (for bug
numbers and keywords).

Currently we just take the numeric data before the illegal character as the one
and only bug and give incorrect results silently.

Comment 3

[Christopher Aillon (sabbatical, not receiving bugmail)]

Mass moving to new product Bugzilla...

Comment 4

[Matthew Tuck [:CodeMachine]]

*** Bug 97433 has been marked as a duplicate of this bug. ***

Comment 5

[Dave Miller [:justdave]]

We are currently trying to wrap up Bugzilla 2.16.  We are now close enough to
release time that anything that wasn't already ranked at P1 isn't going to make
the cut.  Thus this is being retargetted at 2.18.  If you strongly disagree with
this retargetting, please comment, however, be aware that we only have about 2
weeks left to review and test anything at this point, and we intend to devote
this time to the remaining bugs that were designated as release blockers.

Comment 6

[Joel Peshkin]

[e-mail] seems to care about these so they wont make 2.18.  If
someone adopts them, they can move them back. Retargetting to 2.20

Comment 7

[Max Kanat-Alexander]

This looks... really simple to handle.

Comment 8

[Max Kanat-Alexander]

This is really trivial, but I don't plan to actually work on it any time soon.