default value for defaultquery isn't unescaped

RESOLVED WONTFIX

Status

()

--
trivial
RESOLVED WONTFIX
12 years ago
12 years ago

People

(Reporter: spam, Unassigned)

Tracking

Details

Attachments

(1 attachment)

defaultquery has a unescaped default value that should be escaped:
 default => 'bug_status=NEW&bug_status=ASSIGNED&bug_status=REOPENED&emailassigned_to1=1&emailassigned_to2=1&emailreporter2=1&emailcc2=1&emailqa_contact2=1&order=Importance&long_desc_type=substring'
Created attachment 248809 [details] [diff] [review]
patch for tip/2.22
Attachment #248809 - Flags: review?
WONTFIX per irc comment from core devs
Status: NEW → RESOLVED
Last Resolved: 12 years ago
Resolution: --- → WONTFIX
Target Milestone: Bugzilla 3.0 → ---
Could you paste that comment here so that we have it in Bugzilla?
Status: RESOLVED → REOPENED
Resolution: WONTFIX → ---
17:22	reed 	I'll work on a patch for url_quote
17:22	reed 	but the patch in bug 364025 is still valid
17:22	reed 	can somebody review it? :)
17:27	LpSolit 	I don't think we should escape it
17:27	LpSolit 	as mkanat said, it should be done in templates
17:28	reed 	LpSolit: You have one escaped and one not.
17:28	reed 	Either make them both unescaped
17:28	reed 	or both escaped
Status: REOPENED → RESOLVED
Last Resolved: 12 years ago12 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.