Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

jquery migrate fails on FreeBSD - ReferenceError: u is not defined #71

Closed
mike-pt opened this issue Aug 19, 2014 · 5 comments
Closed

jquery migrate fails on FreeBSD - ReferenceError: u is not defined #71

mike-pt opened this issue Aug 19, 2014 · 5 comments

Comments

@mike-pt
Copy link

mike-pt commented Aug 19, 2014

In every browser I tried I complains that "ReferenceError: u is not defined jquery-migrate-1.1.1.js:268"

$.os = {
name: (/(win|mac|linux|sunos|solaris|iphone|ipad)/.exec(navigator.platform.toLowerCase()) || [u])[0].replace('sunos', 'solaris')
};

@mike-pt
Copy link
Author

mike-pt commented Aug 20, 2014

Note, navigator.platform returns:

"FreeBSD amd64"

But looking at this it fails because it tries to use the "u" var and that var is not set (maybe its set by jquery it self not jquer-migrate.

I tested this at:
https://use.cloudshare.com/test.mvc

they seem to cal jquery 1.9 and I also tried using a Firefox add-on to embed jquery (using latest jquery version)... no luck!

@mike-pt
Copy link
Author

mike-pt commented Aug 20, 2014

Also fails on BB10 (qnx), so basically it fails when using a "navigator.platform" that's not listed here:
win|mac|linux|sunos|solaris|iphone|ipad because "u" is not defined!

@dmethvin
Copy link
Member

The latest version is 1.2.1, does it happen there as well? Where are you getting the file from? I don't see that line in our code.

@mike-pt
Copy link
Author

mike-pt commented Aug 20, 2014

No it doe not, and not that you mention it I had not tested by downloading 1.1 my self form cod.jquey!

The issue happens in the refereed site but according to them they were simply using jquery.migrate, but looking at "http://code.jquery.com/jquery-migrate-1.1.1.js" I also don't see that line!

So its not at you're side but cloudshare

This is it: https://use.cloudshare.com/resources/js/jquery-migrate-1.1.1.js the line is just after the "

// Don't clobber any existing jQuery.browser in case it's different Block!

So I guess you can close this and sorry for the noise :)

@dmethvin
Copy link
Member

That copy has definitely been modified. Seems to me that is just some random file off the Internet and not intended for use outside the site. There is definitely a downside in filling dependencies via a random choice from a Google search!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants