Closed Bug 267231 Opened 20 years ago Closed 20 years ago

Dragging a virtual folder in an imap account to another folder fails

Categories

(Thunderbird :: Mail Window Front End, defect)

x86
All
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED
Thunderbird0.9

People

(Reporter: mscott, Assigned: mscott)

References

Details

(Keywords: fixed-aviary1.0)

Attachments

(1 file)

You get weird errors about not being able to unsubscribe / subscribe to the
folder. After the drop, we think the folder is a real imap folder and no longer
remember that it is a virtual folder.

We should allow you to drag a virtual folder inside of an imap account to
another folder in that account without messing up.
Status: NEW → ASSIGNED
Target Milestone: --- → Thunderbird0.9
Attached patch the fixSplinter Review
Comment on attachment 164245 [details] [diff] [review]
the fix

David, you might want to take a close look at what I added to make sure I
didn't mess anything up. I copied most of it from
nsLocalMailFolder::CopyFolderLocal
.
The idea is to move the summary file to the new location, delete the old folder
(which already handles the case of a virtual folder) and then notify the UI
that we've added a new folder underneath the current folder.
Attachment #164245 - Flags: superreview?(bienvenu)
Comment on attachment 164245 [details] [diff] [review]
the fix

this looks OK; I can try running with it. - is there any way we can share this
code, e.g., by moving it into nsMsgDBFolder.cpp?
Attachment #164245 - Flags: superreview?(bienvenu) → superreview+
*** Bug 267213 has been marked as a duplicate of this bug. ***
Keywords: fixed-aviary1.0
OS: Windows 2000 → All
now fixed on the trunk too
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
this now works for me using 2004110305-0.9 on linux fc2.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: