@refinitiv-ui/efx-grid 6.0.57 → 6.0.59

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.74";
 };
 /** {@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/plugins/SortableTitlePlugin.js

@@ -781,9 +781,9 @@ SortableTitlePlugin.prototype.sortColumns = function (sortOptions, opt_arg) {
 		var states = [];
 		for (var i = 0; i < sortOptions.length; i++) {
 			var opt = sortOptions[i];
-			var colRef = opt["colIndex"];
-			if(colRef < 0 || colRef == null) {
-				colRef = opt["colId"] || opt["field"];
+			var colRef = opt["colId"] || opt["field"];
+			if(colRef == null) {
+				colRef = opt["colIndex"];
 			}
 			var state = this._prepareSorting(
 				colRef,

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.59" };

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) {
@@ -45735,16 +45767,16 @@ Grid.prototype.replaceColumn = function (columnOption, colRef) {
 	}
 
 	var colDef = this.getColumnDefinition(colIndex);
-	if(colDef.getChildren()) { // Parent time series field doesn't provide hidden property
+	if(colDef && colDef.getChildren().length > 0) { // Parent time series field doesn't provide hidden property
 		colConfig["hidden"] = false;
 	}
 
-	if(colConfig.id == null) {
+	if(colDef && colConfig.id == null) {
 		colConfig.id =  colDef.getId(); // retain ID
 	}
 	this._grid.startBatch("reset");
+	this.removeColumn(colIndex);
 	this.insertColumn(colConfig, colIndex);
-	this.removeColumn(colIndex + 1); // remove existing column after insert
 	this._grid.stopBatch("reset");
 };
 
@@ -46449,9 +46481,9 @@ Grid.prototype._initDuplicateRicData = function(rowDef) {
 	}
 };
 /** @public
-* @param {Object=} rowOption
+* @param {(RowDefinition~Options|string)=} rowOption
 * @param {Grid~RowReference=} rowRef Reference (i.e. row index, row id, or row definition) of the insert position
-* @returns {Object}
+* @returns {RowDefinition}
 * @example
 * var grid = new rt.Grid(grid_div, options);
 * grid.insertRow({"ric": "RIC"}, 0); // A new row is added at the top
@@ -46482,7 +46514,7 @@ Grid.prototype.insertRow = function(rowOption, rowRef) {
 	return rowDef;
 };
 /** @public
-* @param {Array.<Object>} rowOptions Array of row option object
+* @param {Array.<RowDefinition~Options|string>} rowOptions Array of row option object
 * @param {Grid~RowReference=} rowRef Reference (i.e. row index, row id, or row definition) of the insert position
 * @param {Array.<string>=} opt_fields
 * @example
@@ -48132,8 +48164,8 @@ Grid.prototype._logData = function(rowDefs, options) {
 /** @public
 * @description Replace existing row with a new row. Row ID would be changed, after row is replaced.
 * @param {Grid~RowReference} rowRef Reference (i.e. row index, row id, or row definition) of the insert position
-* @param {Object=} rowOption
-* @returns {Object}
+* @param {(RowDefinition~Options|string)=} rowOption
+* @returns {RowDefinition}
 */
 Grid.prototype.replaceRow = function(rowRef, rowOption) {
 	var rowId = this._getRowId(rowRef);