Comment on attachment 8371072
WIP patch 3
>+ nsAutoString newFolderName(EmptyString());
[Strings default to empty...]
>+ while (containsChild) {
>+ rv = ContainsChildNamed(newFolderName, &containsChild);
>+ NS_ENSURE_SUCCESS(rv, rv);
>+ if (!containsChild)
>+ break;
[Weird use of loop condition.]
>+ newFolderName.AppendLiteral("(");
[Could use Append('(');]
>+confirmDuplicateFolderRename=A folder with that name already exists. Would
>you like to copy the folder under a different name?
You should include the suggested new folder name in the message, otherwise the
user will expect to choose the new name.
--
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/214366
Title:
Can't delete a folder if Trash already contains a folder of the same
name
To manage notifications about this bug go to:
https://bugs.launchpad.net/thunderbird/+bug/214366/+subscriptions
--
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs