Closed
Bug 578258
Opened 14 years ago
Closed 14 years ago
Calling votes.cgi with no arguments should either display the user's votes, or give a better error message if the extension is disabled
Categories
(Bugzilla :: Creating/Changing Bugs, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 4.0
People
(Reporter: LpSolit, Assigned: LpSolit)
References
Details
Attachments
(1 file)
1.55 KB,
patch
|
mkanat
:
review+
|
Details | Diff | Splinter Review |
In Bugzilla 2.x and 3.x, accessing votes.cgi directly with no arguments displays all your votes. In Bugzilla 4.0, it throws this very unhelpful error message: "I could not figure out what you wanted to do." Guess what? I wanted to see my votes! :-D If the extension is enabled, it should display your votes. If the extension is disabled, it should explain that you cannot access your votes because this extension is disabled.
Flags: blocking4.0+
Assignee | ||
Comment 1•14 years ago
|
||
One more thing: if you pass the correct arguments but the extension is disabled, you get an even more confusing error message: "The requested format does not exist with a content type of html." This is the generic error message when one template cannot be found with the correct extension (here: .html.tmpl).
Assignee | ||
Comment 2•14 years ago
|
||
Assignee: create-and-change → LpSolit
Status: NEW → ASSIGNED
Attachment #457714 -
Flags: review?(mkanat)
Comment 3•14 years ago
|
||
Comment on attachment 457714 [details] [diff] [review] patch, v1 This is a really great fix, thanks! :-)
Attachment #457714 -
Flags: review?(mkanat) → review+
Updated•14 years ago
|
Flags: approval4.0+
Flags: approval+
Assignee | ||
Comment 4•14 years ago
|
||
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/ modified votes.cgi modified template/en/default/global/code-error.html.tmpl Committed revision 7378. Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/4.0/ modified votes.cgi modified template/en/default/global/code-error.html.tmpl Committed revision 7330.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•