Comment on attachment 8370331 WIP patch 2 >+ PRUint32 i = 2; >+ nsAutoString newFolderNameUnderTrash(folderName); >+ while (1) { >+ rv = CheckIfFolderExists(newFolderNameUnderTrash, this, NULL); >+ if (rv == NS_MSG_FOLDER_EXISTS) { >+ newFolderNameUnderTrash.Assign(folderName); >+ newFolderNameUnderTrash.AppendLiteral("-"); // do we want this >localizable? "<folder>-<SeqNo>" >+ newFolderNameUnderTrash.AppendInt(i); >+ i++; >+ } else if (NS_SUCCEEDED(rv)) { >+ // NS_OK so the tried folder name doesn't exist >+ break; >+ } else { >+ return rv; >+ } >+ } CheckIfFolderExists is overkill if you don't want to alert, use ContainsChildNamed instead. Also, the loop is horribly written, though maybe you might think my version is going too far the other way: bool containsChild; for (int32_t i = 2; ContainsChildNamed(destFolderName, &containsChild), containsChild; i++) { // compute new folder name }
-- You received this bug notification because you are a member of Desktop Packages, which is subscribed to thunderbird in Ubuntu. https://bugs.launchpad.net/bugs/214366 Title: Can't delete a folder if Trash already contains a folder of the same name Status in Mozilla Thunderbird Mail and News: In Progress Status in “thunderbird” package in Ubuntu: Triaged Bug description: Binary package hint: mozilla-thunderbird 1. Create a new folder, eg. "Temp" 2. Delete it, OK 3. Create the same folder again 4. Delete it, OK I expect it to be moved to my Trash folder, perhaps with a suffix appended (it being the second instance of that folder in the trash). Thunderbird popped up an error message. On an imap host: The current command did not succeed. The mail server responded: Mailbox already exists. In the local folders: A folder with that name already exists. Please enter a different name (!) Workarounds: 1. Empty the trash 2. Rename the folder before deletion 3. Rename the folder already in the trash 4. Delete (permanently) the folder that is already in the trash Ubuntu 7.10 Gutsy thunderbird 2.0.0.12+nobinonly-0ubuntu0.7.10.0 To manage notifications about this bug go to: https://bugs.launchpad.net/thunderbird/+bug/214366/+subscriptions -- Mailing list: https://launchpad.net/~desktop-packages Post to : desktop-packages@lists.launchpad.net Unsubscribe : https://launchpad.net/~desktop-packages More help : https://help.launchpad.net/ListHelp