Bug searches should be case-insensitive with respect to status values
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
| Launchpad itself |
Critical
|
Tim Penhey | ||
| lazr.enum |
Critical
|
Tim Penhey |
Bug Description
Searches should not be case-sensitive with respect to the values passed as importance, status, etc.
The following search generates an OOPS because we use 'critical' instead of 'Critical':
https:/
For the feeds project we'd like to normalize the query string for caching performance. To do that we sort the query string and would like to use a normalized case by converting the entire query string to lower case. Currently that is not possible.
Related branches
- Gavin Panella: Approve on 2011-03-03
-
Diff: 47 lines (+16/-2)3 files modifiedHACKING.txt (+10/-0)
src/lazr/enum/README.txt (+3/-0)
src/lazr/enum/_enum.py (+3/-2)
- Tim Penhey (community): Approve on 2011-04-20
-
Diff: 12 lines (+1/-1)1 file modifiedversions.cfg (+1/-1)
Curtis Hovey (sinzui) wrote : | #1 |
Changed in malone: | |
importance: | Undecided → High |
status: | New → Confirmed |
Francis J. Lacoste (flacoste) wrote : | #2 |
Do bug searches uses the DBItem infrastructure? I though there was some URL mangling done for bug search to support old status names. I'm adding a task to launchpad to represent the lazr infrastructure needed for this.
Changed in launchpad: | |
importance: | Undecided → Medium |
status: | New → Confirmed |
Changed in launchpad-registry: | |
importance: | Medium → Low |
status: | Confirmed → Triaged |
Robert Collins (lifeless) wrote : | #3 |
Any reason that lazr can't do case-insensitive matches on enums?
Robert Collins (lifeless) wrote : | #4 |
(high, as per zero-oops-policy)
tags: | added: oops |
Changed in launchpad-foundations: | |
importance: | Low → High |
visibility: | private → public |
Gary Poster (gary) wrote : | #5 |
I think that doing a case insensitive match ought to be fine. We appear to be guaranteeing uniqueness in a case insensitive way already, if I read the code correctly.
Changed in launchpad: | |
importance: | High → Critical |
Changed in lazr.enum: | |
status: | New → In Progress |
assignee: | nobody → Tim Penhey (thumper) |
importance: | Undecided → Medium |
Changed in lazr.enum: | |
importance: | Medium → Critical |
Jonathan Lange (jml) wrote : | #6 |
The lazr.enum part of this is marked as In Progress, but the branch is merged. What needs to happen to fix it?
Changed in lazr.enum: | |
status: | In Progress → Fix Released |
milestone: | none → 1.1.3 |
Changed in launchpad: | |
status: | Triaged → In Progress |
assignee: | nobody → Tim Penhey (thumper) |
tags: | added: qa0ok |
tags: |
added: qa-ok removed: qa0ok |
Changed in launchpad: | |
status: | In Progress → Fix Released |
lazr.enum handles title and token, but not cases-insensiti vity. So while it is backwards compatible, it is not forward looking. I think Item and DBItem need revision to make our URLs lowercase.