1005954 - Images are not loaded in merge list

Status: RESOLVED FIXED

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

Comment 1

[Jason Smith [:jsmith]]

Is this a regression from the work done in bug 967277? If so, then we should add the regression keyword here.

Comment 2

[Cristian Rodriguez (:crdlc)]

I would like that QA team tests it in 1.3/1.4 because I guess that it is reproducible there

Comment 3

[Jason Smith [:jsmith]]

Can you give sample STR to use to reproduce this?

Comment 4

[Cristian Rodriguez (:crdlc)]

Attachment: Patch v1

Comment 5

[Noemí Freire (:noemi)]

Attachment: 2014-05-05-master.png

please see screenshot attached showing today's (5/5) master build behavior:
Device: Hamachi
BuildId: 20140505081601
Gecko: 6fafefc
Gaia: 2650621
Platform version: 32.0a1

Notice that no image issue will be handled within this bug, the border issue will be tackled in Bug 1005939

Comment 6

[Cristian Rodriguez (:crdlc)]

STR:

1) Open contacts app
2) Add a new contact with Name: 'A', Last name: 'B' and photo
3) Add a new one with Name: 'A', Last name: 'B' and another photo
4) "The Duplicates found" view will be displayed but no images rendered on the list

Comment 7

[Pi Wei Cheng [:piwei] (inactive)]

Attachment: Screenshot of issue on 1.3

(In reply to Cristian Rodriguez (:crdlc) from comment #2)
> I would like that QA team tests it in 1.3/1.4 because I guess that it is
> reproducible there

The issue where contacts' avatars/images are not displayed on the merge contacts screen is reproducible on today's 1.3 and 1.4. Also attaching a screenshot for issue occurring on 1.3.

Tested on:
Device: Buri 1.3 MOZ
BuildID: 20140505024001
Gaia: 667539f1ed4becc45b182a5f1046221d3eeb9e7c
Gecko: bf3fe474bf50
Version: 28.0
Firmware Version: v1.2-device.cfg

Device: Buri 1.4 MOZ
BuildID: 20140505000201
Gaia: fccb15d6940db51615545574877a62d69406b1c2
Gecko: bf937fbec8c1
Version: 30.0
Firmware Version: v1.2-device.cfg

Comment 8

[Noemí Freire (:noemi)]

regression affecting 1.3 and 1.4 branches too so asking for 1.3?

Comment 9

[Jose Manuel Cantera]

Comment on attachment 8417439 [details]
Patch v1

works perfectly 

thanks!

Comment 10

[Cristian Rodriguez (:crdlc)]

Merged in master:

https://github.com/mozilla-b2g/gaia/commit/22ff683c1b9fea85c220446ca0874a5269eaadcf

Comment 11

[Jason Smith [:jsmith]]

Vance - Can you find out if this is a cert blocker?

Comment 12

[Vance Chen [:vchen][skype:vance.lucida][[e-mail]]]

This is not a cert blocker, thanks