Uhh, seems not to be this easy! Forget the patch! 
The scroller gets destructed and the table becomes unusable after the CellEvent 
was fired and disposed.

Greetings,
Andreas

-----Ursprüngliche Nachricht-----
Von: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] Im Auftrag von Fink, Andreas
Gesendet: Freitag, 6. Juni 2008 14:56
An: qooxdoo Development
Betreff: Re: [qooxdoo-devel] destruct() question

Ok, recognized :) here is a patch for qx.ui.table.pane.CellEvent:

Index: CellEvent.js
===================================================================
--- CellEvent.js        (revision 13880)
+++ CellEvent.js        (working copy)
@@ -112,5 +112,17 @@
       }
       return this._column;
     }
+  },
+  
+  /*
+  *****************************************************************************
+     DESTRUCTOR
+  *****************************************************************************
+  */
+
+  destruct : function()
+  {
+       this._disposeFields("_row", "_column");
+       this._disposeObjects("_scroller");
   }
 });

Greetings,
Andreas

-----Ursprüngliche Nachricht-----
Von: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] Im Auftrag von thron7
Gesendet: Freitag, 6. Juni 2008 14:09
An: qooxdoo Development
Betreff: Re: [qooxdoo-devel] destruct() question

Ah, you guys got us with our pants down :). Yes, these are actually 
leaks in the framework. I've fixed those from the log output in 
legacy_0_7_x (but maybe there are more). If you check your applications 
and come across further 'Missing' entries that pertain to framework 
classes, do file them (either through mailing list or bugzilla), this is 
always helpful for us. Of course, patches are welcome.

I have also expanded the destructor wiki page a little.

Cheers,
Thomas



-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
_______________________________________________
qooxdoo-devel mailing list
qooxdoo-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel

-------------------------------------------------------------------------
Check out the new SourceForge.net Marketplace.
It's the best place to buy or sell services for
just about anything Open Source.
http://sourceforge.net/services/buy/index.php
_______________________________________________
qooxdoo-devel mailing list
qooxdoo-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/qooxdoo-devel

Reply via email to