@refinitiv-ui/efx-grid 6.0.57 → 6.0.58

package/lib/core/es6/data/DataTable.js

@@ -126,6 +126,7 @@ DataTable.prototype.dispose = function() {
 
 	this._compMap = null; // Release user function that may be bound
 	this._clsSource = null;
+	DataTable._removeIndexArray();
 	if(this._segments) {
 		this._segments.dispose();
 		this._segments = null;
@@ -1939,6 +1940,13 @@ DataTable._createIndexArray = function(len) {
 	}
 	return ary.slice(); // Fastest way to clone an array is to perform slice() method
 };
+/** Remove _idxAryMap and _idAryCount
+* @private
+*/
+DataTable._removeIndexArray = function() {
+	DataTable._idxAryMap = {};
+	DataTable._idxAryCount = 0;
+};
 /** @private
 * @type {Object.<number, Array.<number>>}
 */

package/lib/core/es6/grid/Core.js

@@ -562,7 +562,7 @@ Core.prototype._batches = null;
 * @return {string}
 */
 Core.getVersion = function () {
-	return "5.1.72";
+	return "5.1.73";
 };
 /** {@link ElementWrapper#dispose}
 * @override
@@ -617,6 +617,8 @@ Core.prototype.dispose = function () {
 	this._columnBoundConflator.dispose();
 	this._columnPositionConflator.dispose();
 	this._rowPositionConflator.dispose();
+	this._layoutX.dispose();
+	this._layoutY.dispose();
 
 
 	// Clean Top node

package/lib/core/es6/grid/event/EventListeners.js

@@ -83,6 +83,9 @@ EventListeners.prototype.dispatch = function (eventArg) {
 		this._listeners[i](eventArg);
 	}
 
+	if(this._sender !== null) {
+		eventArg["sender"] = null; // Clear ref variable to prevent memory leak
+	}
 	this._dispatching = false;
 };
 /** @public

package/lib/core/es6/grid/util/TrackLayout.d.ts

@@ -6,6 +6,8 @@ declare class TrackLayout {
 
     public hitTest(val: number): number;
 
+    public dispose(): void;
+
     public isAtDefaultSize(opt_index?: number|null): boolean;
 
     public isHomogeneous(): boolean;

package/lib/core/es6/grid/util/TrackLayout.js

@@ -76,6 +76,14 @@ TrackLayout.prototype.hitTest = function (val) {
 	return hitIndex;
 };
 
+/**
+* @public
+*/
+TrackLayout.prototype.dispose = function () {
+	// TODO: Column dragging extension refers to this variable, so the reference instance should be deleted instead of set a new pointer
+	this._cols = this._ends = null;
+};
+
 /** If optional index is not given, the entire track is taken into account
 * @public
 * @param {number=} opt_index

package/lib/grid/index.js

@@ -1,3 +1,3 @@
 import {Grid} from "./lib/efx-grid.js";
 export {Grid}
-window.EFX_GRID = { version: "6.0.57" };
+window.EFX_GRID = { version: "6.0.58" };

package/lib/rt-grid/dist/rt-grid.js

@@ -8227,6 +8227,9 @@ EventListeners.prototype.dispatch = function (eventArg) {
 		this._listeners[i](eventArg);
 	}
 
+	if(this._sender !== null) {
+		eventArg["sender"] = null; // Clear ref variable to prevent memory leak
+	}
 	this._dispatching = false;
 };
 /** @public
@@ -11299,6 +11302,7 @@ DataTable.prototype.dispose = function() {
 
 	this._compMap = null; // Release user function that may be bound
 	this._clsSource = null;
+	DataTable._removeIndexArray();
 	if(this._segments) {
 		this._segments.dispose();
 		this._segments = null;
@@ -13112,6 +13116,13 @@ DataTable._createIndexArray = function(len) {
 	}
 	return ary.slice(); // Fastest way to clone an array is to perform slice() method
 };
+/** Remove _idxAryMap and _idAryCount
+* @private
+*/
+DataTable._removeIndexArray = function() {
+	DataTable._idxAryMap = {};
+	DataTable._idxAryCount = 0;
+};
 /** @private
 * @type {Object.<number, Array.<number>>}
 */
@@ -18984,6 +18995,14 @@ TrackLayout.prototype.hitTest = function (val) {
 	return hitIndex;
 };
 
+/**
+* @public
+*/
+TrackLayout.prototype.dispose = function () {
+	// TODO: Column dragging extension refers to this variable, so the reference instance should be deleted instead of set a new pointer
+	this._cols = this._ends = null;
+};
+
 /** If optional index is not given, the entire track is taken into account
 * @public
 * @param {number=} opt_index
@@ -36127,7 +36146,7 @@ Core.prototype._batches = null;
 * @return {string}
 */
 Core.getVersion = function () {
-	return "5.1.71";
+	return "5.1.73";
 };
 /** {@link ElementWrapper#dispose}
 * @override
@@ -36182,6 +36201,8 @@ Core.prototype.dispose = function () {
 	this._columnBoundConflator.dispose();
 	this._columnPositionConflator.dispose();
 	this._rowPositionConflator.dispose();
+	this._layoutX.dispose();
+	this._layoutY.dispose();
 
 
 	// Clean Top node
@@ -40010,18 +40031,21 @@ Core.prototype.getValidColumnList = function (colIds, columnMap) {
 };
 
 /** @public
-* @description Create mapping object from an array of strings
-* @param {Array.<string>=} colIds
+* @description Create mapping object from an array of strings or column indices
+* @param {Array.<string|number>=} colRefs
 * @return {!Object} Column mapping object
 */
-Core.prototype.createColumnMap = function (colIds) {
-	if(!colIds){
-		colIds = this.getColumnIds();
+Core.prototype.createColumnMap = function (colRefs) {
+	if(!colRefs){
+		colRefs = this.getColumnIds();
 	}
 	var mappingObj = {};
-	var count = colIds.length;
+	var count = colRefs.length;
 	for(var i = 0; i < count; i++){
-		var colId = colIds[i];
+		var colId = colRefs[i];
+		if(typeof colId ===  "number"){
+			colId = this.getColumnId(colId);
+		}
 		if(colId){
 			mappingObj[colId] = true;
 		}
@@ -44775,6 +44799,14 @@ Grid.prototype.dispose = function() {
 	this._grid.dispose();
 	this._connector.reset();
 
+	if(!this._sharedDataSource) { // make sure that it is the final grid, and that it will be dispose data
+		if(this._dt) {
+			this._dt.dispose();
+		}
+		if(this._dv) {
+			this._dv.dispose();
+		}
+	}
 	this._mainGrid = this._dt = this._dv = null;
 
 	if(this._subs) {