midsuit
/
core-jgi
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

IDEMPIERE-5457 Billboard Charts can cause memory leaks (#1544)

This commit is contained in:
hengsin 2022-10-27 15:09:06 +08:00 committed by GitHub
parent 59b7e3c899
commit 15737d2238
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
org.idempiere.zk.billboard/src/metainfo/zk/lang-addon.xml
View File

@ -19,7 +19,7 @@
</component>
<stylesheet href="~./js/zul/billboard/css/billboard.css" type="text/css"/>
<javascript-module name="zul.billboard" version="3.5.1.20220908"/>
<javascript-module name="zul.billboard" version="3.5.1.20221027"/>
<!-- this js module doesn't actually exists and it is here for billboard.css version -->
<javascript-module name="zul.billboard.css" version="3.5.1.20220905"/>
</language-addon>

9
org.idempiere.zk.billboard/src/web/js/zul/billboard/Billboard.js
View File

@ -34,7 +34,8 @@
timeSeries: null,
timeSeriesInterval: null,
timeSeriesFormat: null,
xAxisAngle: null
xAxisAngle: null,
chart: null
},
_dataPrepare : function() {
@ -216,7 +217,7 @@
model.data["colors"] = colors;
}
}
var chart = bb.generate(model);
wgt.chart = bb.generate(model);
}
}
},
@ -237,6 +238,10 @@
unbind_ : function() {
this.$supers(Billboard, 'unbind_', arguments);
if (this.chart) {
this.chart.destroy();
this.chart = null;
}
},
doClick_ : function(event) {