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)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 1023261
People
(Reporter: greatwall2686, Unassigned)
Details
(Whiteboard: torch)
Attachments
(1 file)
272.94 KB,
application/x-rar
|
Details |
* 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%
Comment 1•10 years ago
|
||
Same than bug 1023252, we should not be able to send the SMS.
Please precise which build you use.
Updated•10 years ago
|
Flags: needinfo?(greatwall2686)
Reporter | ||
Comment 2•10 years ago
|
||
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)
Comment 3•10 years ago
|
||
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.
Comment 4•10 years ago
|
||
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.
Comment 5•10 years ago
|
||
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.
Comment 6•10 years ago
|
||
gonna dupe then
Status: UNCONFIRMED → RESOLVED
Closed: 10 years ago
Resolution: --- → DUPLICATE
Comment 7•10 years ago
|
||
(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
Comment 8•10 years ago
|
||
(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.
Description
•