Closed Bug 847307 Opened 12 years ago Closed 11 years ago

[SMS]Confirmation dialog is not shown to the user while airplane is enabled and try to send an SMS to a phone number (no contact)

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(blocking-b2g:-, b2g18+ fixed)

RESOLVED FIXED
blocking-b2g -
Tracking Status
b2g18 + fixed

People

(Reporter: bov, Assigned: borjasalguero)

References

Details

(Whiteboard: testrun 5.1)

User Agent: Mozilla/5.0 (Windows NT 6.1; rv:19.0) Gecko/20100101 Firefox/19.0
Build ID: 20130215130331

Steps to reproduce:

1-Go to the settings menu and turn Airplane Mode ON.

2-Open sms app from the home screen.

3-Write a phone number. (Not a contact number that is saved in contacts app.)

4-Write a text and press Send

  
    



Actual results:

Confirmation dialog is not shown to the user stating "in order to send a message you must first disable Flight Safe Mode" and an "OK" button. 

The user is returned to the SMS thread view. 



Expected results:

Confirmation dialog should be shown to the user stating "in order to send a message you must first disable Flight Safe Mode" and an "OK" button. 

After press OK, dialogue is closed and the user is returned to the SMS thread view. The sms must be shown with an ! icon on the right. The new SMS thread must be created.
TEST CASE IN MOZTRAP:  #4971 "Send a SMS creating a new thread with airplane mode enabled"
blocking-b2g: --- → tef?
Whiteboard: testrun 5.1
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: All → Gonk (Firefox OS)
Hardware: All → ARM
Assignee: nobody → fbsc
tracking-b2g18: --- → +
(tef- as this is does not seem to be a critical UI issue, and the user knows they are in airplane mode by the status bar area already)
blocking-b2g: tef? → -
this is working properly on master, but not on the rest, we will probably need to uplift Bug 874043 which solves the issue.

Can we have more information on the gaia (master, v1.0.1, v1-train) this was tested on, and close it or uplift the patch where necessary?
Depends on: 874043
Flags: needinfo?(bov)
It's strange, I am not reproducing the bug in v1-train but the patch is not uplifted in that branch. Beatriz, please, confirm if you reproduces it
Tried again after 874043 was closed, and it's working fine on master and v1-train, so I'm closing it.

Please reopen if you are able to repro
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Flags: needinfo?(bov)
You need to log in before you can comment on or make changes to this bug.