Closed
Bug 583530
Opened 14 years ago
Closed 14 years ago
Typo in the restartToEnable entity in extensions.properties breaks the details pane
Categories
(Toolkit :: Add-ons Manager, defect)
Toolkit
Add-ons Manager
Tracking
()
VERIFIED
FIXED
mozilla2.0b3
Tracking | Status | |
---|---|---|
blocking2.0 | --- | final+ |
People
(Reporter: Unfocused, Assigned: Unfocused)
Details
Attachments
(1 file, 1 obsolete file)
1.61 KB,
patch
|
mossop
:
review+
|
Details | Diff | Splinter Review |
There's a typo in the first formatting token in the restartToEnable entity in extension.properties - which, when used, breaks the details pane. Specifically, %1$% should be %1$S
Assignee | ||
Comment 1•14 years ago
|
||
Assignee: nobody → bmcbride
Status: NEW → ASSIGNED
Attachment #461842 -
Flags: review?(robert.bugzilla)
Comment 2•14 years ago
|
||
Comment on attachment 461842 [details] [diff] [review] Patch ># HG changeset patch ># User Blair McBride <bmcbride@mozilla.com> ># Date 1280623364 -43200 ># Node ID 05004702a96b70996a79647fa8dee8b7e33bc5c1 ># Parent 823e3f0ac5470efae9616237ecf72c4c1abea120 >Bug 583530 - Typo in the restartToEnable entity in extensions.properties breaks the details pane > >diff --git a/toolkit/locales/en-US/chrome/mozapps/extensions/extensions.properties b/toolkit/locales/en-US/chrome/mozapps/extensions/extensions.properties >--- a/toolkit/locales/en-US/chrome/mozapps/extensions/extensions.properties >+++ b/toolkit/locales/en-US/chrome/mozapps/extensions/extensions.properties >@@ -45,7 +45,7 @@ installFailed=Error installing > installCancelled=Install cancelled > > #LOCALIZATION NOTE: %1$S is the addon name, %2$S is brand name >-restartToEnable=%1$% will be enabled after you restart %2$S >+restartToEnable=%1$S will be enabled after you restart %2$S For l10n to pick up this change you need to change restartToEnable to a new name. :(
Attachment #461842 -
Flags: review?(robert.bugzilla) → review-
Comment 3•14 years ago
|
||
Remember that this will probably have l10n effect... There could be localization teams that have already translated the string, but have not noticed the faulty token... They will not catch on to this change unless 1) you change the key of the string (restartToEnable -> restartToEnable2) or 2) you announce this change in mozilla.dev.l10n
Assignee | ||
Comment 4•14 years ago
|
||
Grr, stupid typos.
Attachment #461842 -
Attachment is obsolete: true
Attachment #461844 -
Flags: review?(robert.bugzilla)
Comment 5•14 years ago
|
||
even if you change restartToEnable to restartToEnable2, an announcement of the change to moz.dev.l10n is good practise - especially when it's a technical change and not a re-wording...
Assignee | ||
Comment 6•14 years ago
|
||
(In reply to comment #3) > 1) you change the key of the string (restartToEnable -> restartToEnable2) I did this. Surprised no one (including myself) picked up on this until now. Thanks go to Søren for finding it. (In reply to comment #5) > even if you change restartToEnable to restartToEnable2, an announcement of the > change to moz.dev.l10n is good practise - especially when it's a technical > change and not a re-wording... Ok, will do that after this lands.
Assignee | ||
Comment 7•14 years ago
|
||
Given this breaks the details pane in a certain situation, would like to see this blocking (I know how late this is!). Note that it is kinda edge-casey though: * Disable an enabled extension (not a restartless extension) * Go to details pane of that extension (before restarting)
blocking2.0: --- → ?
Comment 8•14 years ago
|
||
(In reply to comment #6) > (In reply to comment #3) > > 1) you change the key of the string (restartToEnable -> restartToEnable2) > > I did this. > > Surprised no one (including myself) picked up on this until now. Thanks go to > Søren for finding it. in fairness; kudos should go to Jesper Kristensen/dikrib, fellow member of the Danish Localization team, who alerted me of this earlier this evening... credit, where credit is due... ;-D
Updated•14 years ago
|
Attachment #461844 -
Flags: review?(robert.bugzilla) → review+
Updated•14 years ago
|
blocking2.0: ? → final+
Assignee | ||
Updated•14 years ago
|
Keywords: checkin-needed
Comment 9•14 years ago
|
||
Landed: http://hg.mozilla.org/mozilla-central/rev/b91faba6e48f
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla2.0b3
Comment 10•14 years ago
|
||
Verified fixed by landing.
Status: RESOLVED → VERIFIED
Flags: in-testsuite-
Flags: in-litmus-
You need to log in
before you can comment on or make changes to this bug.
Description
•