Bug searches should be case-insensitive with respect to status values

Bug #154556 reported by Brad Crittenden
2
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
Critical
Tim Penhey
lazr.enum
Fix Released
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://bugs.edge.launchpad.net/ubuntu/+bugs?search=Search&field.importance=critical&field.status=New&field.status=Incomplete&field.status=Confirmed&field.status=Triaged&field.status=In+Progress&field.status=Fix+Committed
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

Revision history for this message
Curtis Hovey (sinzui) wrote :

lazr.enum handles title and token, but not cases-insensitivity. So while it is backwards compatible, it is not forward looking. I think Item and DBItem need revision to make our URLs lowercase.

Changed in malone:
importance: Undecided → High
status: New → Confirmed
Revision history for this message
Francis J. Lacoste (flacoste) wrote :

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
Curtis Hovey (sinzui)
Changed in launchpad-registry:
importance: Medium → Low
status: Confirmed → Triaged
Revision history for this message
Robert Collins (lifeless) wrote :

Any reason that lazr can't do case-insensitive matches on enums?

Revision history for this message
Robert Collins (lifeless) wrote :

(high, as per zero-oops-policy)

tags: added: oops
Changed in launchpad-foundations:
importance: Low → High
visibility: private → public
Revision history for this message
Gary Poster (gary) wrote :

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
Tim Penhey (thumper)
Changed in lazr.enum:
status: New → In Progress
assignee: nobody → Tim Penhey (thumper)
importance: Undecided → Medium
Changed in lazr.enum:
importance: Medium → Critical
Revision history for this message
Jonathan Lange (jml) wrote :

The lazr.enum part of this is marked as In Progress, but the branch is merged. What needs to happen to fix it?

Tim Penhey (thumper)
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
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.