PDA

View Full Version : [FIXED-193][3.0.0] RowEditor tooltip event listeners not removed



DaveX
26 Aug 2009, 5:30 PM
It appears that the RowEditor tooltip monitors list (this.mon) grows without bound.

To reproduce the problem run the RowEditor sample application with firebug.
http://extjs.com/deploy/dev/examples/grid/row-editor.html

Set a breakpoint at RowEditor.showTooltip() line 460.

t.initTarget(this.items.last().getEl());

Edit a row and enter some invalid data to hit the breakpoint. Step over the line and examine t.mons, there are three events (mouseover, mouseout and mouseover) as expected. Now disable the break point and edit several fields each with invalid data, then reenable the break point and look at the size of the t.mons array. There are many (in my case hundreds) of events in the list. It appears in the RowEditor case that t.initTarget is not removing any existing target event listeners when initTarget is called. The application functions correctly, I just noticed this while stepping through some code.


Dave

evant
27 Aug 2009, 12:43 AM
Confirmed. Fix applied to svn in rev #5226 for patch release 3.0.2.