1005239 - [B2G][Contacts]Contacts can be saved with an empty phone number tag leading to an infinite saving loop

Status: RESOLVED DUPLICATE of bug 1013786

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

Description

[Ross Kuhlman ([e-mail])]

Attachment: ContactBrokenSave.txt

The device can enter an infinite loop while saving a contact if the contact has a phone tag with no phone number in it. The progress bar is displayed, and never goes away. User is never returned to contact menu, and is still capable of editing details in the contact. If the user presses the 'X' button, they are returned to the contact details screen but no new details have been saved.

Repro Steps:
1) Update a Buri to BuildID:
2) Create a new contact or edit an existing contract.
3) Add a phone number.
4) Tap the '+' button beneath the phone number to add a second phone number.
5) Add a number to the second tag.
6) Tap the 'x' button next to the phone number from step 3 to erase the number.
7) Tap the 'Done' button to save the contact.

Actual:
Progress bar appears to indicate saving, but process never completes. Must be manually aborted by user.

Expected:
The empty phone number slot is erased and the contact is saved properly.

v1.4 Environmental Variables:
Device: Buri v1.4 MOZ
BuildID:
Gaia:
Gecko:
Version:
Firmware Version: P821A10_ENG_20140410

Notes:

Repro frequency: 100%
See attached: logcat

Comment 1

[Ross Kuhlman ([e-mail])]

Comment 0 has incorrect environmental variables. Here are the correct vars:
v1.4 Environmental Variables:
Device: Open C v1.4
BuildID: 20140502000201
Gaia: 7b2b82d72cbdd1c7e0f4542cb3390802e65f473e
Gecko: 50be03cea340
Version: 30.0
Firmware Version: P821A10_ENG_20140410

This issue also occurs on Buri(1.4) and Tarako(1.3) and Open C(Master)

Comment 2

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

FYI, David.  It's an edge case.

Comment 3

[Maria Angeles Oteo (:oteo)]

I am able to reproduce this bug in latest 1.3 version in buri but in my comercial v1.1 OpenC is working fine and the contact is saved without problem.


Builds info:
------------

1.3 ICS.eng build 
B-150
Gecko-1460160
Gaia-667539f

Comment 4

[David Scravaglieri [:scravag]]

putting this in the backlog as per Comment 2

Comment 6

[Noemí Freire (:noemi)]

Hi,

I'm not able to reproduce this issue on today's (5/21) master build:
Device: Hamachi
BuildId: 20140521111011
Gecko: b2cdbab
Gaia: 7c55cc2
Platform version: 32.0a1

following the STRs provided in the description:
1) Create a new contact or edit an existing contract
2) Add a phone number
3) Tap the '+' button beneath the phone number to add a second phone number
4) Add a number to the second tag
5) Tap the 'x' button next to the phone number from step 3 to erase the number
6) Tap the 'Done' button to save the contact

The contact is properly saved with the first phone number introduced.

Comment 7

[Maria Angeles Oteo (:oteo)]

Confirming with Noemi that the issue is reproducing also in master and this bug is a duplicated of Bug 101378

Comment 8

[Maria Angeles Oteo (:oteo)]

A mistake in the bug number, sorry