Skip to main content

Bug Tracker

Side navigation

#10817 closed enhancement (fixed)

Opened November 17, 2011 05:03PM UTC

Closed November 17, 2011 08:02PM UTC

Last modified March 02, 2012 04:29PM UTC

Change .bind() in ajax.js to .on()

Reported by: chrisr Owned by: rwaldron
Priority: low Milestone: 1.7.1
Component: misc Version: git
Keywords: Cc:
Blocked by: Blocking:

In line 260 of ajax.js there is:

return this.bind( o, f );

I believe, it should be .on() since .bind() is only wrapper for backward compatibility.

Also, there is .bind() in comment in line 70 in event.js ;)

Attachments (0)
Change History (5)

Changed November 17, 2011 05:05PM UTC by rwaldron comment:1

owner: → rwaldron
status: newassigned


Changed November 17, 2011 05:06PM UTC by rwaldron comment:2

component: unfiledmisc
priority: undecidedlow

Changed November 17, 2011 05:21PM UTC by rwaldron comment:3

Changed November 17, 2011 08:02PM UTC by Rick Waldron comment:4

resolution: → fixed
status: assignedclosed

Landing pull request 604. Updates bind to on in src/ajax.js. Fixes #10817.

More Details:

Changeset: 327bc07c54526979527f9622c99efdcc8c715f7a

Changed March 02, 2012 04:29PM UTC by dmethvin comment:5

milestone: 1.next1.7.1