Closed Bug 422992 Opened 16 years ago Closed 16 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)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mkmelin, Assigned: jminta)

References

Details

(Keywords: dogfood, regression)

Attachments

(1 file)

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
This is me.  Patch coming shortly.
Assignee: nobody → jminta
Attached patch patchSplinter Review
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)
Since "" is falsy, that could be simplified to |if (!uri || !name)|
Blocks: 421382
OS: Linux → All
Hardware: PC → All
Comment on attachment 309506 [details] [diff] [review]
patch

r=mkmelin, with previous comment fixed
Attachment #309506 - Flags: review?(mkmelin+mozilla) → review+
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
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: 16 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: