Closed
Bug 876774
Opened 12 years ago
Closed 12 years ago
[SMS] Thread List should find contact if thread has non-internationalized number
Categories
(Firefox OS Graveyard :: Gaia::SMS, defect)
Tracking
(blocking-b2g:leo+, b2g18 fixed)
People
(Reporter: gnarf, Assigned: gerard-majax)
References
()
Details
(Keywords: regression)
Attachments
(1 file)
Steps to Reproduce:
* Create a contact and store an internationalized number I.E. +18155551212
* Goto SMS App, make sure no thread with this contact exists
* Send SMS to non-internationalized number I.E. 8155551212 - ignore the contact it is suggesting.
Expected:
* Finds the internationalized contact and shows the name correctly
Actual:
* Displays the phone number without prefix, doesn't find contact
Interestingly:
* The contact suggestion pane finds the number while you are typing.
Reporter | ||
Updated•12 years ago
|
blocking-b2g: --- → leo?
Reporter | ||
Updated•12 years ago
|
Summary: [SMS] Thread List doesn't find contact if thread has non-internationalized number → [SMS] Thread List should find contact if thread has non-internationalized number
Reporter | ||
Comment 1•12 years ago
|
||
Note, the opposite is also true - The thread list will not find the contact if the non-internationalized number is stored.
Comment 2•12 years ago
|
||
Regression landing with MMS changes, going to fix after merge.
blocking-b2g: leo? → leo+
Keywords: regression
Assignee | ||
Comment 3•12 years ago
|
||
I'll try to have a look on this one.
Assignee | ||
Comment 4•12 years ago
|
||
Bug 862250 introduced a telMatch index for this purpose, but it's not being used by the SMS app.
Assignee | ||
Comment 5•12 years ago
|
||
(In reply to Corey Frang [:gnarf] [:gnarf37] from comment #0)
> Steps to Reproduce:
>
> * Create a contact and store an internationalized number I.E. +18155551212
> * Goto SMS App, make sure no thread with this contact exists
> * Send SMS to non-internationalized number I.E. 8155551212 - ignore the
> contact it is suggesting.
>
> Expected:
> * Finds the internationalized contact and shows the name correctly
>
> Actual:
> * Displays the phone number without prefix, doesn't find contact
>
> Interestingly:
> * The contact suggestion pane finds the number while you are typing.
I've checked, and it seems like the matching is working, I have my cell registered as +336xxx number in a contact, and when I try to send a SMS, typing 06xxx shows my contact.
However, there is no highlight on the number. Is it the bug you spotted ?
Flags: needinfo?(gnarf37)
Assignee | ||
Comment 6•12 years ago
|
||
(In reply to Alexandre LISSY :gerard-majax from comment #5)
> (In reply to Corey Frang [:gnarf] [:gnarf37] from comment #0)
> > Steps to Reproduce:
> >
> > * Create a contact and store an internationalized number I.E. +18155551212
> > * Goto SMS App, make sure no thread with this contact exists
> > * Send SMS to non-internationalized number I.E. 8155551212 - ignore the
> > contact it is suggesting.
> >
> > Expected:
> > * Finds the internationalized contact and shows the name correctly
> >
> > Actual:
> > * Displays the phone number without prefix, doesn't find contact
> >
> > Interestingly:
> > * The contact suggestion pane finds the number while you are typing.
>
> I've checked, and it seems like the matching is working, I have my cell
> registered as +336xxx number in a contact, and when I try to send a SMS,
> typing 06xxx shows my contact.
>
> However, there is no highlight on the number. Is it the bug you spotted ?
Okay sorry, I re-read, and I have worse when reproducing: even entering the internationalized version, it does not identify the contact when I ignore the suggestion.
Assignee | ||
Comment 7•12 years ago
|
||
According to https://www.dropbox.com/s/mrtjwvzk6xgoc02/HTML5_SMS-MMSUserStorySpecifications_20130503_V8.0.pdf page 7, we should only suggest according to contact names (first, last, etc.). There is no mention of using the phone number for this.
What should we do for this ?
Flags: needinfo?(firefoxos-ux-bugzilla)
Assignee | ||
Comment 8•12 years ago
|
||
Ayman, what should we do with this ?
Flags: needinfo?(firefoxos-ux-bugzilla) → needinfo?(aymanmaat)
Comment 9•12 years ago
|
||
In other words, should we remove the "search by phone number" ? Would fix Bug 876861 too, but I think we added it for some reason in the first time...
Reporter | ||
Comment 10•12 years ago
|
||
(In reply to Alexandre LISSY :gerard-majax from comment #5)
> (In reply to Corey Frang [:gnarf] [:gnarf37] from comment #0)
> > Steps to Reproduce:
> >
> > * Create a contact and store an internationalized number I.E. +18155551212
> > * Goto SMS App, make sure no thread with this contact exists
> > * Send SMS to non-internationalized number I.E. 8155551212 - ignore the
> > contact it is suggesting.
> >
> > Expected:
> > * Finds the internationalized contact and shows the name correctly
> >
> > Actual:
> > * Displays the phone number without prefix, doesn't find contact
> >
> > Interestingly:
> > * The contact suggestion pane finds the number while you are typing.
>
> I've checked, and it seems like the matching is working, I have my cell
> registered as +336xxx number in a contact, and when I try to send a SMS,
> typing 06xxx shows my contact.
>
> However, there is no highlight on the number. Is it the bug you spotted ?
No, the Contact Suggestion works - it's the thread list view that can't find your internationalized contact after you send a message.
Flags: needinfo?(gnarf37)
Assignee | ||
Comment 11•12 years ago
|
||
(In reply to Corey Frang [:gnarf] [:gnarf37] from comment #10)
> (In reply to Alexandre LISSY :gerard-majax from comment #5)
> > (In reply to Corey Frang [:gnarf] [:gnarf37] from comment #0)
> > > Steps to Reproduce:
> > >
> > > * Create a contact and store an internationalized number I.E. +18155551212
> > > * Goto SMS App, make sure no thread with this contact exists
> > > * Send SMS to non-internationalized number I.E. 8155551212 - ignore the
> > > contact it is suggesting.
> > >
> > > Expected:
> > > * Finds the internationalized contact and shows the name correctly
> > >
> > > Actual:
> > > * Displays the phone number without prefix, doesn't find contact
> > >
> > > Interestingly:
> > > * The contact suggestion pane finds the number while you are typing.
> >
> > I've checked, and it seems like the matching is working, I have my cell
> > registered as +336xxx number in a contact, and when I try to send a SMS,
> > typing 06xxx shows my contact.
> >
> > However, there is no highlight on the number. Is it the bug you spotted ?
>
> No, the Contact Suggestion works - it's the thread list view that can't find
> your internationalized contact after you send a message.
Thanks for the clarification, I know understand what you reported :). And I've been able to reproduce :)
Updated•12 years ago
|
Assignee: nobody → lissyx+mozillians
Assignee | ||
Comment 12•12 years ago
|
||
Please find the link to the Github pull request https://github.com/mozilla-b2g/gaia/pull/10106 that fixes this issue.
Attachment #755856 -
Flags: review?(waldron.rick)
Assignee | ||
Comment 13•12 years ago
|
||
Comment on attachment 755856 [details]
Link to Github https://github.com/mozilla-b2g/gaia/pull/10106
><html>
> <head>
> <meta http-equiv="Refresh"
> content="2; url=https://github.com/mozilla-b2g/gaia/pull/10106" />
> </head>
> <body>
> Redirect to pull request #10106
> </body>
></html>
Attachment #755856 -
Flags: review?(waldron.rick) → review?(felash)
Comment 14•12 years ago
|
||
Comment on attachment 755856 [details]
Link to Github https://github.com/mozilla-b2g/gaia/pull/10106
works perfect
r=me
Attachment #755856 -
Flags: review?(felash) → review+
Comment 15•12 years ago
|
||
master: 9f4237a7fffd852702d3997bf2724c3e71c8a5db
I think this fixes other bugs too, will try to dupe them.
clearing need info because it wasn't the purpose of this bug after all, but we'll need to ask maat for this though
Status: NEW → RESOLVED
Closed: 12 years ago
Flags: needinfo?(aymanmaat)
Resolution: --- → FIXED
Updated•12 years ago
|
status-b2g18:
--- → affected
Comment 18•12 years ago
|
||
Uplifted 9f4237a7fffd852702d3997bf2724c3e71c8a5db to:
v1-train: ae284cb689d2b74541e741d39b9773ca7c2913f9
Assignee | ||
Comment 19•12 years ago
|
||
(In reply to Corey Frang [:gnarf] [:gnarf37] from comment #1)
> Note, the opposite is also true - The thread list will not find the contact
> if the non-internationalized number is stored.
I've been looking at this counterpart now that the bug is fixed, and it might still happen in some corner cases (never had a SIM card in the phone, things like this).
Updated•12 years ago
|
Flags: in-moztrap?
Updated•12 years ago
|
Flags: in-moztrap? → in-moztrap?(ahubenya)
Updated•12 years ago
|
Flags: in-moztrap?(ahubenya) → in-moztrap+
You need to log in
before you can comment on or make changes to this bug.
Description
•