I set up my archive options to archive to local folders (obviously I didn't test this when I reviewed the bug). When I click archive, an Archives folder is created, but no sub-folders, and the message isn't moved. On the error console I'm getting: Error: uncaught exception: [Exception... "Component returned failure code: 0x80550013 [nsIMsgFolder.createStorageIfMissing]" nsresult: "0x80550013 (<unknown>)" location: "JS frame :: chrome://messenger/content/mailWindowOverlay.js :: anonymous :: line 1066" data: no] This should block, as people who want to store archives locally, won't be able to.

actually, the archives folder is created, and the sub-folders, but we're returning an error when none occurred. I have my suspicions as to what the error is but I need to debug it more.

nsLocalMailFolder::CreateStorageIfMissing shouldn't treat existing folders as an error. The folders exist because of AddSubfolder, which for local folders creates the folder. In theory, we don't need to call CreateStorageIfMissing for local folders, but I'd need to verify that works. I'm not sure if that would make the js cleaner or messier. also, the folder creation logic shouldn't expect CreateStorageIfMissing to be async for local folders. requesting sr just for the nsLocalMailFolder change.

Comment on attachment 356796 [details] [diff] [review] proposed fix This works - the only issue I see now is when the Archives folder or year folders are in existence and a new sub folder is created, the folder pane isn't updated. >+ if (isImap) >+ return; >+ else >+ dstFolder = subfolder; nit: no need for else statement.

create all the folders the same way...

Comment on attachment 357312 [details] [diff] [review] proposed fix That's now working. >+ archiveFolderUri += "/" + msgYear; >+ subFolder = GetMsgFolderFromUri(archiveFolderUri, false); >+ if (!subFolder.parent) > { >- subfolder = archiveFolder.addSubfolder(msgYear); >- subfolder.createStorageIfMissing(this); >- return; >+ subFolder = GetMsgFolderFromUri(archiveFolderUri, false); >+ subFolder.createStorageIfMissing(this); You're getting the subFolder twice here, I don't think you need to.

fix checked in, with extra getting of subFolder removed.