Created attachment 609152
proof of concept patch

I managed to strip the patch down and update it for the current architecture.
I need feedback from David Bienvenu if this is a good direction and reaction to 
the points below.

What it does:
- it does work to fix the problem as reported.
- if the folder 'Folder' is already existing in Trash, this will try to move it 
as 'Folder2', 'Folder3' (to fix comment 8).

What it is missing for now:
- if CopyFolderLocal is used not only for moving to trash then this renaming 
functionality must be done only if isChildOfTrash is true.
- it doesn't have the alert dialog informing the user the folder was renamed.

Problems:
- for some reason when a folder is deleted and then a new folder with the same 
name is created in its place, then it can't be deleted, without any error. It 
that is some other known bug or a bug in this code? After TB restart that same 
folder can be deleted fine.

-- 
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
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to