275510 - Regressie install.js in langnlnl.xpi

Status: RESOLVED FIXED

(Mozilla Localizations :: nl / Dutch, defect)

Description

[Rob Janssen]

Het script install.js in langnlnl.xpi doet een 'confirm ("Dit proces zal het
taalpakket globaal ....' om aan de gebruiker te vragen of men het taalpakket
locaal wil installeren.
Dit geeft een probleem als men langnlnl.xpi installeert met een unattended install.
(wij voegen langnlnl.xpi altijd toe aan de lijst in CONFIG.INI van de installer,
zodat het Nederlandse pakket, naast het Engelse, meteen geinstalleerd wordt bij
een Mozilla installatie of update.  als je het niet zo doet dan komt Mozilla in
een inconsistente status na een update omdat de gebruiker de Nederlandse taal
geselecteerd heeft maar deze niet meer aanwezig is.  het ding praat dan Engels,
en als je alsnog het Nederlandse pakket installeert is ie in de war en praat
soms Engels en soms Nederlands.  Dan moet XUL.mfl weggegooid worden om dit te
repareren.  deze problemen treden niet op als je langnlnl meteen mee installeert)

Nou is er bij de 1.6 versie van langnlnl.xpi een oplossing voor het probleem met
die confirm() ingebouwd, waardoor we hem unattended konden installeren zonder
problemen.  In de 1.7 versie is deze fix echter verloren gegaan.
Het zou fijn zijn als dit weer ingebouwd werd.

Volgens mij is de essentie dat de regel:

if (confirm ("Dit proces zal ...

veranderd was in:

if ((ProfileFolder == null) || (confirm ("Dit proces zal ...

maar die variabele ProfileFolder bestaat niet meer in de 1.7 install.js dus er
zal nog iets meer nodig zijn.

Comment 1

[Tim Maks van den Broek [:mad_maks]]

geld deze bug ook voor de laatste seamonkey versie op mozilla.org?

of kan deze bug gesloten worden?

Comment 2

[Rob Janssen]

In seamonkey werkt het goed.  Althans ik heb dit probleem al een tijd niet meer gezien.