394592 - Extension Update URL broken when Extension is hosted on AMO

Status: RESOLVED FIXED

(Mozilla Localizations :: ro / Romanian, defect)

Description

[Carsten Book [:Tomcat]]

http://mxr.mozilla.org/l10n-mozilla1.8/source/ro/toolkit/chrome/mozapps/extensions/extensions.properties#34 points to https://update.mozilla.org and this should be addons.mozilla.org

This cause errors like Eroare: channel.securityInfo has no properties
Fișier sursă: file:///C:/Program%20Files/Mozilla%20Firefox/components/nsExtensionManager.js
Rândul: 192

and 

Eroare: niciun element găsit
Fișier sursă: http://www.mozilla.com/?reqVersion=1&id=%7b5d1ffc76-4f99-11dc-8314-0800200c9a66%7d&version=1.00&maxAppVersion=2.0.0.*&appID=%7bec8030f7-c20a-464f-9b0e-13a3a9e97384%7d&appVersion=2.0.0.7pre&appOS=WINNT&appABI=x86-msvc
Rândul: 1, coloana: 1
Cod sursă:
^

in the Error Console while Checking extensions for Updates that are hosted on AMO

Reproduced with Mozilla/5.0 (Windows; U; Windows NT 5.1; ro; rv:1.8.1.7pre) Gecko/20070831 BonEcho/2.0.0.7pre and on Linux.

Also with the 2.0.0.6 Release ro Build.

Comment 1

[Carsten Book [:Tomcat]]

also a problem:
line 35 and line 36  extensions.getMoreExtensionsURL=https://update.mozilla.org/extensions/?application=%APPID%
extensions.getMoreThemesURL=https://update.mozilla.org/themes/?application=%APPID%

Comment 2

[Axel Hecht [:Pike]]

This blocks un-betaing. Thanks to Carsten for finding this.

Comment 3

[Alexandru Szasz]

Attachment: Patch to replace update.mozilla.org with addons.mozilla.org

Don't know which approval needs to be set, if you could help here Axel, please.

Comment 4

[Axel Hecht [:Pike]]

Comment on attachment 279487 [details] [diff] [review]
Patch to replace update.mozilla.org with addons.mozilla.org

Sorry, this should be the same thing as in en-US,

http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla%2Ftoolkit%2Flocales%2Fen-US%2Fchrome%2Fmozapps%2Fextensions%2Fextensions.properties&rev=&cvsroot=%2Fcvsroot&mark=42-44#42

For landing, you want to request approval1.8.1.7 by setting that flag to '?'

Comment 5

[Alexandru Szasz]

Attachment: Patch to replace update.mozilla.org with addons.mozilla.org

Comment 6

[Alexandru Szasz]

Patch updated. requested approval

Comment 7

[Axel Hecht [:Pike]]

Comment on attachment 280724 [details] [diff] [review]
Patch to replace update.mozilla.org with addons.mozilla.org

>Index: extensions.properties
>+extensions.getMorePluginsURL=https://%LOCALE%.add-ons.mozilla.com/%LOCALE%/%APP%/%VERSION%/plugins/

Please don't add extensions.getMorePluginsURL, that's only used on the trunk, but not on the branch.

With that fixed, a=me for landing on the branch. Please use the fixed1.8.1.7 and verified1.8.1.7 keywords to track landing and testing, resp.

Comment 8

[Alexandru Szasz]

Removed extensions.getMorePluginsURL, added to cvs, added keywords.

Comment 9

[Carsten Book [:Tomcat]]

verified fixed on Windows and Mac. The Code is fixed now.

Adding verified keyword.

Comment 10

[Daniel Veditz [:dveditz]]

1.8.1.7 is a firedrill coming off a side branch. Stuff landed on the 1.8 branch will end up in 1.8.1.8