1192180 - [Contacts]It can't enter contact details page.

Status: RESOLVED WONTFIX

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

Description

[Lance（leave from Mozilla）]

Attachment: logcat_1616.txt

[1.Description]:
[Aries v2.5][Flame v2.5][Contacts]When you add contact as attachment, then view it, device will enter import page, and it can't display contact details.
Found time:16:16
See attachment:Aries_2.5.3gp and logcat_1616.txt

[2.Testing Steps]: 
1.Launch Message.
2.Create new message.
3.Tap attachment icon -> "Communications", then select a contact.
4.Tap the attachment -> "View".

[3.Expected Result]: 
4.It should display the contact details.

[4.Actual Result]: 
4.Device will enter import page, and only display portrait and name.

[5.Reproduction build]: 
Device: Aries KK 2.5(Affected)
Build ID               20150806003241
Gaia Revision          581de383687dc441a878d2c91a0167c6ec688fef
Gaia Date              2015-08-05 01:48:40
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/9801f91760d9
Gecko Version          42.0a1
Device Name            aries
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.worker.20150805.235801
Firmware Date          Wed Aug  5 23:58:08 UTC 2015
Bootloader             s1

Device: Flame KK 2.2(Unaffected)
Build ID               20150806032503
Gaia Revision          f8b119ac30e97df991c97682ac4d4f9ca22e1793
Gaia Date              2015-07-31 13:20:55
Gecko Revision         https://hg.mozilla.org/releases/mozilla-

b2g37_v2_2/rev/a9379f716e9f
Gecko Version          37.0
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150806.065240
Firmware Date          Thu Aug  6 06:52:51 EDT 2015
Firmware Version       v18D v4
Bootloader             L1TC000118D0

Device: Flame KK 2.5(Affected)
Build ID               20150806103226
Gaia Revision          497fe3f938722b0aa49c93f975fad5d9ed3b0a82
Gaia Date              2015-08-06 05:30:02
Gecko Revision         https://hg.mozilla.org/mozilla-central/rev/2fad09a0540d
Gecko Version          42.0a1
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150806.135403
Firmware Date          Thu Aug  6 13:54:17 EDT 2015
Firmware Version       v18D v4
Bootloader             L1TC000118D0

[6.Reproduction Frequency]: 
Always Recurrence,5/5

[7.TCID]: 
Free Test

Comment 1

[Lance（leave from Mozilla）]

Attachment: Aries_2.5.3GP

Comment 2

[Norry.L.F(Leave from Mozilla)]

[Blocking Requested - why for this release]:functional regression issue

Comment 3

[Johan Lorenzo [:jlorenzo] - On PTO - Back on July 28th]

Comms triage: Functional regression.

Comment 4

[Annabelle]

b2g-inbound Regression Window:
Last Working Environmental Variables:
Device: Flame KK v2.5
Build ID               20150730014841
Gaia Revision          ac76af0d04df109601cc5e5893a32c8435eb12e5
Gaia Date              2015-07-30 08:08:42
Gecko Revision         https://hg.mozilla.org/integration/b2g-inbound/rev/c84484ec8c70
Gecko Version          42.0a1
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150728.064331
Firmware Date          Tue Jul 28 06:43:41 EDT 2015
Firmware Version       v18D v4
Bootloader             L1TC000118D0

First Broken Environmental Variables:
Device: Flame KK v2.5
Build ID               20150730032845
Gaia Revision          5e31fb6f88cd8c42e53214183defa2ffd455ea00
Gaia Date              2015-07-30 09:39:02
Gecko Revision         https://hg.mozilla.org/integration/b2g-inbound/rev/c05339efba8d
Gecko Version          42.0a1
Device Name            flame
Firmware(Release)      4.4.2
Firmware(Incremental)  eng.cltbld.20150728.064331
Firmware Date          Tue Jul 28 06:43:41 EDT 2015
Firmware Version       v18D v4
Bootloader             L1TC000118D0

First Broken Gaia & Last Working Gecko - issue DOES repro
Gaia Revision          5e31fb6f88cd8c42e53214183defa2ffd455ea00
Gecko Revision         https://hg.mozilla.org/integration/b2g-inbound/rev/c84484ec8c70

First Broken Gecko & Last Working Gaia – issue DOES NOT repro
Gecko Revision         https://hg.mozilla.org/integration/b2g-inbound/rev/c05339efba8d
Gaia Revision          ac76af0d04df109601cc5e5893a32c8435eb12e5

Gaia pushlog:
https://github.com/mozilla-b2g/gaia/compare/ac76af0d04df109601cc5e5893a32c8435eb12e5...5e31fb6f88cd8c42e53214183defa2ffd455ea00

Possibly caused by bug 1184075

Comment 5

[Maria Angeles Oteo (:oteo)]

Fernando, can you have a look at this.
It seems to be a regression introduced when merging #open and #import activities

Comment 6

[Fernando Campo (:fcampo)]

I wouldn't say it is a bug, nor a regression, as it is the expected behaviour after merging the OPEN VCARD and IMPORT VCARD activities to follow the same steps.
The new process is described in bug 1184075 comment 0

As an explanation, the process described above for SMS is not opening the Contact details, but the VCard attached on the message, that certainly contains information about a contact from the phone, but without the internal ID for it.

In the old version, this process would end up opening not the details page, but the New contact one, giving the option to save the information. In that case, we could see the rest of the information, but we wouldn't be opening the details per se.

The idea is to solve this lack of information on the NGA by making the importing list clickable, and show the details page (bug 1184080)

Comment 7

[Maria Angeles Oteo (:oteo)]

You are right, Fernando and we considered that as an extra feature.
In that case, perhaps we should try to implement the clickable list feature (if we have enough time) after finishing the views isolation as this bug has been set as 2.5+.
Francisco, what do you think?

Comment 8

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

I would focus first on stability and blockers, and if we have time will go for this.

Marking it as a feature p1.

Comment 9

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

[Blocking Requested - why for this release]:
Not a regression.

Comment 10

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

Obviously this needs to block :(

Comment 11

[Maria Angeles Oteo (:oteo)]

I am going to clear the assignee for this bug, as we are not sure about how we are going to proceed yet in case the pre-rendering is not in time for 2.5.

In that case and if we had to do roll back of the views separation already landed in master it will also include the merging of #open and #import activities that were the cause of this issue (bug 1184075).

As soon as the importing list is clickable and show the details page (bug 1184080) the issue should be fixed but this a not a quick feature to implement so setting ni to Francisco to confirm how to proceed and in case the rollback happens this bug would be invalid.

Comment 12

[Mahendra Potharaju [:mahe]]

Removed tracking-b2g as its a feature.

Comment 13

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

Clearing ni, as we have a plan to go for 2.5

Comment 14

[BMO Automation]

Firefox OS is not being worked on