830730 - [CONTACTS][DIALER] The contacts added from the dialer app are not displayed

Status: VERIFIED FIXED

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

Description

[mbarone]

Unagi device
Build
Tested in Spanish

STR
1. Open the dialer
2. Edit a phone number and click on "Add Contact" button from the dialer app
3. Create the contact and click on "Hecho" button
4. Without close the dialer, change to the contact list (DON'T use the contact app)

ACTUAL RESULT
The new contact is not displayed

EXPECTED RESULT
The new contact must be displayed

Comment 1

[mbarone]

Forgot to add the build info

Gecko-ec7aa64
Gaia-df38c1b

Comment 2

[Alex Keybl [:akeybl]]

Jason is going to check whether this can be resolved by switching tabs, or if it requires the user to kill the app or restart the phone. If it can be resolved by switching tabs, this is not tef+

Comment 3

[Joe Cheng [:jcheng] (please needinfo)]

Triage: per test in the triage meeting, we need to kill the app the recover.

Comment 4

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

I found out it's more of a painting issue.  I put the lockscreen on and then unlocked and it recovered as well.

Comment 5

[Francisco Jordano [:arcturus] [:francisco]]

Stealing :)

Comment 6

[Francisco Jordano [:arcturus] [:francisco]]

Rewording the STR cause i realised we need an extra step to reproduce, we need the contacst tab to be opened before adding the contact:

STR
1. Open the dialer
2. Click on the contacts tab
3. Click in the dialer tab
4. Edit a phone number and click on "Add Contact" button from the dialer app
5. Create the contact and click on "Hecho" button
6. Without close the dialer, change to the contact list (DON'T use the contact app)

Comment 7

[Francisco Jordano [:arcturus] [:francisco]]

Attachment: Patch v1

Comment 8

[Naoki Hirata :nhirata (please use needinfo instead of cc)]

removing qawanted.  looks like francisco was able to reproduce and has a patch.

Comment 9

[Francisco Jordano [:arcturus] [:francisco]]

Patch updated on github with recommendations

Comment 10

[Jose Manuel Cantera]

landed https://github.com/mozilla-b2g/gaia/commit/5e5486cda14007fd73c0b68f42bba08284bc142f

Comment 11

[Walter Chen[:ypwalter][:wachen]]

Verified fixed in pvt b2g18 user build.

Gaia:
  Rev     7cacbcb8042b49d816a34de8e9767dcedefa6d84
  B-D     2013-01-25 02:57:10
Gecko:    http://hg.mozilla.org/releases/mozilla-b2g18/rev/32c2cc3e19ff
BuildID   20130124140859
Version   18.0

Comment 12

[Lukas Blakk [:lsblakk] use ?needinfo]

Landed on mozilla-b2g18/gaia master prior to the 1/25 branching to mozilla-b2g18_v1_0_0/v1.0.0, updating status-b2g-v1.0.0 to fixed.