[email/IMAP] syncFolderList will duplicate folders

VERIFIED FIXED in B2G C2 (20nov-10dec)

Status

Firefox OS
Gaia::E-Mail
P2
normal
VERIFIED FIXED
5 years ago
5 years ago

People

(Reporter: asuth, Assigned: asuth)

Tracking

unspecified
B2G C2 (20nov-10dec)
x86_64
Linux

Firefox Tracking Flags

(blocking-basecamp:+)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
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.
(Assignee)

Comment 1

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

Pointer to Github pull-request
Gaia triage : P2, +... once per day duplication sounds bad.
blocking-basecamp: ? → +
Priority: -- → P2

Comment 3

5 years ago
Milestoning for C2 (deadline of 12/10), as this meets the criteria of "known P2 bugs found before or during C1".
Target Milestone: --- → B2G C2 (20nov-10dec)
(Assignee)

Comment 4

5 years ago
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
Attachment #679796 - Flags: review+
(Assignee)

Comment 5

5 years ago
landed on gaia/master by pull:
https://github.com/mozilla-b2g/gaia/pull/6386
  with merge
https://github.com/mozilla-b2g/gaia/commit/522cb8f84f730917d013f922951709ef81ce7c31
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED

Comment 6

5 years ago
Verified on Unagi device Build ID: 20130112070202 v1.o
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.