Closed Bug 1093105 Opened 10 years ago Closed 8 years ago

Contacts/mozContacts API should automatically lowercase email "equals" filterValue for consistency with how it is stored

Categories

(Core Graveyard :: DOM: Contacts, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED WONTFIX

People

(Reporter: asuth, Unassigned)

References

Details

In email bug 1087184 it was observed that if we store a contact with an email address like "MixedCase@example.com", the search value used to populate the index gets lower-cased to be "mixedcase@example.com". However, a corresponding transformation is not made when using the find() API with the "equals" filterOp. So if a call like: var req = contactsAPI.find({ filterBy: ['email'], filterOp: 'equals', filterValue: 'MixedCase@example.com" }); is made, no result is found. Right now the caller must manually lowercase filterValue to get a result. Neither the documentation at https://developer.mozilla.org/en-US/docs/Web/API/Contacts_API or https://developer.mozilla.org/en-US/docs/Web/API/ContactManager.find currently mention this. I'm currently unclear on whether we've completely abandoned standardizing mozContacts in favor of DataStore/its successor or what so it's not clear to me if it's better to fix this (arguable) bug or leave it this way so we're consistent across all versions. However, I'm going to update the find() doc page now to address this.
I think this is a real inconsistency we should fix. What do you think, Reuben?
Flags: needinfo?(reuben.bmo)
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: needinfo?(reuben.bmo)
Resolution: --- → WONTFIX
What is the rationale for WONTFIX?
The DOM Contacts API was a Firefox OS-only subsystem and no longer exists. If you are looking for Thunderbird-related contacts bugs, there is no overlap with this deprecated bugzilla component. All Thunderbird related bugs should be under the MailNews or Thunderbird products in bugzilla.
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.