RFE: LoadResources() should take optional locale ID param

VERIFIED INVALID

Status

Core Graveyard
Installer: XPInstall Engine
P3
normal
VERIFIED INVALID
19 years ago
2 years ago

People

(Reporter: Samir Gehani, Assigned: Samir Gehani)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Assignee)

Description

19 years ago
The stub installer would have to pass in locale info to the JARs to maintain
"locale consistency." In such a situation, LoadResources() may potentially
display the wrong locale since it uses OS-exported info which may differ from
the user's selection in the stub country picker. A modification must be made to
LoadResources() to take two params if the country picker feature does
happen:

  Object LoadResources( String aBaseName, String aLocaleID );

where aLocaleID is optional. This way one of the arguments to the install script
could be the locale ID passed in from the stub, reflecting the user's country
picker selection.

Dependency:
-----------
This is contingent upon commitment to the "country picker" in the Install
Wizards.

Updated

19 years ago
Target Milestone: M11

Comment 1

19 years ago
setting target milestone to M11
(Assignee)

Updated

19 years ago
Target Milestone: M11
(Assignee)

Comment 2

19 years ago
Removing target milestone since we have not yet committed to the country picker
and without that feature in the Install Wizards this feature will probably not
be very useful. We will us ethis bug merely for tracking purposes for now. It is
also contingent upon locale-specific file name deductions that are not yet in
the codebase.
Target Milestone: M14
This doesn't need to work until after the beta because we're not going to be
translating anything then.  Might be nice to pull in, but only if everything
else is working.
(Assignee)

Updated

19 years ago
Summary: LoadResources() should take optional locale ID param → RFE: LoadResources() should take optional locale ID param
Target Milestone: M14
(Assignee)

Comment 4

19 years ago
See previous comment about using this as a placeholder: not committed yet, hence
resetting target milestone to "empty."
This is a useful feature apart from any "country picker" UI in the wizards. A
script could attempt to use navigator.language to load appropriate resources,
such as a code-only security patch served from NetCenter.
(Assignee)

Updated

19 years ago
Status: NEW → ASSIGNED
Target Milestone: M14
(Assignee)

Updated

19 years ago
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → INVALID
(Assignee)

Comment 6

19 years ago
One feature of LoadResources is the locale "fallbacks" whereby the browser's
locale info is appended to the basename provided (unimplemented). Once this
feature goes in I don't see that the optional localeID param will be useful. If
an install script writer wants to load by explicitly specifying the localeID
then he she can include this in the basename in the original API that just
accepts the basename.

Given the above and the fact that the install wizard will not be implementing
the country picker I'm closing this out as invalid. Feel free to reopen if there
is a counter argument to the thoughts presented above.

Updated

19 years ago
Status: RESOLVED → VERIFIED

Comment 7

19 years ago
Going once...
Going twice...
Marking Verified.

Comment 8

19 years ago
Bulk move of XPInstall (component to be deleted) bugs to Installer: XPInstall
Engine
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.