Closed
Bug 937483
Opened 11 years ago
Closed 10 years ago
[email/IMAP] ensureEssentialFolders no longer called during account setup
Categories
(Firefox OS Graveyard :: Gaia::E-Mail, defect)
Tracking
(b2g-v1.3 affected, b2g-v1.3T affected, b2g-v1.4 affected, b2g-v2.0 affected, b2g-v2.1 fixed)
People
(Reporter: asuth, Unassigned)
References
Details
For IMAP, our account creation process used to manually invoke syncFolderList and then once that completed we called ensureEssentialFolder to fix-up unhappy IMAP accounts. I moved syncFolderList out to be a separate job, losing the call to ensureEssentialFolders in the process.
IMAP's syncFolderList should call ensureEssentialFolders after each run. It's reasonably harmless, mostly lightweight, and we'd be screwed without those folders.
Reporter | ||
Comment 1•11 years ago
|
||
It's notable that the one spot where we do call ensureEssentialFolders is in local_do_delete. The logic is sound but ideally would never be triggered since the deferral process breaks ordering and not something we want all.
Reporter | ||
Comment 3•10 years ago
|
||
This was likely fixed in bug 921050 by the introduction of a call to ensureEssentialOnlineFolders in _syncFolderComputeDeltas. There may be some awkward edge-cases relating to creating an account with a sufficiently flaky network such that we end up failing to create the folder in a sufficiently timely fashion.
Status: NEW → RESOLVED
Closed: 10 years ago
status-b2g-v1.3:
--- → affected
status-b2g-v1.3T:
--- → affected
status-b2g-v1.4:
--- → affected
status-b2g-v2.0:
--- → affected
status-b2g-v2.1:
--- → fixed
Depends on: 921050
Resolution: --- → FIXED
Could you provide the detailed reproduce steps and video for me to verify this bug. Thanks!
Flags: needinfo?(bugmail)
Reporter | ||
Comment 5•10 years ago
|
||
This bug is superseded by bug 1096058 which provides back-end automated tests and is not yet landed on v2.1 but will be landed there soon. This bug cannot be verified itself on v2.1 because of the breakage from other v2.1 changes that are dealt with in that bug.
Flags: needinfo?(bugmail)
You need to log in
before you can comment on or make changes to this bug.
Description
•