details:   https://code.openbravo.com/erp/devel/pi/rev/f2a4f4808579
changeset: 13849:f2a4f4808579
user:      Martin Taal <martin.taal <at> openbravo.com>
date:      Wed Sep 28 07:33:21 2011 +0200
summary:   Related to issue 18227: Closing a tab doesn't release all objects
MiniDateRangeItem needs specific code as it is not a daterangeitem but a 
TextItem
Other mem leak remaining, will revisit for MP5

diffstat:

 
modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-minidaterange.js
 |  22 ++++++++++
 1 files changed, 22 insertions(+), 0 deletions(-)

diffs (32 lines):

diff -r 9705f4af28aa -r f2a4f4808579 
modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-minidaterange.js
--- 
a/modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-minidaterange.js
       Wed Sep 28 07:06:25 2011 +0200
+++ 
b/modules/org.openbravo.client.application/web/org.openbravo.client.application/js/form/formitem/ob-formitem-minidaterange.js
       Wed Sep 28 07:33:21 2011 +0200
@@ -442,6 +442,28 @@
     }
     return true;
   },
+   
+  // Explicit destroy of the rangedialog as formitems don't have 
+  // an auto delete of autochilds
+  destroy: function() {
+    this.destroying = true;
+    if (this.rangeDialog) {
+      this.rangeDialog.rangeForm.destroy();
+      this.rangeDialog.mainLayout.destroy();
+      this.rangeDialog.destroying  = true;
+      this.rangeDialog.destroy();
+      this.rangeDialog.destroying  = false;
+    }
+    this.Super('destroy', arguments);
+    this.destroying = false;
+  },
+  
+  clear: function() {
+    if (this.destroying) {
+      return;
+    }
+    this.Super('clear', arguments);
+  },
   
   formatDate: function(dt) {
     return OB.Utilities.Date.JSToOB(dt, OB.Format.date);

------------------------------------------------------------------------------
All the data continuously generated in your IT infrastructure contains a
definitive record of customers, application performance, security
threats, fraudulent activity and more. Splunk takes this data and makes
sense of it. Business sense. IT sense. Common sense.
http://p.sf.net/sfu/splunk-d2dcopy1
_______________________________________________
Openbravo-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openbravo-commits

Reply via email to