809979 - [email/IMAP] syncFolderList will duplicate folders

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::E-Mail, defect, P2)

Description

[Andrew Sutherland [:asuth] (he/him)]

The folder map that is used to detect new folders gets clobbered in the process of discovering new folders, so we do not suppress duplicates.

I'll do the simple fix, upgrade the map to const for future safety, and add a unit test for both IMAP and ActiveSync.

This bug actually creates distinct folders, which means that every Inbox will get its own local copies of the mesages, etc.  This will happen once per syncFolderList call, which is once per day that the e-mail app is used.

Comment 1

[Andrew Sutherland [:asuth] (he/him)]

Attachment: Pointer to Github pull request: https://github.com/mozilla-b2g/gaia-email-libs-and-more/pull/77/files

Pointer to Github pull-request

Comment 2

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

Gaia triage : P2, +... once per day duplication sounds bad.

Comment 3

[Alex Keybl [:akeybl]]

Milestoning for C2 (deadline of 12/10), as this meets the criteria of "known P2 bugs found before or during C1".

Comment 4

[Andrew Sutherland [:asuth] (he/him)]

Comment on attachment 679796 [details]
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia-email-libs-and-more/pull/77/files

r=squib on github

Comment 5

[Andrew Sutherland [:asuth] (he/him)]

landed on gaia/master by pull:
https://github.com/mozilla-b2g/gaia/pull/6386
  with merge
https://github.com/mozilla-b2g/gaia/commit/522cb8f84f730917d013f922951709ef81ce7c31

Comment 6

[Deepa Subramanian]

Verified on Unagi device Build ID: 20130112070202 v1.o