Closed Bug 899586 Opened 11 years ago Closed 11 years ago

Consecutive whitespaces in field values are ignored in <select> elements

Categories

(Bugzilla :: Query/Bug List, defect)

defect
Not set
minor

Tracking

()

RESOLVED FIXED
Bugzilla 4.4

People

(Reporter: edam, Assigned: LpSolit)

References

Details

(Keywords: regression)

Attachments

(1 file)

User Agent: Mozilla/5.0 (Windows NT 5.1; rv:22.0) Gecko/20100101 Firefox/22.0 (Beta/Release) Build ID: 20130618035212 Steps to reproduce: 1. In Administration -> Field Values -> Severity, add a new value, "A (blocker)" (NOTE: there is a double-space between the 'A' and the open-bracket). 2. Change a bug to have this new severity value (or create a new bug with it) 3. Perform a search for all bugs of the new severity value Actual results: Zarro bugs found Expected results: The bug should have been returned
I can reproduce the issue. In query.cgi, the severity drop-down correctly has double spaces, but they are concatenated once passed to buglist.cgi. But if you type |sev:"A (blocker)"| in the QuickSearch field, then it works correctly.
Assignee: general → query-and-buglist
Severity: normal → minor
Status: UNCONFIRMED → NEW
Component: Bugzilla-General → Query/Bug List
Ever confirmed: true
I just tested with a simple HTML page, and whitespaces are always concatenated before being passed to the script, so this is not exactly a Bugzilla bug but rather how all browsers behave (I tested with Firefox, Opera, Google Chrome and Konqueror). In text fields, consecutive spaces are not concatenated, and so things work correctly using the Custom Search section.
Actually, this is a regression due to bug 636416, which means 4.2 and older are not affected.
Depends on: 636416
Keywords: regression
Summary: Double space in severity values are not searchable → Consecutive whitespaces in field values are ignored in <select> elements
Target Milestone: --- → Bugzilla 4.4
Attached patch patch, v1Splinter Review
Consecutive whitespaces in value="..." are correctly taken into account, so let's set it in all cases (this is how it worked in 4.2 and older).
Assignee: query-and-buglist → LpSolit
Status: NEW → ASSIGNED
Attachment #783220 - Flags: review?(glob)
Attachment #783220 - Flags: review?(glob) → review?(sgreen)
Attachment #783220 - Flags: review?(sgreen) → review+
Flags: approval?
Flags: approval4.4?
Flags: approval?
Flags: approval4.4?
Flags: approval4.4+
Flags: approval+
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/ modified template/en/default/search/field.html.tmpl Committed revision 8695. Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/4.4/ modified template/en/default/search/field.html.tmpl Committed revision 8591.
Status: ASSIGNED → RESOLVED
Closed: 11 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: