If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Deal with hyphens in flag names

RESOLVED DUPLICATE of bug 880282

Status

()

Bugzilla
Query/Bug List
RESOLVED DUPLICATE of bug 880282
10 years ago
2 years ago

People

(Reporter: Matthew Cline, Unassigned)

Tracking

Details

(URL)

Attachments

(1 attachment)

(Reporter)

Description

10 years ago
If you copy a flag name from a bug, like "blocking‑firefox3", the dash will be the special non-breaking hyphen character (‑) rather than an ASCII "-".  Thus if you copy-paste a flag name into the Bugzilla search form you won't get any hits; you have to delete the dash and type it in again as an ASCII "-" to get the expected results.

Comment 1

10 years ago
For the record, the reason is that flags are filtered using FILTER no_break, which do the following changes:

 $var =~ s/ /\ /g;
 $var =~ s/-/\‑/g;

I'm not sure how this bug should be fixed. Either in buglist.cgi before passing data to Search.pm or in Search.pm itself.

Comment 2

10 years ago
Created attachment 299903 [details] [diff] [review]
search.pm fix

This will de-filter flag names in Search.pm but I run into a tainting error when I try to test it.
Attachment #299903 - Flags: review?(mkanat)

Comment 3

10 years ago
This bug is really visible on b.m.o as many flag names have hyphens in them. Definitely something we should fix. Max, could you have a look at this small patch?
Assignee: query-and-buglist → jjclark1982
Target Milestone: --- → Bugzilla 3.0
Version: unspecified → 3.0.2

Comment 4

10 years ago
Comment on attachment 299903 [details] [diff] [review]
search.pm fix

I'm not getting a taint error, but this doesn't seem to fix the bug for me, on testing.
Attachment #299903 - Flags: review?(mkanat) → review-

Comment 5

9 years ago
The Bugzilla 3.0 branch is now locked to security bugs and dataloss fixes only. This bug doesn't fit into one of these two categories and is retargetted to 3.2 as part of a mass-change. To catch bugmails related to this mass-change, use lts081207 in your email client filter.
Target Milestone: Bugzilla 3.0 → Bugzilla 3.2

Comment 6

8 years ago
Bugzilla 3.2 is restricted to security bugs only. Moreover, this bug is either assigned to nobody or got no traction for several months now. Rather than retargetting it at each new release, I'm clearing the target milestone and the bug will be retargetted to some sensible release when someone starts fixing this bug for real (Bugzilla 3.8 more likely).
Target Milestone: Bugzilla 3.2 → ---

Updated

4 years ago
Assignee: jjclark1982 → query-and-buglist

Comment 7

2 years ago
This has been fixed in Bugzilla 5.0.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → DUPLICATE
Duplicate of bug: 880282
You need to log in before you can comment on or make changes to this bug.