Closed
Bug 313546
Opened 19 years ago
Closed 19 years ago
long_list.cgi URLs don't redirect to show_bug correctly
Categories
(Bugzilla :: Creating/Changing Bugs, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.20
People
(Reporter: justdave, Assigned: LpSolit)
References
Details
(Keywords: regression, selenium)
Attachments
(1 file)
701 bytes,
patch
|
Wurblzap
:
review+
|
Details | Diff | Splinter Review |
When you use an existing URL to a bug viewed in long_list.cgi, it attempts to redirect you to show_bug.cgi?format=multiple, however, it forgets to include the bug numbers you passed in, so show_bug then complains that you didn't give it any bug numbers.
Example: https://bugzilla.mozilla.org/long_list.cgi?id=283623
Assignee | ||
Comment 1•19 years ago
|
||
We should kill long_list.cgi instead of fixing it. Anyway... here is a patch which restores its initial behavior. Works on both tip and 2.20.
Assignee: create-and-change → LpSolit
Status: NEW → ASSIGNED
Attachment #200594 -
Flags: review?(justdave)
Assignee | ||
Comment 2•19 years ago
|
||
This is a regression due to bug 201818.
Comment 3•19 years ago
|
||
Comment on attachment 200594 [details] [diff] [review]
patch, v1
Ok, this now accepts buglist, bug_id and id equivalently as parameters.
It spills "Use of uninitialized value in split" warnings if we don't give either of these parameters, but seeing that long_list.cgi is bound for oblivion, I don't think we should mind.
Attachment #200594 -
Flags: review?(justdave) → review+
Updated•19 years ago
|
Flags: approval?
Flags: approval2.20?
Updated•19 years ago
|
Flags: approval?
Flags: approval2.20?
Flags: approval2.20+
Flags: approval+
Assignee | ||
Comment 4•19 years ago
|
||
tip:
Checking in long_list.cgi;
/cvsroot/mozilla/webtools/bugzilla/long_list.cgi,v <-- long_list.cgi
new revision: 1.47; previous revision: 1.46
done
2.20:
Checking in long_list.cgi;
/cvsroot/mozilla/webtools/bugzilla/long_list.cgi,v <-- long_list.cgi
new revision: 1.45.6.1; previous revision: 1.45
done
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Comment 5•19 years ago
|
||
I once had a patch that made long_list a format....
You need to log in
before you can comment on or make changes to this bug.
Description
•