[qooxdoo-commit] SF.net SVN: qooxdoo:[22094] trunk

[<mar...@us...>]

Revision: 22094
          http://qooxdoo.svn.sourceforge.net/qooxdoo/?rev=22094&view=rev
Author:   martinwittemann
Date:     2010-04-28 09:30:16 +0000 (Wed, 28 Apr 2010)

Log Message:
-----------
Minor fix to the form renderer dispose.

Modified Paths:
--------------
    trunk/.gitignore
    trunk/qooxdoo/framework/source/class/qx/ui/form/renderer/Double.js
    trunk/qooxdoo/framework/source/class/qx/ui/form/renderer/Single.js

Modified: trunk/.gitignore
===================================================================
--- trunk/.gitignore	2010-04-28 08:35:42 UTC (rev 22093)
+++ trunk/.gitignore	2010-04-28 09:30:16 UTC (rev 22094)
@@ -2,5 +2,6 @@
 qooxdoo/application/playground/test
 qooxdoo/application/performance/test
 qooxdoo/application/performance/build
+qooxdoo/application/portal/build
 qooxdoo/application/performance/source/script
 .settings/*
\ No newline at end of file

Modified: trunk/qooxdoo/framework/source/class/qx/ui/form/renderer/Double.js
===================================================================
--- trunk/qooxdoo/framework/source/class/qx/ui/form/renderer/Double.js	2010-04-28 08:35:42 UTC (rev 22093)
+++ trunk/qooxdoo/framework/source/class/qx/ui/form/renderer/Double.js	2010-04-28 09:30:16 UTC (rev 22094)
@@ -161,7 +161,9 @@
   destruct : function() {
     // first, remove all buttons from the botton row because they
     // should not be disposed
-    this._buttonRow.removeAll();
-    this._disposeObjects("_buttonRow");
+    if (this._buttonRow) {
+      this._buttonRow.removeAll();
+      this._disposeObjects("_buttonRow");      
+    }
   }
 });

Modified: trunk/qooxdoo/framework/source/class/qx/ui/form/renderer/Single.js
===================================================================
--- trunk/qooxdoo/framework/source/class/qx/ui/form/renderer/Single.js	2010-04-28 08:35:42 UTC (rev 22093)
+++ trunk/qooxdoo/framework/source/class/qx/ui/form/renderer/Single.js	2010-04-28 09:30:16 UTC (rev 22094)
@@ -157,7 +157,9 @@
   {
     // first, remove all buttons from the botton row because they
     // should not be disposed
-    this._buttonRow.removeAll();
-    this._disposeObjects("_buttonRow");
+    if (this._buttonRow) {
+      this._buttonRow.removeAll();
+      this._disposeObjects("_buttonRow");      
+    }
   }
 });


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.