Closed
Bug 330963
Opened 20 years ago
Closed 20 years ago
install.js needs to be corrected for Calendar XPI
Categories
(Calendar :: General, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: mozilla, Unassigned)
Details
Attachments
(1 file)
|
1.94 KB,
patch
|
mostafah
:
first-review+
|
Details | Diff | Splinter Review |
When installing a calendar XPI from the current trunk in SeaMonkey, the en-US locale does not get registered correctly. That is because the install.js file still goes the default_lang way from when en-US was contained in the main calendar.jar file. Since that changed this won't work any more.
Patch coming up.
|
Reporter | |
Comment 1•20 years ago
|
||
This treats en-US as a normal locale and seems to work on my OS/2 build. No more messages about undefined command_key entities. :-)
|
|
Reporter | |
Comment 2•20 years ago
|
||
Hmm, I guess I should have diffed against the install.js in calendar/xpi/ instead. What is the other one for?
|
|
Reporter | |
Comment 3•20 years ago
|
||
Comment on attachment 215535 [details] [diff] [review]
correct install.js
Even if I used the wrong install.js this patch is also valid for the correct one (calendar/xpi/install.js).
Attachment #215535 -
Flags: first-review?(mostafah)
|
||
Updated•20 years ago
|
Attachment #215535 -
Flags: first-review?(mostafah) → first-review+
|
|
||
Comment 4•20 years ago
|
||
Patch checked in.
I've also removed the install.js and install.rdf files from calendar/ since they now live in calendar/xpi/
Status: UNCONFIRMED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•