Closed
Bug 422992
Opened 17 years ago
Closed 17 years ago
can't create new folder/subfolder anymore - error Error: NewFolder is not defined Source File: chrome://messenger/content/messenger.xul Line: 1
Categories
(Thunderbird :: Mail Window Front End, defect)
Thunderbird
Mail Window Front End
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mkmelin, Assigned: jminta)
References
Details
(Keywords: dogfood, regression)
Attachments
(1 file)
1.11 KB,
patch
|
mkmelin
:
review+
|
Details | Diff | Splinter Review |
Since at least a couple of days, create folder (or subfolder) for all types of servers fail. No dialog asking for name is put up, simply does nothing.
Error: NewFolder is not defined
Source File: chrome://messenger/content/messenger.xul
Line: 1
Assignee | ||
Comment 2•17 years ago
|
||
This is fallout from bug 421382, where my searches didn't pick up the way the NewFolder function was being used. Since GetCompositeDatasource and DoRDFCommand are gone, this is a reworked version of the function that was removed there.
Attachment #309506 -
Flags: review?(mkmelin+mozilla)
Reporter | ||
Comment 3•17 years ago
|
||
Since "" is falsy, that could be simplified to |if (!uri || !name)|
Reporter | ||
Comment 4•17 years ago
|
||
Comment on attachment 309506 [details] [diff] [review]
patch
r=mkmelin, with previous comment fixed
Attachment #309506 -
Flags: review?(mkmelin+mozilla) → review+
Updated•17 years ago
|
Keywords: dogfood
Summary: can't create new folder/subfolder anymore → can't create new folder/subfolder anymore - error Error: NewFolder is not defined Source File: chrome://messenger/content/messenger.xul Line: 1
Assignee | ||
Comment 5•17 years ago
|
||
Fixed comment #3 and landed.
Checking in mail/base/content/mailCommands.js;
/cvsroot/mozilla/mail/base/content/mailCommands.js,v <-- mailCommands.js
new revision: 1.40; previous revision: 1.39
done
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•