Closed Bug 535030 Opened 15 years ago Closed 15 years ago

$.jsonp is not a function error on Plugin Check page

Categories

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

Product:
www.mozilla.org
Component:
General
Type:
defect
Priority:
Not set
Severity:
critical

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: stephend, Assigned: ozten)

References

(
URL
)

Details

(Whiteboard: [3.6Br5][1.9.2-fixed])

Attachments

(1 file)

Fixes problem.
(deleted), patch
Details | Diff | Splinter Review
I'm using Firefox 3.5.5 with Shockwave Flash 10.0 r42, and http://www.mozilla.com/en-US/plugincheck/ is no longer working; instead, I get: Error: $.jsonp is not a function Source File: http://www.mozilla.com/js/plugincheck.js Line: 17
Whiteboard: [3.6Br5]
Disabling all plugins and the page works. Enabling any plugin and the page breaks on www-trunk. Still working to identify the root cause. My dev env works, trunk, authstage, and prod are broken.
The updates made to mozilla.com/includes/min/groupsConfig.php break our copy of jquery-jsonp.
Moving inclusion before our script appears to be a viable work-around. Committing and testing on www-trunk.
(In reply to comment #4) > Moving inclusion before our script appears to be a viable work-around. > Committing and testing on www-trunk. http://www-trunk.stage.mozilla.com/en-US/plugincheck/ now WFM, thanks!
Trying to push en-US/plugincheck/index.html via Kubla. It's failing to get to authstage on first attempt.
Probably should find another way to fix this. On trunk <script> is outside of <html> which breaks validation.
Attached patch Fixes problem. (deleted) — — Splinter Review
This is one way to fix it. Another way(probably better, but a larger change) would be to move jquery to the js group in groupsConfig.php and rip it out of plugincheck.js.
Attachment #417853 - Attachment is patch: true
Attachment #417853 - Attachment mime type: application/octet-stream → text/plain
(In reply to comment #7) Thanks for catching that. (In reply to comment #8) Yes, I want to rework the way jquery is included, but it will take longer and this page is in the middle of refactoring for L10n and web badge code. I'll back out my work and try to get it working w/o jquery inline instead of pushing a quick prod fix.
On trunk is a version that validates. It uses the $extra_footers global variable to add JavaScript after the footer. This should be a suitable quick fix. Still working on 1) no jQuery version 2) conflict free jQuery version for web badge
This is fixed in production. Keeping open to track the *real* fixes.
Flags: blocking-firefox3.6?
this is all server side, right? if so then we could remove the blocking 3.6
(In reply to comment #10) #1 done and on www-trunk. Using noConflict and grabbing jQuery into Pfs.$
(In reply to comment #10) #2 done and on www-trunk plugincheck.js will depend on mozilla.com's jquery plugincheck_badge.js will continue to include a copy of jquery.
Status: NEW → RESOLVED
Closed: 15 years ago
Resolution: --- → FIXED
Flags: blocking-firefox3.6? → blocking-firefox3.6+
Whiteboard: [3.6Br5] → [3.6Br5][1.9.2-fixed]
Component: www.mozilla.org/firefox → www.mozilla.org
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.

Attachment

General

Creator:
Created:
Updated:
Size: