Fix #7579 #124

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
2 participants
@dmethvin
Member

dmethvin commented Dec 9, 2010

There's a performance boost to this as well, we only enter the try/catch when the string looks like an object or array.

Only treat data- strings starting with '[' or '{' as potentially JSON…
…; return everything else as string. Fixes #7579.
@jeresig

This comment has been minimized.

Show comment
Hide comment
@jeresig

jeresig Dec 9, 2010

Member

Uhhh... what? This is a major regression! As mentioned in other tickets - if you want the actual precise value just do: .attr("data-version").

Member

jeresig commented Dec 9, 2010

Uhhh... what? This is a major regression! As mentioned in other tickets - if you want the actual precise value just do: .attr("data-version").

This issue was closed.

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