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)

3.7.2
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Bugzilla 4.0

People

(Reporter: LpSolit, Assigned: LpSolit)

References

Details

Attachments

(1 file)

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+
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).
Attached patch patch, v1Splinter Review
Assignee: create-and-change → LpSolit
Status: NEW → ASSIGNED
Attachment #457714 - Flags: review?(mkanat)
Comment on attachment 457714 [details] [diff] [review]
patch, v1

This is a really great fix, thanks! :-)
Attachment #457714 - Flags: review?(mkanat) → review+
Flags: approval4.0+
Flags: approval+
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.

Attachment

General

Created:
Updated:
Size: