Error: formatURLPref: Couldn't get pref: extensions.getAddons.link.url

RESOLVED FIXED in seamonkey2.38

Status

SeaMonkey
Preferences
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: Philip Chee, Assigned: Philip Chee)

Tracking

Trunk
seamonkey2.38

Firefox Tracking Flags

(firefox41 affected, seamonkey2.37 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
Error: formatURLPref: Couldn't get pref: extensions.getAddons.link.url
Source file: resource://gre/components/nsURLFormatter.js
Line: 136

Fallout from Bug 1151509
(Assignee)

Comment 1

3 years ago
Created attachment 8609776 [details] [diff] [review]
Patch v1.0 add some default prefs

> +pref("extensions.getAddons.recommended.url", "https://services.addons.mozilla.org/%LOCALE%/%APP%/api/%API_VERSION%/list/recommended/all/%MAX_RESULTS%/%OS%/%VERSION%?src=seamonkey");
Bug 628785 - Firefox should define a value for extensions.getAddons.recommended.url so AddonRepoistory.retrieveRecommendedAddons() works
> (In reply to comment #3)
>> Dave, do we make use of the URL in any of our UI? Or is it a method which can
>> be used by add-ons?
> We don't use it in the UI anymore but the API is still available for add-ons.

> +pref("extensions.getAddons.link.url", "https://addons.mozilla.org/%LOCALE%/%APP%/");
Fallout from Bug 1151509. We aren't going to do addon signing, this just stops nsURLFormatter.js complaining.
Attachment #8609776 - Flags: review?(iann_bugzilla)

Updated

3 years ago
OS: Unspecified → All
Hardware: Unspecified → All

Comment 2

3 years ago
Comment on attachment 8609776 [details] [diff] [review]
Patch v1.0 add some default prefs

Nit: is it worth inserting the two new preferences into the middle of the rest of the extension.getAddons.* prefs? (between getWith... and search.*)
Attachment #8609776 - Flags: review?(iann_bugzilla) → review+
(Assignee)

Comment 3

3 years ago
> Nit: is it worth inserting the two new preferences into the middle of the
> rest of the extension.getAddons.* prefs? (between getWith... and search.*)
Will do this before check-in

Comment 4

3 years ago
Comment on attachment 8609776 [details] [diff] [review]
Patch v1.0 add some default prefs

a=me for CLOSED TREE
(Assignee)

Comment 5

3 years ago
http://hg.mozilla.org/comm-central/rev/87b9eddaf6b6
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
status-seamonkey2.37: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → seamonkey2.38

Updated

3 years ago
Blocks: 1169697

Updated

3 years ago
No longer blocks: 1169697
You need to log in before you can comment on or make changes to this bug.