Closed
Bug 289076
Opened 19 years ago
Closed 19 years ago
plugin finder URL should not be localizable
Categories
(Core Graveyard :: Plug-ins, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: dbaron, Assigned: doronr)
References
Details
Attachments
(1 file)
4.33 KB,
patch
|
benjamin
:
review+
benjamin
:
approval-aviary1.1a2+
|
Details | Diff | Splinter Review |
I don't see any reason for the plugin finder URL to be localizable. Perhaps customizable via something like prefs, but not in a file in the language pack. See bug 289013 for problems this causes.
Reporter | ||
Updated•19 years ago
|
Flags: blocking-aviary1.1+
Comment 1•19 years ago
|
||
The locale switching on the server side of the plugin finder service should probably use the accept locale http header instead of some URL param, right?
Comment 2•19 years ago
|
||
doron - this would help us as well (customizing plugin finder for inside a company)
Assignee | ||
Comment 3•19 years ago
|
||
So would a preference be ok? Note that I used the same thing EM uses (http://lxr.mozilla.org/seamonkey/source/toolkit/locales/en-US/chrome/mozapps/extensions/extensions.properties#15).
Assignee | ||
Comment 5•19 years ago
|
||
Preference it is.
Comment 6•19 years ago
|
||
doron, for reference we should not rely on the accept-lang HTTP header, we should dynamically code locale=@LOCALE@ in the preference and replace it at runtime with the correct app locale (this is done by several of our services already).
Assignee | ||
Comment 7•19 years ago
|
||
Assignee | ||
Updated•19 years ago
|
Attachment #186477 -
Flags: review?(benjamin)
Attachment #186477 -
Flags: approval-aviary1.1a2?
Assignee | ||
Updated•19 years ago
|
Status: NEW → ASSIGNED
Updated•19 years ago
|
Attachment #186477 -
Flags: review?(benjamin)
Attachment #186477 -
Flags: review+
Attachment #186477 -
Flags: approval-aviary1.1a2?
Attachment #186477 -
Flags: approval-aviary1.1a2+
Assignee | ||
Comment 8•19 years ago
|
||
checked in.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Updated•2 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•