Side navigation
#8995 closed bug (invalid)
Opened April 28, 2011 07:08AM UTC
Closed April 30, 2011 07:36PM UTC
Last modified April 30, 2011 08:53PM UTC
jQuery.map( iframejQuery() ) enumerating properties.
Reported by: | rkatic | Owned by: | |
---|---|---|---|
Priority: | low | Milestone: | 1.next |
Component: | core | Version: | 1.5.2 |
Keywords: | Cc: | ||
Blocked by: | Blocking: |
Description
jQuery.map( iframejQuery() ) enumerating properties.
Attachments (0)
Change History (9)
Changed April 28, 2011 07:40AM UTC by comment:1
Changed April 30, 2011 07:36PM UTC by comment:2
component: | unfiled → core |
---|---|
priority: | undecided → low |
resolution: | → invalid |
status: | new → closed |
Mixing versions of jQuery will break things as they should not be expected to work together. jQuery() is a different object than the jQuery you are using so it is expected that it would iterate the prototype. Here is an example using the iframe's jQuery: http://jsfiddle.net/timmywil/GJ3MH/2/
Changed April 30, 2011 07:42PM UTC by comment:3
Are you serious? That makes sense to you?
jQuery supports to use more version together from ages. Is it changed lately?
Changed April 30, 2011 07:47PM UTC by comment:4
It's not out of the ordinary. Having multiple versions of jQuery on a page is supported, but they should always be used modularly. window.jQuery can only be one thing on a page and when used within an iframe, the iframe's jQuery should be used.
Changed April 30, 2011 07:52PM UTC by comment:5
Not true. jQuery supports more versions on the same window! That is also the reason why jQuery.expando is something like "jQuery325535543552352", where it is generated in the way it will not conflict with other jQuery-es.
Are you new here?
Changed April 30, 2011 08:00PM UTC by comment:6
The purpose of the jQuery expando is to keep track of unique events and data. Yes, this means that different versions of jQuery can be attaching these items. Nevertheless, as I said, using more than one version of jQuery is fine, but the way you were mixing them has never been supported.
Changed April 30, 2011 08:04PM UTC by comment:7
timmywil, please consults yourself with John or Dave on this.
Changed April 30, 2011 08:36PM UTC by comment:8
very well. I've been wrong before. I'll have Dave look at the ticket.
Patch: https://github.com/jquery/jquery/pull/356