Skip to main content

Bug Tracker

Side navigation

#3162 closed bug (invalid)

Opened July 15, 2008 07:19PM UTC

Closed January 20, 2009 03:04AM UTC

Typo in clone function causes error in IE

Reported by: hjess Owned by: flesler
Priority: major Milestone: 1.3
Component: core Version: 1.2.6
Keywords: Cc:
Blocked by: Blocking:
Description

After "ret" is assigned as the result of "this.map", events are processed according to "this", rather than to "ret".

Attachments (1)
  • jquery.diff (0.8 KB) - added by hjess July 15, 2008 07:20PM UTC.

    diff file

Change History (4)

Changed July 22, 2008 01:26AM UTC by flesler comment:1

need: ReviewTest Case

Eh.. I'm not sure that's wrong. I really can't follow that piece of code completely.

Can you provide a test case that shows the situation where this causes an error ?

Just a simple html file with the needed js.

Thanks

Changed July 27, 2008 09:19AM UTC by zimbatm comment:2

After "ret" is assigned as the result of "this.map", events are processed according to "this", rather than to "ret".

Yes, because events are copied from this to the cloned elements. This is why this and it's childs must be visited.

Changed August 03, 2008 03:04AM UTC by flesler comment:3

owner: → flesler
status: newassigned

Will check this out more closely when I get some time.

Changed January 20, 2009 03:04AM UTC by dmethvin comment:4

resolution: → invalid
status: assignedclosed

It looks correct as-is; "this" is the original elements and clone is the cloned elements.