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)
Tracking
(Not tracked)
VERIFIED
FIXED
Thunderbird0.9
People
(Reporter: mscott, Assigned: mscott)
References
Details
(Keywords: fixed-aviary1.0)
Attachments
(1 file)
4.17 KB,
patch
|
Bienvenu
:
superreview+
|
Details | Diff | Splinter Review |
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.
Assignee | ||
Updated•20 years ago
|
Status: NEW → ASSIGNED
Target Milestone: --- → Thunderbird0.9
Assignee | ||
Comment 1•20 years ago
|
||
Assignee | ||
Comment 2•20 years ago
|
||
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 3•20 years ago
|
||
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+
Assignee | ||
Comment 4•20 years ago
|
||
*** Bug 267213 has been marked as a duplicate of this bug. ***
Assignee | ||
Updated•20 years ago
|
Keywords: fixed-aviary1.0
Updated•20 years ago
|
OS: Windows 2000 → All
Assignee | ||
Comment 5•20 years ago
|
||
now fixed on the trunk too
Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
Comment 6•20 years ago
|
||
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.
Description
•