Closed Bug 364025 Opened 18 years ago Closed 18 years ago

default value for defaultquery isn't unescaped

Categories

(Bugzilla :: Bugzilla-General, defect)

defect
Not set
trivial

Tracking

()

RESOLVED WONTFIX

People

(Reporter: spam, Unassigned)

Details

Attachments

(1 file)

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'
WONTFIX per irc comment from core devs
Status: NEW → RESOLVED
Closed: 18 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
Closed: 18 years ago18 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: