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

RESOLVED INVALID

Status

RESOLVED INVALID
16 years ago
10 years ago

People

(Reporter: timeless, Assigned: timeless)

Tracking

Trunk
x86
Windows 2000

Firefox Tracking Flags

(Not tracked)

Details

(URL)

Attachments

(1 attachment)

(Assignee)

Description

16 years ago
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 = [...]
(Assignee)

Comment 1

16 years ago
Created attachment 124456 [details] [diff] [review]
fix
(Assignee)

Updated

16 years ago
Attachment #124456 - Flags: superreview?(dmose)
Attachment #124456 - Flags: review?(neil.parkwaycc.co.uk)

Comment 2

16 years ago
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
Last Resolved: 16 years ago
Resolution: --- → INVALID

Updated

16 years ago
Attachment #124456 - Flags: review?(neil.parkwaycc.co.uk) → review-

Comment 3

16 years ago
thank you, yes the names for the special folders are localized in another way.
(Assignee)

Updated

16 years ago
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.