289013 - ga-IE 1.0.3 locale has malformed plugin finder URL

Status: RESOLVED FIXED

(Mozilla Localizations :: Other, defect)

Description

[Brian King [:kinger]]

From l10n list:

Hasse wrote ... (file should be plugins.properties):

The pfs.datasource.url in mozapps/plugins/plugins.dtd looks malformed 
in ga-IE and it-IT. The correct URL should be
https://update.mozilla.org/plugins/PluginFinderService.php?mimetype=%
PLUGIN_MIMETYPE%&appID=%APP_ID%&appVersion=%APP_VERSION%&clientOS=%
CLIENT_OS%&chromeLocale=%CHROME_LOCALE%

ga-IE has a couple of "\n" in the URL:
https://update.mozilla.org/plugins/PluginFinderService.php?mimetype=%
\nPLUGIN_MIMETYPE%&appID=%APP_ID%&appVersion=%APP_VERSION%&clientOS=%
CLIENT_OS%\n&chromeLocale=%CHROME_LOCALE%

Comment 1

[Kevin Scannell]

Attachment: plugins.properties patch

Comment 2

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

Comment on attachment 179623 [details] [diff] [review]
plugins.properties patch

this should be an approval-l10n request, not approval-aviary1.0.3

Comment 3

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

I filed bug 289076 on making this not be localizable.

Comment 4

[Brian King [:kinger]]

Patch has been checked in.

Comment 5

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

l10n changes should not be in our blocking queue