Closed Bug 318560 Opened 19 years ago Closed 19 years ago

replace javascript on Firefox Central to point to locally hosted searchplugins

Categories

(www.mozilla.org :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: beltzner, Unassigned)

References

()

Details

(Whiteboard: [mozilla.com])

Replace the addEngines function on www.mozilla.com/firefox/central to point to the searchPlugins that are locally hosted on AMO. This is the new code that should be used. function addEngine(name,ext,cat,type) { if ((typeof window.sidebar == "object") && (typeof window.sidebar.addSearchEngine == "function")) { window.sidebar.addSearchEngine( "http://addons.mozilla.org/search-engines-static/"+name+".src", "http://addons.mozilla.org/search-engines-static/"+name+"."+ext, name, cat ); } else { errorMsg(name,ext,cat); } }
thanks Mike
*** Bug 318562 has been marked as a duplicate of this bug. ***
function judgePopUp(id,type) can go entirely and function errorMsg(name,ext,cat) { alert("Netscape 6 or Mozilla is needed to install a sherlock plugin"); } would be fine (though might be better updated ;) ) function addEngine(name,ext,cat,type) is correct in the opening comment
I've checked in the changes to addEngine() and removed judgePopUp() Please test.
Status: NEW → ASSIGNED
looks fine to me and installed all 4 satisfactorily you could edit the errorMsg() function as described in comment #4 but I wouldn't worry about it too much.
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
Whiteboard: [mozilla.com]
*** Bug 259863 has been marked as a duplicate of this bug. ***
Product: mozilla.org → Websites
Component: www.mozilla.org → General
Product: Websites → www.mozilla.org
You need to log in before you can comment on or make changes to this bug.