Closed
Bug 110712
Opened 23 years ago
Closed 23 years ago
Default should be `contains' for e-mail SELECTs in new query.cgi
Categories
(Bugzilla :: User Interface, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 2.16
People
(Reporter: mpt, Assigned: myk)
References
Details
(Whiteboard: [blocker will fix])
To reproduce:
1. Go to <http://landfill.tequilarista.org/bbaetz/query.cgi>.
2. Look at the e-mail SELECTS in the `E-mail and numbering' section.
What you should see by default:
* `contains'.
What you see by default:
* `is'.
Assignee | ||
Comment 1•23 years ago
|
||
The code to which you refer is not yet part of Bugzilla, so you should post this
as a comment on the bug report for the new code (bug 16775) rather than as an
independent bug. Marking invalid.
Status: NEW → RESOLVED
Closed: 23 years ago
Resolution: --- → INVALID
Comment 2•23 years ago
|
||
this is so in cvs already. It was checked in with bug 98707.
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
Assignee | ||
Comment 3•23 years ago
|
||
My mistake. Duh, how did I miss the fact that bug 98707 had been fixed?
Comment 4•23 years ago
|
||
So mark it fixed already people! =)
Status: REOPENED → RESOLVED
Closed: 23 years ago → 23 years ago
Resolution: --- → FIXED
Target Milestone: --- → Bugzilla 2.16
Comment 5•23 years ago
|
||
Matty, it didn't get resolved because it isn't fixed yet. (I just checked to
make sure, and the original description as stated is still a problem).
The second comment stated that the reporter was filing a bug against code that
was not checked into CVS yet, and the additional comments after that were
stating that yes indeed the relevant code had been checked into cvs, and
therefore this was a valid bug.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 6•23 years ago
|
||
OK, I had thought I noticed this being fixed when I reviewed the new query page,
I guess not.
I'm not entirely sure I think this should be fixed though. "contains" does not
use an index and would be slower, although not hugely. I know all this because
I've tested it for bug #96101.
Now if 'contains' is the default, we know 'is' will be rarely selected, putting
extra load on the system and not improving the friendliness for anyone if
significant.
So I'd like someone to take some (local) timings in various situations on this
when they've got time.
Comment 7•23 years ago
|
||
*** Bug 118774 has been marked as a duplicate of this bug. ***
Updated•23 years ago
|
Whiteboard: [blocker will fix]
Comment 8•23 years ago
|
||
Fixed in bug 110711.
Gerv
Status: REOPENED → RESOLVED
Closed: 23 years ago → 23 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
QA Contact: matty_is_a_geek → default-qa
You need to log in
before you can comment on or make changes to this bug.
Description
•