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)

2.15
defect
Not set
normal

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'.
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
this is so in cvs already. It was checked in with bug 98707.
Status: RESOLVED → REOPENED
Resolution: INVALID → ---
My mistake. Duh, how did I miss the fact that bug 98707 had been fixed?
Depends on: 110711
So mark it fixed already people! =)
Status: REOPENED → RESOLVED
Closed: 23 years ago23 years ago
Resolution: --- → FIXED
Target Milestone: --- → Bugzilla 2.16
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 → ---
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.
*** Bug 118774 has been marked as a duplicate of this bug. ***
Whiteboard: [blocker will fix]
Fixed in bug 110711. Gerv
Status: REOPENED → RESOLVED
Closed: 23 years ago23 years ago
Resolution: --- → FIXED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.