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

Data: "@(undefined)" #9

Open
jamesjessian opened this issue Sep 11, 2015 · 3 comments
Open

Data: "@(undefined)" #9

jamesjessian opened this issue Sep 11, 2015 · 3 comments

Comments

@jamesjessian
Copy link

Hi, Great extension.
However, for all elements the jQuery Audit displays...
Data: "@(undefined)"
But using $($0).data() in the console shows me that the data object for this element has stuff in it.
Any ideas?

@jamesjessian jamesjessian changed the title Data: incorrect Data: "@(undefined)" Sep 11, 2015
@zertosh
Copy link
Owner

zertosh commented Sep 11, 2015

What version of jQuery? And do you have a site I can test it on?

@jamesjessian
Copy link
Author

Sorry, no, I don't have a public site to test it on at the moment, but the version of jQuery we are using is 2.1.4.
If you can't repeat the problem, I'll try and provide a public-accessible example.

@zertosh
Copy link
Owner

zertosh commented Sep 20, 2015

I looked into this. So jQuery Audit doesn't actually call $(el).data() – it does some other trickery because that method used to result in jQuery creating an empty internal data object on the first call for each node it was called on. It seems that at some point that changed, so that's no longer the case.

I don't have the bandwidth these days (I don't even use jQuery anymore) to fix this. Though, I welcome a PR that fixes via some form of version detection.

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

No branches or pull requests

2 participants