436783 - config.dtd contains duplicated entities (aboutWarning*.label)

Status: VERIFIED FIXED

(Mozilla Localizations :: pl / Polish, defect)

Description

[Hubert Gajewski]

Attachment: Patch

l10n/pl/toolkit/chrome/global/config.dtd contains duplicated entities:

<!ENTITY aboutWarningTitle.label    "Grozi utratą gwarancji!">
<!ENTITY aboutWarningText.label     "Modyfikacja tych ustawień może spowodować problemy, takie jak utrata stabilności i wydajności programu oraz zagrożenia bezpieczeństwa. Należy kontynuować tylko wtedy, gdy ma się pewność tego, co się robi.">
<!ENTITY aboutWarningButton.label   "Zachowam ostrożność, obiecuję!">
<!ENTITY aboutWarningCheckbox.label "Wyświetl to ostrzeżenie następnym razem">

<!ENTITY aboutWarningTitle.label    "Ostrzeżenie przed niebezpieczeństwem">
<!ENTITY aboutWarningText.label     "Ta strona konfiguracyjna pozwala zmodyfikować wszystkie ustawienia programu. Modyfikacja niektórych z nich może spowodować problemy, takie jak: utrata stabilności, uszkodzenie danych, niedziałanie niektórych funkcji programu lub nawet zagrożenia bezpieczeństwa.">
<!ENTITY aboutWarningButton.label   "Rozumiem i chcę kontynuować">
<!ENTITY aboutWarningCheckbox.label "Wyświetl to ostrzeżenie następnym razem">

BTW I have changed contributors' e-mail addresses (there are out-of-date).

Comment 1

[Marek Stępień [:marcoos, inactive]]

Holy crap. 

Comment 2

[Marek Stępień [:marcoos, inactive]]

At least this doesn't break anything and Firefox uses the first-defined entities.

It'd be nice to have it in for 3.0.1 or next RC of 3.0, if that happens.

Comment 3

[Axel Hecht [:Pike]]

Comment on attachment 323285 [details] [diff] [review]
Patch

a=me, please land this with a check-in comment referencing this bug and my approval.

Please use the fixed1.9.0.1 and verified1.9.0.1 keywords to track landing and testing.

Please cancel the 1.9 flag, too.

Comment 4

[Axel Hecht [:Pike]]

Comment on attachment 323285 [details] [diff] [review]
Patch

I can actually cancel myself.

Comment 5

[Marek Stępień [:marcoos, inactive]]

Should I just check this into trunk, or do we have a branch for 3.0.x now?  

Comment 6

[Axel Hecht [:Pike]]

cvs trunk is where we're going to work on 3.0.x, we're not going to branch for that at all. So yes, trunk it is.

Comment 7

[Hubert Gajewski]

Checked in.

Comment 8

[Hubert Gajewski]

Verified on Mozilla/5.0 (X11; U; Linux i686; pl; rv:1.9.0.1) Gecko/2008070206 Firefox/3.0.1