Closed Bug 998905 Opened 10 years ago Closed 10 years ago

After importing contacts photos are not shown in the contacts list

Categories

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

ARM
Gonk (Firefox OS)
defect
Not set
major

Tracking

(b2g-v1.3 unaffected, b2g-v1.4 unaffected, b2g-v2.0 fixed)

RESOLVED FIXED
1.4 S6 (25apr)
Tracking Status
b2g-v1.3 --- unaffected
b2g-v1.4 --- unaffected
b2g-v2.0 --- fixed

People

(Reporter: jmcf, Assigned: crdlc)

References

Details

(Keywords: regression)

Attachments

(1 file)

Prerequisites:

a/ Empty contact list
b/ Facebook account

STR:

Go to FB Import
Select one friend to import
Import

Go back to the Contact list. The Contact (Friend) Photo does not appear on the list, however it appears in the contacts details. 

If you close the contacts app and then reopen it the photo will appear on the list, thus it is a problem of the contact list component not the importing module.
QA wanted to check whether this happens on 1.3 / 1.4 as well
Keywords: qawanted
Ben,

Any idea of what can be the problem? 

thanks!
Flags: needinfo?(bkelly)
I cannot reproduce it on v1.3. On v1.4 I have not tested
This issue does NOT reproduce on the latest 1.3 or 1.4

1.3 Environmental Variables: 
Device: Buri 1.3 MOZ 
BuildID: 20140419031527 
Gaia: 69dcf31d3a95765a7d7c54a14acf6adca23f351d 
Gecko: b43a6c8e467d 
Version: 28.0 
Firmware Version: v1.2-device.cfg

1.4 Environmental Variables:
Device: Buri 1.4 MOZ
BuildID: 20140421062728
Gaia: 3c53bebcc7541e5d19c96cf38ee2603927478f3c
Gecko: 633c0420e98f
Version: 30.0a2
Firmware Version: v1.2-device.cfg
Keywords: qawanted
QA Contact: jharvey
As far as I can tell its getting this:

E/GeckoConsole( 7404): Content JS ERROR at about:neterror?e=fileNotFound&u=app%3A//communications.gaiamobile.org/contacts/null&c=UTF-8&f=regular&m=app%3A//communications.gaiamobile.org/manifest.webapp&d=Firefox%20can%27t%20find%20the%20file%20at%20app%3A//communications.gaiamobile.org/contacts/null.:1226 in initPage: net-error

After fb_resolver.js fires the update event, but before the image_loader has its onUpdate handler called.

Does that help?  Not really sure where that is coming from.
Flags: needinfo?(bkelly)
Regression on master branch so nominating to 2.0?
blocking-b2g: --- → 2.0?
Keywords: regression
Assignee: nobody → crdlc
Status: NEW → ASSIGNED
Depends on: 995097
Attached file Patch v1
Good catch, thanks for the review Jose
Attachment #8410188 - Flags: review?(jmcf)
Blocks: 995097
No longer depends on: 995097
Comment on attachment 8410188 [details]
Patch v1

thanks Cristian

works perfectly!
Attachment #8410188 - Flags: review?(jmcf) → review+
added new test, 10x
Merged in master:

https://github.com/mozilla-b2g/gaia/commit/f7f448ab229abb104392e1f1c461fca050708c9d
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.4 S6 (25apr)
Already landed in master so clearing 2.0? flag
blocking-b2g: 2.0? → ---
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: