Closed
Bug 432267
Opened 18 years ago
Closed 18 years ago
Cannot Move/Copy using Recent menu
Categories
(Thunderbird :: Mail Window Front End, defect)
Thunderbird
Mail Window Front End
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 3
People
(Reporter: lrombes, Assigned: mkmelin)
References
Details
(Keywords: regression)
Attachments
(2 files)
|
1.14 KB,
patch
|
Details | Diff | Splinter Review | |
|
1.15 KB,
patch
|
Bienvenu
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9pre) Gecko/2008050506 Minefield/3.0pre
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9pre) Gecko/2008050506 Minefield/3.0pre
Cannot use the recent folder to move or copy from an IMAP folder to a local folder. Move/Copy directly to local folder works fine
Reproducible: Always
Steps to Reproduce:
1.Right click a message
2.Choose Move To or Copy To
3.Choose Recent
4.Choose a local folder
5.Nothing happens
Repeat steps 1 & 2 then
3. Choose a local folder - works fine
|
Reporter | |
Comment 1•18 years ago
|
||
Sorry wrong build ID should be version 3.0a1pre (2008050503)
|
|
Reporter | |
Updated•18 years ago
|
Version: unspecified → Trunk
|
Assignee | |
Comment 2•18 years ago
|
||
Warning: reference to undefined property event.target._folder
Source File: chrome://messenger/content/messenger.xul
Line: 1
Error: event.target._folder is undefined
Source File: chrome://messenger/content/messenger.xul
Line: 1
Confirmed on trunk. (IMAP/local doesn't make a difference it seems.)
Status: UNCONFIRMED → NEW
Ever confirmed: true
Flags: blocking-thunderbird3+
Keywords: regression
OS: Windows XP → All
Hardware: PC → All
Summary: Recent folder and IMAP Cannot Move/Copy to local using - Move/Copy directly works fine → Cannot Move/Copy using Recent menu
|
|
||
Comment 3•18 years ago
|
||
Wow, whoops. Probably just need a node._folder = folder; line here http://mxr.mozilla.org/seamonkey/source/mail/base/content/mail-folder-bindings.xml#518
|
|
Assignee | |
Comment 5•18 years ago
|
||
Assignee: nobody → mkmelin+mozilla
Status: NEW → ASSIGNED
Attachment #325163 -
Flags: review?(bienvenu)
|
|
Assignee | |
Comment 6•18 years ago
|
||
Whops.
Attachment #325163 -
Attachment is obsolete: true
Attachment #325164 -
Flags: review?(bienvenu)
Attachment #325163 -
Flags: review?(bienvenu)
|
|
||
Comment 7•18 years ago
|
||
Comment on attachment 325163 [details] [diff] [review]
proposed fix
node.setAttribute("label", label);
+ node._folder = folder
My version had a semicolon. :)
Attachment #325163 -
Attachment is obsolete: false
|
|
Assignee | |
Comment 8•18 years ago
|
||
That's what I get for steeling ;)
|
|
||
Updated•18 years ago
|
Attachment #325164 -
Flags: review?(bienvenu) → review+
|
|
Assignee | |
Comment 9•18 years ago
|
||
Checking in mail/base/content/mail-folder-bindings.xml;
/cvsroot/mozilla/mail/base/content/mail-folder-bindings.xml,v <-- mail-folder-bindings.xml
new revision: 1.7; previous revision: 1.6
done
->FIXED
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 3
You need to log in
before you can comment on or make changes to this bug.
Description
•