Closed Bug 759237 Opened 9 years ago Closed 9 years ago
IMsg Pluggable Store .discover Sub Folders should not raise NS _MSG _FOLDER _EXISTS exception .
The method traverses all sub folders so that NS_MSG_FOLDER_EXISTS is not proper for this method.
The previous patch has an incorrect fix in nsMsgMaildirStore.cpp. Sorry for my careless.
create_sub_folders had been accidentally removed in unit test. I am so sorry for my careless.
Does this fix any known problems? There is a mention of this error code in bug 749574.
(In reply to :aceman from comment #3) > Does this fix any known problems? There is a mention of this error code in > bug 749574. Wow! Nice catch! As far as I can confirm, the fix for bug 756316 (and the fix for this bug) fixes bug 749574 too.
Comment on attachment 627855 [details] [diff] [review] Fix Hiro, thx very much for the patch. I'm wondering why you didn't change AddSubFolders instead - you could certainly argue that it should not return NS_MSG_FOLDER_EXISTS. I think that would be cleaner, but I my build isn't finished so I haven't been able to try it.
Hiro, this is what I had in mind...with this change, the test works, like it does with initial version. I think I prefer this version, so I've r+'d it. If you're OK with it, we can land it. I'd like to get this into tb 13 which means landing it today, if possible.
Attachment #627988 - Flags: review+
I'd like to consider this for tb 13 final beta, since it fixes several issues users with localized builds have been seeing.
I've requested a try server build here - http://email@example.com I think this change is relatively safe, since discoverSubFolders shouldn't care about folders existing.
Comment on attachment 627988 [details] [diff] [review] tweak AddSubFolders instead... [Triage Comment] Per discussion on irc, we're going to take the risk for this as it is hurting users in some cases, especially with localised folders.
trunk - http://hg.mozilla.org/comm-central/rev/27498a828ee9 (forgot bug #) aurora - http://hg.mozilla.org/releases/comm-aurora/rev/3c22e945104c beta - http://hg.mozilla.org/releases/comm-beta/rev/c2e021ccbd53 first two commits omitted the bug # unfortunately
Comment on attachment 627855 [details] [diff] [review] Fix I believe this request is obsolete, clearing.
You need to log in before you can comment on or make changes to this bug.