Closed Bug 1023240 Opened 10 years ago Closed 10 years ago

[Flame][v1.4][Gaia::SMS] The phone back to SMS conversion page when tapping “Call “button of a contact which is an email address in contact detail screen

Categories

(Firefox OS Graveyard :: Gaia::SMS, defect)

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 1023261

People

(Reporter: greatwall2686, Unassigned)

Details

(Whiteboard: torch)

Attachments

(1 file)

* Description: The phone back to SMS conversion page when tapping “Call” button of a contact which is an email address in contact detail screen * Precondition: None * Reproduction steps: 1. Open “SMS” app and create new text. 2. Input an email address in the “To” field,e.g:48924@qq.com 3. Write the SMS text and send it. 4. Click email address in main header and select “Call”. 5. Check the behavior. * Expected result: The “Call” icon should be change to “Email” or “message”. * Actual result: The phone back to SMS conversion page immediately * Reproduction build:(Buri - Firefox OS V1.4 inside) - Gaia 17b102ee8d9a724b62285547cc5f1c5d30bfb01c - Gecko https://hg.mozilla.org/releases/mozilla-b2g30_v1_4/rev/95be84248033 - BuildID 20140520000201 - Version 30.0 * Reproduction Frequency - 100%
Same than bug 1023252, we should not be able to send the SMS. Please precise which build you use.
Flags: needinfo?(greatwall2686)
We try to enter some Email addresses, only input QQ Email can be identified in "To" field,e.g:48924@qq.com Below is the software version information: - Gaia 17b102ee8d9a724b62285547cc5f1c5d30bfb01c - Gecko https://hg.mozilla.org/releases/mozilla-b2g30_v1_4/rev/95be84248033 - BuildID 20140520000201 - Version 30.0
Flags: needinfo?(greatwall2686)
Please ref bug 840515, message app could not support email before v2.0. If it's a v1.4 specific bug, the problem would be we should treat this input as invalid recipient.
Steve, we do in v1.4, but in reality I'm quite sure they modified the "invalid" check on their build, and that's why they have this bug.
ok, same issue than bug 1023261: if it starts with a digit, we assume it's a real number, and we try to send it as a number (ie: we drop the '@' and '.' characters and we replace 'qq' and 'com' by their number equivalents.
gonna dupe then
Status: UNCONFIRMED → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
(In reply to Julien Wajsberg [:julienw] from comment #4) > Steve, we do in v1.4, but in reality I'm quite sure they modified the > "invalid" check on their build, and that's why they have this bug. That's weird, I don't think we support for v1.4 because gecko land the email in 2.0, not 1.4 (see bug 866938). And reporter also used our 1.4, not their build.
Resolution: DUPLICATE → FIXED
(In reply to Steve Chung [:steveck] from comment #7) > (In reply to Julien Wajsberg [:julienw] from comment #4) > > Steve, we do in v1.4, but in reality I'm quite sure they modified the > > "invalid" check on their build, and that's why they have this bug. > > That's weird, I don't think we support for v1.4 because gecko land the email > in 2.0, not 1.4 (see bug 866938). And reporter also used our 1.4, not their > build. Yeah, I was wrong, I understood after your comment in bug 1023261 :)
Resolution: FIXED → DUPLICATE
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: