Closed Bug 207475 Opened 21 years ago Closed 21 years ago

pref("mail.identity.default.fcc_folder","mailbox://nobody@Local%20Folders/Sent"); is wrong

Categories

(MailNews Core :: Networking, defect)

x86
Windows 2000
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: timeless, Assigned: timeless)

References

()

Details

Attachments

(1 file)

The mail.identity.default.fcc_folder pref is treated as a complex value, so per the documentation (well ...) the default value should be a property file, and the property file should have an entry mail.identity.default.fcc_folder=mailbox://nobody@Local Folders/Sent (or something like that). 304 bienvenu 3.121 pref("mail.identity.default.fcc_folder","mailbox://nobody@Local%20Folders/Sent"); 12 Sep 2001 20:00 default for unsent messages should be escaped uri, r=sspitzer, sr=mscott 79245 steps: [use mozilla debug build] create a new mozilla (1.4) profile load about:config actual results: aURL mailbox://nobody@Local%20Folders/Sent rv = GetFileFromURL(aURL, getter_AddRefs(file)); rv 2152857601 [ NS_ERROR_FILE_UNRECOGNIZED_PATH ] NS_ENSURE_SUCCESS(rv, rv); nsMsgProtocol::OpenFileSocket(nsIURI * 0x10fcf8c4, unsigned int 0, int -1) line 237 nsMailboxProtocol::Initialize(nsIURI * 0x10fcf8c4) line 190 + 19 bytes nsMailboxService::NewChannel(nsMailboxService * const 0x0b77a9f4, nsIURI * 0x10fcf8c4, nsIChannel * * 0x0012d548) line 532 + 12 bytes nsIOService::NewChannelFromURI(nsIOService * const 0x03dcfb28, nsIURI * 0x10fcf8c4, nsIChannel * * 0x0012d548) line 490 + 40 bytes NS_NewChannel(nsIChannel * * 0x0012d5c4, nsIURI * 0x10fcf8c4, nsIIOService * 0x03dcfb28, nsILoadGroup * 0x00000000, nsIInterfaceRequestor * 0x00000000, unsigned int 0) line 164 + 20 bytes nsStringBundle::LoadProperties() line 125 + 46 bytes nsStringBundle::GetStringFromName(nsStringBundle * const 0x11039838, const unsigned short * 0x0012d6fc, unsigned short * * 0x10fcb798) line 262 + 8 bytes // the default value contains a URL to a .properties file ... // string names are in unicdoe Yeah this probably should be unicode return bundle->GetStringFromName(stringId.get(), return_buf); nsPrefBranch::GetDefaultFromPropertiesFile(const char * 0x11026a18, unsigned short * * 0x10fcb798) line 855 + 49 bytes nsPrefBranch::GetComplexValue(nsPrefBranch * const 0x010980a0, const char * 0x11026a18, const nsID & {...}, void * * 0x0012dc90) line 294 + 39 bytes Note that the call here is to GetComplexValue XPTC_InvokeByIndex(nsISupports * 0x010980a0, unsigned int 11, unsigned int 3, nsXPTCVariant * 0x0012dc70) line 102 XPCWrappedNative::CallMethod(XPCCallContext & {...}, XPCWrappedNative::CallMode CALL_METHOD) line 2023 + 42 bytes 0 [native frame] 1 fetchPref(prefName = "mail.identity.default.fcc_folder", prefIndex = 37) ["chrome://global/content/config.js":248] pref = [object Object] this = [object Window @ 0x111fa210] 2 onConfigLoad() ["chrome://global/content/config.js":280] prefCount = [object Object] prefArray = [...]
Attached patch fixSplinter Review
Attachment #124456 - Flags: superreview?(dmose)
Attachment #124456 - Flags: review?(neil.parkwaycc.co.uk)
about:config is a red herring; because there is no way for prefs to indicate that they're localized it tries to retrieve each pref as a localized string (because caillon asked me to) just in case.
Status: NEW → RESOLVED
Closed: 21 years ago
Resolution: --- → INVALID
Attachment #124456 - Flags: review?(neil.parkwaycc.co.uk) → review-
thank you, yes the names for the special folders are localized in another way.
Attachment #124456 - Flags: superreview?(dmose)
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: