Closed
Bug 1127318
Opened 9 years ago
Closed 9 years ago
Contacts with only a last name and a phone number in Gmail will be displayed as "{phoneNumber} {LastName}" while importing contacts from Gmail
Categories
(Firefox OS Graveyard :: Gaia::Contacts, defect)
Tracking
(b2g-v2.2 affected, b2g-master fixed)
RESOLVED
FIXED
People
(Reporter: jlorenzo, Assigned: ythej)
References
Details
Attachments
(7 files, 1 obsolete file)
Prerequisites Have a contact with only a last name and a phone number. This issue is also reproducible if this contact has a title in Gmail. We'll take the example here of "Dr Who", "Dr" is the title and "Who" is the last name STR 1. Go to import from Gmail 2. Scroll down the list to the bottom. Expected result Like in Gmail, "Dr Who" should be displayed and should be indexed at "W". See "Gmail screenshot" Actual result You see the contact at the "#" index and your contact is called "+44123456789 Who". See "Contact import screenshot".
Reporter | ||
Comment 1•9 years ago
|
||
Francisco, I have the feeling that this issue has been here for a while. Do you think it's worth it to ask for a branch check?
Flags: needinfo?(francisco)
Comment 2•9 years ago
|
||
I'm sure it's not a regression, and has been there for a while.
Flags: needinfo?(francisco)
Updated•9 years ago
|
Assignee: nobody → yvtheja
Comment 3•9 years ago
|
||
Assignee | ||
Comment 4•9 years ago
|
||
Hi Francisco, I have added a mock contact file for testing.Please let me know if we need more mock contacts to test the condition. Thank you !
Attachment #8559801 -
Flags: review?(francisco)
Comment 5•9 years ago
|
||
Comment on attachment 8559801 [details] [review] Pull request for the bug 1127318 Canceling review of this patch, and doing it over the patch submitted by autolander.
Attachment #8559801 -
Flags: review?(francisco)
Comment 6•9 years ago
|
||
Comment on attachment 8559740 [details] [review] [PullReq] ythej:bug_1127318 to mozilla-b2g:master Great job!
Attachment #8559740 -
Flags: review+
Updated•9 years ago
|
Keywords: checkin-needed
Updated•9 years ago
|
Keywords: checkin-needed
Comment 7•9 years ago
|
||
https://github.com/mozilla-b2g/gaia/pull/27963 The pull request could not be applied to the integration branch. Please try again after current integration is complete.
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Updated•9 years ago
|
Keywords: checkin-needed
Comment 8•9 years ago
|
||
https://github.com/mozilla-b2g/gaia/pull/27963 The pull request could not be applied to the integration branch. Please try again after current integration is complete.
Assignee | ||
Updated•9 years ago
|
Keywords: checkin-needed
Updated•9 years ago
|
Keywords: checkin-needed
Comment 9•9 years ago
|
||
http://docs.taskcluster.net/tools/task-graph-inspector/#Ei4kqXLzR22CJmeJ9_pYzA The pull request failed to pass integration tests. It could not be landed, please try again.
Assignee | ||
Comment 10•9 years ago
|
||
Hi Francisco, The treeherder is green when I rebased on master but its says as comment 9.And it also says it can be merged by who has right access to the repository ! Please see this : https://github.com/mozilla-b2g/gaia/pull/27963#issuecomment-73308142 . Thank you.
Flags: needinfo?(francisco)
Updated•9 years ago
|
Flags: needinfo?(francisco)
Keywords: checkin-needed
Updated•9 years ago
|
Keywords: checkin-needed
Assignee | ||
Updated•9 years ago
|
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Updated•9 years ago
|
Comment 11•9 years ago
|
||
Landed: https://github.com/mozilla-b2g/gaia/commit/ab84a7a670853acf86e6f5c25ba8254187c7586e
Assignee | ||
Comment 12•9 years ago
|
||
Finally landed \o/
Comment hidden (obsolete) |
Comment 14•9 years ago
|
||
Updated•9 years ago
|
QA Whiteboard: [MGSEI-Triage+]
Updated•9 years ago
|
Attachment #8637826 -
Attachment is obsolete: true
Comment 15•9 years ago
|
||
Hi Vishnu, This bug has been verified as "Fail" on the latest nightly Flame master by the STR in comment 0. The "Who" contact (shows "Mr Who" in gmail) is indexed at "#" and not indexed at "W". Could you help to check this problem? Thank you very much. ------------------------------------------------------------------------------ Actual results: When importing contacts from Gmail, it displays the last name "Who" in the list as expected, but the "Who" contact (shows "Mr Who" in gmail) is indexed at "#" and not indexed at "W". See attachment: gmail_conatcts.PNG, verify_Fail_Flame_master.png and logcat.txt Reproduce rate: 10/10 Device: Flame master (Fail) Build ID 20150722010204 Gaia Revision 84c3bf622e211046d905803b34de5d331761f22d Gaia Date 2015-07-21 16:28:35 Gecko Revision https://hg.mozilla.org/mozilla-central/rev/8e5c888d0d89 Gecko Version 42.0a1 Device Name flame Firmware(Release) 4.4.2 Firmware(Incremental) eng.cltbld.20150722.042634 Firmware Date Wed Jul 22 04:26:43 EDT 2015 Bootloader L1TC000118D0 Firmware Version v18D v3 Note: The contact app has some problem on the latest Flame master, so I use 20150722010204 to verify. Please let me know if necessary.
Flags: needinfo?(yvtheja)
Updated•9 years ago
|
Comment 16•9 years ago
|
||
Comment 17•9 years ago
|
||
Assignee | ||
Comment 18•9 years ago
|
||
Hey Shally, I can't do it very soon as I am engaged in many other career works. When would you need to be fixed?
Flags: needinfo?(yvtheja) → needinfo?(lixia)
Comment 19•9 years ago
|
||
Hi Vishnu, I am not sure, but I think it can depend on your time, thanks. In addition, I will submit a following bug to track this bug.
Flags: needinfo?(lixia) → needinfo?(yvtheja)
Comment 20•9 years ago
|
||
The following up bug is bug 1187824.
Assignee | ||
Comment 21•9 years ago
|
||
I will try to complete it by soon ! Thanks Shally :)
Flags: needinfo?(yvtheja)
You need to log in
before you can comment on or make changes to this bug.
Description
•