@adaptabletools/adaptable 23.0.4-canary.0 → 23.0.5-canary.0

package/src/Api/Implementation/ColumnFilterApiImpl.js

@@ -135,13 +135,6 @@ export class ColumnFilterApiImpl extends ApiBase {
         const columnFilter = this.getColumnFilterForColumn(columnId);
         return columnFilter ? this.isColumnFilterActive(columnFilter) : false;
     }
-    /**
-     * Checks if a filter is applied. It is applied when:
-     * - has no inputs
-     * - has inputs and inputs have values
-     *
-     * @param filter
-     */
     isColumnFilterActive(filter) {
         if (!filter || !filter.Predicates?.length) {
             return false;
@@ -152,7 +145,6 @@ export class ColumnFilterApiImpl extends ApiBase {
         return filter.Predicates.some((predicate) => {
             const predicateDef = this.getPredicateApi().getPredicateDefById(predicate.PredicateId);
             if (ArrayExtensions.IsNull(predicateDef.inputs) &&
-                // In and NotIn predicates ave no inputs in the definition
                 !this.getPredicateApi().internalApi.IsInOrNotInPredicate(predicate)) {
                 return true;
             }
@@ -160,7 +152,6 @@ export class ColumnFilterApiImpl extends ApiBase {
             if (ArrayExtensions.IsEmpty(inputs)) {
                 return false;
             }
-            // all values in the array need to be non-empty (e.g. between has two values, [min, max])
             const allInputsHaveValues = inputs.every((value) => {
                 return value !== undefined && value !== null && value !== '';
             });

package/src/Api/Implementation/ColumnScopeApiImpl.js

@@ -7,7 +7,6 @@ export class ColumnScopeApiImpl extends ApiBase {
             return false;
         }
         if ('All' in scope) {
-            // do we need a true check here?
             return true;
         }
         if ('DataTypes' in scope) {
@@ -47,7 +46,6 @@ export class ColumnScopeApiImpl extends ApiBase {
             return 'Nothing';
         }
         if ('All' in scope) {
-            // do we need a true check here?
             return 'Columns: All';
         }
         if ('ColumnTypes' in scope) {
@@ -169,19 +167,15 @@ export class ColumnScopeApiImpl extends ApiBase {
         return undefined;
     }
     isColumnInNumericScope(column, scope) {
-        // if column is not even numeric then return false
         if (column == null || column == undefined || column.dataType !== 'number') {
             return false;
         }
-        // if no scope then return false
         if (scope == undefined) {
             return false;
         }
-        // check if the scope has ColumnIds and whether this column is contained
         if ('ColumnIds' in scope && scope.ColumnIds.includes(column.columnId)) {
             return true;
         }
-        // check if the scope has ColumnIds and whether this column is contained
         if ('DataTypes' in scope && scope.DataTypes.includes('number')) {
             return true;
         }
@@ -192,19 +186,15 @@ export class ColumnScopeApiImpl extends ApiBase {
         return false;
     }
     isColumnInTextScope(column, scope) {
-        // if column is not even string then return false
         if (column == null || column == undefined || column.dataType !== 'text') {
             return false;
         }
-        // if no scope then return false
         if (scope == undefined) {
             return false;
         }
-        // check if the scope has ColumnIds and whether this column is contained
         if ('ColumnIds' in scope && scope.ColumnIds.includes(column.columnId)) {
             return true;
         }
-        // check if the scope has ColumnIds and whether this column is contained
         if ('DataTypes' in scope && scope.DataTypes.includes('text')) {
             return true;
         }
@@ -215,19 +205,15 @@ export class ColumnScopeApiImpl extends ApiBase {
         return false;
     }
     isColumnInDateScope(column, scope) {
-        // if column is not even numeric then return false
         if (column == null || column == undefined || column.dataType !== 'date') {
             return false;
         }
-        // if no scope then return false
         if (scope == undefined) {
             return false;
         }
-        // check if the scope has ColumnIds and whether this column is contained
         if ('ColumnIds' in scope && scope.ColumnIds.includes(column.columnId)) {
             return true;
         }
-        // check if the scope has ColumnIds and whether this column is contained
         if ('DataTypes' in scope && scope.DataTypes.includes('date')) {
             return true;
         }
@@ -238,8 +224,6 @@ export class ColumnScopeApiImpl extends ApiBase {
         return false;
     }
     isScopeInScope(a, b) {
-        // AFL: Scope:'All' for object 'a' is NOT handled
-        // this is probably wrong, but we will touch it when a bug is filed
         if ('All' in b) {
             return true;
         }
@@ -265,11 +249,9 @@ export class ColumnScopeApiImpl extends ApiBase {
             a.ColumnTypes.every((columnType) => b.ColumnTypes.includes(columnType))) {
             return true;
         }
-        // check if each colum in a has column types in b
         if ('ColumnIds' in a && 'ColumnTypes' in b) {
             const inScope = a.ColumnIds.every((columnId) => {
                 const column = this.getColumnApi().getColumnWithColumnId(columnId);
-                // check if at least one column-type is in scope
                 return column.columnTypes.some((columnColumnType) => b.ColumnTypes.includes(columnColumnType));
             });
             if (inScope) {
@@ -280,9 +262,6 @@ export class ColumnScopeApiImpl extends ApiBase {
     }
     createCellColorRangesForScope(scope) {
         const distinctColumnsValues = this.getGridApi().internalApi.getDistinctRawValuesForColumn(this.getSingleColumnInScope(scope));
-        // One neutral band by default — same gray as RangesComponent "Add New Range".
-        // Users who want a diverging negative/positive scale should use Zero Centred
-        // (Gradient) or add/split ranges manually.
         if (this.distinctColumnValuesAreEmpty(distinctColumnsValues)) {
             return [
                 {

package/src/Api/Implementation/DashboardApiImpl.js

@@ -123,7 +123,6 @@ export class DashboardApiImpl extends ApiBase {
         }
     }
     getCurrentToolbars() {
-        // if hidden or collapsed return fasle
         if (this.getDashboardState().IsCollapsed == true || this.getDashboardState().IsHidden == true) {
             return [];
         }

package/src/Api/Implementation/EntitlementApiImpl.js

@@ -32,7 +32,6 @@ export class EntitlementApiImpl extends ApiBase {
                     }
                 }
             }
-            // if still here then return their default Access Level
             return defaultAccessLevel;
         }
         return ACCESS_LEVEL_FULL;

package/src/Api/Implementation/ExportApiImpl.js

@@ -24,7 +24,6 @@ export class ExportApiImpl extends ApiBase {
         return this.getReportByName(reportName);
     }
     getReportByName(reportName) {
-        // if system report then create
         if (this.internalApi.isSystemReport(reportName)) {
             return this.internalApi.createSystemReport(reportName);
         }
@@ -64,11 +63,9 @@ export class ExportApiImpl extends ApiBase {
             })
             : systemReportFormatsOption;
         return reportFormats.filter((format) => {
-            // JSON format doesn't support Master Detail Grid
             if (this.getGridApi().isMasterDetailGrid()) {
                 return format !== 'JSON';
             }
-            // PivotLayout & AllData can't be exported as VisualExcel
             if (this.getExportApi().getCurrentReportName() === ALL_DATA_REPORT &&
                 this.getLayoutApi().isCurrentLayoutPivot()) {
                 return format !== 'VisualExcel';
@@ -113,7 +110,6 @@ export class ExportApiImpl extends ApiBase {
         return destinationItems;
     }
     getAllFormats() {
-        // FIXME AFL: add here custom formats when available
         return this.getAvailableSystemFormats();
     }
     getAllReports() {
@@ -213,7 +209,6 @@ export class ExportApiImpl extends ApiBase {
         return this.getExportApi()
             .getAllExportDestinations()
             .filter((destination) => {
-            // can NOT export BLOBs to Clipboard
             if (reportFormat === 'Excel' || reportFormat === 'VisualExcel') {
                 return destination !== 'Clipboard';
             }
@@ -227,12 +222,10 @@ export class ExportApiImpl extends ApiBase {
         }
         this.logInfo(`Start Export of ${reportName} in format ${format} to ${destination}`);
         const processExportResult = await this.processExport(report, format, destination);
-        // if FALSE, cancel export
         if (!processExportResult) {
             this.logInfo(`Cancel Export of ${reportName}`);
             return;
         }
-        // if TRUE, continue with standard export
         if (processExportResult === true) {
             const exportedReport = await this._adaptable.agGridExportAdapter.exportData({
                 report,
@@ -242,13 +235,11 @@ export class ExportApiImpl extends ApiBase {
                 customExportParams: exportConfig?.exportParams,
             });
             if (!exportedReport) {
-                // for destination 'Download' and format 'Excel', 'VisualExcel' or 'Csv', AG Grid handles the download as well
                 return;
             }
             this.internalApi.sendReportToDestination(exportedReport, report, format, destination);
         }
         else {
-            // in this case the user has returned a custom ExportResultData object
             const userProcessedExportResult = processExportResult;
             if (!userProcessedExportResult) {
                 this.logWarn(`Processing of Export ${reportName} to ${destination} returned null or undefined`);
@@ -266,7 +257,6 @@ export class ExportApiImpl extends ApiBase {
         this.logInfo(`Start getting Report Data for ${reportName} in format ${format}`);
         const processExportResult = await this.processExport(report, format, 'Clipboard');
         let reportData;
-        // handle Export processing
         if (typeof processExportResult?.type === 'string') {
             reportData = processExportResult;
         }

package/src/Api/Implementation/ExpressionApiImpl.js

@@ -61,7 +61,6 @@ export class ExpressionApiImpl extends ApiBase {
             return Array.from(new Set(columnIds));
         }
         catch (error) {
-            // defensive programing, this should happen only if someone botched Initial Adaptable State
             this.logWarn(`QueryApi.getColumnsFromExpressionInvalid received an invalid expression: ${expression}
                   Error: ${error}`);
             return [];

package/src/Api/Implementation/FormatColumnApiImpl.js

@@ -31,9 +31,6 @@ export class FormatColumnApiImpl extends ApiBase {
             return undefined;
         }
         const formatColumn = formatColumns.find((fc) => fc.DisplayFormat != undefined);
-        // Resolve numeric preset names (e.g. 'Dollar') into the concrete
-        // AdaptableFormat object so external callers always receive a fully
-        // structured format.
         return formatColumn ? resolveDisplayFormat(formatColumn.DisplayFormat) : undefined;
     }
     getSuspendedFormatColumns(config) {

package/src/Api/Implementation/FreeTextColumnApiImpl.js

@@ -52,7 +52,6 @@ export class FreeTextColumnApiImpl extends ApiBase {
                     })
                     : [];
                 storedValues.forEach((sv) => {
-                    // do summat
                     const isExistingPKValue = existingPKValues.find((epk) => epk == sv.PrimaryKey);
                     if (isExistingPKValue) {
                         if (replaceAction == 'Conflicting') {
@@ -66,7 +65,6 @@ export class FreeTextColumnApiImpl extends ApiBase {
                         }
                     }
                     else {
-                        // no existing value so just add it
                         freeTextColumn.FreeTextStoredValues.push(sv);
                     }
                 });

package/src/Api/Implementation/GridApiImpl.js

@@ -22,10 +22,6 @@ export class GridApiImpl extends ApiBase {
         return this.getAdaptableState().Internal;
     }
     loadGridData(dataSource) {
-        // we intentionally set an initial empty array,
-        // to prevent `cellchanged` events from being triggered for rows that have
-        // the same primary key with existing rows
-        // see #testprevent_cellchanged_event_on_load_grid_data
         this._adaptable.agGridAdapter.setGridOption('rowData', []);
         this._adaptable.setGridData(dataSource);
         const allRowNodes = this.getAllRowNodes();
@@ -102,11 +98,10 @@ export class GridApiImpl extends ApiBase {
         return rowNodes;
     }
     undoCellEdit(cellDataChangedInfo) {
-        // for the reason of this hacky solution see the comments in DataService
         this.getAdaptableInternalApi().getDataService().logUndoChange(cellDataChangedInfo);
         const cellUpdateRequest = {
             columnId: cellDataChangedInfo.column.columnId,
-            newValue: cellDataChangedInfo.oldValue, // need old value as we are undoing
+            newValue: cellDataChangedInfo.oldValue,
             primaryKeyValue: cellDataChangedInfo.primaryKeyValue,
             rowNode: cellDataChangedInfo.rowNode,
         };
@@ -169,7 +164,6 @@ export class GridApiImpl extends ApiBase {
         this._adaptable.applyFiltering();
     }
     clearFiltering() {
-        // slightly round the houses but we have to call ColumnFilterAPI as it does it properly
         this.getColumnFilterApi().clearColumnFilters();
     }
     getColumnSorts() {
@@ -268,7 +262,6 @@ export class GridApiImpl extends ApiBase {
             }
         }
         else {
-            // include all row
             this._adaptable.forAllRowNodesDo((rowNode) => {
                 isRowNodeInQuery(rowNode) && filteredRowNodes.push(rowNode);
             });
@@ -278,29 +271,21 @@ export class GridApiImpl extends ApiBase {
         }
         let preapredGridCellRange = gridCellRange;
         if (!preapredGridCellRange) {
-            /**
-             * Select all rows and all columns
-             */
             const currentLayout = this.getLayoutApi().getCurrentLayout();
             preapredGridCellRange = {
                 columnIds: currentLayout.TableColumns,
             };
         }
-        // ranges of row with consecutive indexes
         const rowRanges = filteredRowNodes.reduce((acc, rowNode, index, collection) => {
-            // add start if there is none
             const prevRange = acc[acc.length - 1];
             if (prevRange.startNode === undefined) {
                 prevRange.startNode = rowNode;
             }
-            // add end if next is not consecutive
             const nextNode = collection[index + 1];
             if (nextNode && nextNode.rowIndex - 1 !== rowNode.rowIndex) {
                 prevRange.endNode = rowNode;
-                // start a new one
                 acc.push({});
             }
-            // if at the end close last range
             if (index === collection.length - 1) {
                 prevRange.endNode = rowNode;
             }
@@ -308,7 +293,6 @@ export class GridApiImpl extends ApiBase {
         }, [{}]);
         let preparedClearSelection = clearSelection;
         rowRanges.forEach((range) => {
-            // clear only on first range selection
             this._adaptable.selectCells(preapredGridCellRange.columnIds, range.startNode, range.endNode, preparedClearSelection);
             preparedClearSelection = false;
         });
@@ -559,7 +543,6 @@ export class GridApiImpl extends ApiBase {
         this._adaptable.redrawRows(rowNodes);
     }
     refreshGroupRowNodes() {
-        // see https://www.ag-grid.com/javascript-data-grid/client-side-row-stages/#refreshing-the-client-side-model
         this.getAgGridApi().refreshClientSideRowModel('group');
         this._adaptable.updateRowGroupsAndColumnGroupsExpandedState();
     }
@@ -567,7 +550,6 @@ export class GridApiImpl extends ApiBase {
         this._adaptable.refreshGridHeader();
     }
     isCellEditable(gridCell) {
-        // If not Grid Cell or Column then return false - GridCell.column may be undefined for cells from synthetic columns created by AG Grid (ex. autoGroup columns)
         if (!gridCell || !gridCell.column || !gridCell.rowNode) {
             return false;
         }
@@ -575,7 +557,6 @@ export class GridApiImpl extends ApiBase {
             return false;
         }
         const agGridColumn = this.getColumnApi().internalApi.getAgGridColumnForAdaptableColumn(gridCell.column.columnId);
-        // this will invoke the colDef.editable callback where we check both AdapTable and AG Grid editability
         return agGridColumn?.isCellEditable(gridCell.rowNode);
     }
     isCellEdited(gridCell) {
@@ -696,7 +677,6 @@ export class GridApiImpl extends ApiBase {
             const newColDef = newColumnDefinitions.find((c) => c.colId === colDef.colId);
             return newColDef ? { ...colDef, ...newColDef } : colDef;
         });
-        // find out new columns which are not in the current column definitions
         const currentColIds = [];
         this._adaptable.agGridAdapter.traverseColDefs(currentColDefs, (colDef) => {
             currentColIds.push(colDef.colId);
@@ -726,7 +706,6 @@ export class GridApiImpl extends ApiBase {
     }
     addAgGridColumnDefinition(newColumnDefinition) {
         const currentColDefs = this.getAgGridColumnDefs();
-        // just in case check of there is not already an existing column with the same id and eliminate it
         const sanitizedColDefs = this._adaptable.agGridAdapter
             .traverseColDefs(currentColDefs, (colDef) => {
             return colDef.colId === newColumnDefinition.colId ? null : colDef;

package/src/Api/Implementation/GridFilterApiImpl.js

@@ -26,8 +26,6 @@ export class GridFilterApiImpl extends ApiBase {
         return gridFilter.Expression;
     }
     setGridFilterExpression(filter) {
-        // A blank / nullish expression has no meaningful AST and would otherwise
-        // explode the parser inside LayoutGridFilterSet. Treat it as "clear".
         if (filter == null || filter.trim() === '') {
             this.clearGridFilter();
             return;

package/src/Api/Implementation/LayoutApiImpl.js

@@ -97,7 +97,6 @@ export class LayoutApiImpl extends ApiBase {
         const config = {
             includeLayoutNotExtendedObjects: true,
         };
-        // Alerts
         const alertExtensions = extendedLayoutInfo.Extensions.filter((e) => {
             return e.Module == 'Alert';
         });
@@ -108,7 +107,6 @@ export class LayoutApiImpl extends ApiBase {
                 this.getAlertApi().addAlertDefinition(alertDefinition);
             }
         });
-        // Custom Sorts
         const customSortExtensions = extendedLayoutInfo.Extensions.filter((e) => {
             return e.Module == 'CustomSort';
         });
@@ -119,7 +117,6 @@ export class LayoutApiImpl extends ApiBase {
                 this.getCustomSortApi().addCustomSort(customSort);
             }
         });
-        // Flashing Cells
         const flashingCellExtensions = extendedLayoutInfo.Extensions.filter((e) => {
             return e.Module == 'FlashingCell';
         });
@@ -130,7 +127,6 @@ export class LayoutApiImpl extends ApiBase {
                 this.getFlashingCellApi().addFlashingCellDefinition(flashingCellDefinition);
             }
         });
-        // FormatColumns
         const formatColumnExtensions = extendedLayoutInfo.Extensions.filter((e) => {
             return e.Module == 'FormatColumn';
         });
@@ -141,7 +137,6 @@ export class LayoutApiImpl extends ApiBase {
                 this.getFormatColumnApi().addFormatColumn(formatColumn);
             }
         });
-        // Plus Minus
         const plusMinusExtensions = extendedLayoutInfo.Extensions.filter((e) => {
             return e.Module == 'PlusMinus';
         });
@@ -152,7 +147,6 @@ export class LayoutApiImpl extends ApiBase {
                 this.getPlusMinusApi().addPlusMinusNudge(plusMinusNudge);
             }
         });
-        // Schedule
         const exportExtensions = extendedLayoutInfo.Extensions.filter((e) => e.Module === 'Export');
         exportExtensions.forEach((le) => {
             const raw = le.Object;
@@ -164,7 +158,6 @@ export class LayoutApiImpl extends ApiBase {
                 this.getExportApi().addScheduledReport(schedule);
             }
         });
-        // StyledColumn
         const styledColumnExtensions = extendedLayoutInfo.Extensions.filter((e) => {
             return e.Module == 'StyledColumn';
         });
@@ -186,7 +179,6 @@ export class LayoutApiImpl extends ApiBase {
             includeLayoutNotExtendedObjects: false,
             extendedLayoutName: layoutName,
         };
-        // cannot see a better way than to go through this module by module...
         let extensions = [];
         this.getAlertApi()
             .getAlertDefinitions(config)
@@ -229,19 +221,16 @@ export class LayoutApiImpl extends ApiBase {
         };
     }
     cloneExtendedLayout(extendedLayoutToClone, layoutName) {
-        // first clone the underlying Layout
         const clonedLayout = this.cloneLayout(extendedLayoutToClone.Layout, layoutName);
         if (clonedLayout == false) {
             return false;
         }
-        // Alerts
         const alertExtensions = extendedLayoutToClone.Extensions.filter((e) => e.Module == 'Alert');
         alertExtensions.forEach((le) => {
             const alertDef = le.Object;
             alertDef.Tags?.push(layoutName);
             this.getAlertApi().editAlertDefinition(alertDef);
         });
-        // Custom Sorts
         const customSortExtensions = extendedLayoutToClone.Extensions.filter((e) => {
             return e.Module == 'CustomSort';
         });
@@ -250,7 +239,6 @@ export class LayoutApiImpl extends ApiBase {
             customSort.Tags?.push(layoutName);
             this.getCustomSortApi().editCustomSort(customSort);
         });
-        // Flashing Cells
         const flashingCellExtensions = extendedLayoutToClone.Extensions.filter((e) => {
             return e.Module == 'FlashingCell';
         });
@@ -259,14 +247,12 @@ export class LayoutApiImpl extends ApiBase {
             flashingCellDef.Tags?.push(layoutName);
             this.getFlashingCellApi().editFlashingCellDefinition(flashingCellDef);
         });
-        // FormatColumns
         const formatColumnExtensions = extendedLayoutToClone.Extensions.filter((e) => e.Module == 'FormatColumn');
         formatColumnExtensions.forEach((le) => {
             const formatCol = le.Object;
             formatCol.Tags?.push(layoutName);
             this.getFormatColumnApi().editFormatColumn(formatCol);
         });
-        // Plus Minus
         const plusMinusExtensions = extendedLayoutToClone.Extensions.filter((e) => {
             return e.Module == 'PlusMinus';
         });
@@ -275,7 +261,6 @@ export class LayoutApiImpl extends ApiBase {
             plusMinusNudge.Tags?.push(layoutName);
             this.getPlusMinusApi().editPlusMinusNudge(plusMinusNudge);
         });
-        // Export
         const exportCloneExtensions = extendedLayoutToClone.Extensions.filter((e) => e.Module === 'Export');
         exportCloneExtensions.forEach((le) => {
             const raw = le.Object;
@@ -285,7 +270,6 @@ export class LayoutApiImpl extends ApiBase {
             schedule.Tags?.push(layoutName);
             this.getExportApi().editScheduledReport(schedule);
         });
-        // Shortcut
         const shortCutExtensions = extendedLayoutToClone.Extensions.filter((e) => {
             return e.Module == 'Shortcut';
         });
@@ -294,7 +278,6 @@ export class LayoutApiImpl extends ApiBase {
             shortcut.Tags?.push(layoutName);
             this.getShortcutApi().editShortcut(shortcut);
         });
-        // StyledColumn
         const styledColumnExtensions = extendedLayoutToClone.Extensions.filter((e) => {
             return e.Module == 'StyledColumn';
         });
@@ -303,7 +286,6 @@ export class LayoutApiImpl extends ApiBase {
             styledColumn.Tags?.push(layoutName);
             this.getStyledColumnApi().editStyledColumn(styledColumn);
         });
-        // now create the new Extended Layout with the same extensions
         const extendedLayout = {
             Layout: clonedLayout,
             Extensions: extendedLayoutToClone.Extensions,

package/src/Api/Implementation/LayoutHelpers.js

@@ -29,12 +29,9 @@ function cleanupAdaptableObjectPrimitives(layout) {
     const l = layout;
     for (const key in l) {
         if (key === 'ColumnFilters') {
-            // we don't want to cleanup ColumnFilters
-            // as we rely on the Uuids for identifying the filters
             if (Array.isArray(l[key])) {
                 l[key].forEach((item) => {
                     if (typeof item === 'object') {
-                        // but we do want to cleanup further down - eg: predicates in the filter
                         for (const k in item) {
                             if (typeof item[k] === 'object') {
                                 removeAdaptableObjectPrimitivesInlineDeep(item[k]);
@@ -50,11 +47,6 @@ function cleanupAdaptableObjectPrimitives(layout) {
         }
     }
 }
-/**
- * Compares 2 Layouts for equality
- * @param layout1 First Layout
- * @param layout2 Second Layout
- */
 export const areLayoutsEqual = (layout1, layout2) => {
     if (!layout1 && !layout2) {
         return true;
@@ -83,7 +75,6 @@ export const normalizePivotLayout = (pivotLayout) => {
     return pivotLayout;
 };
 const errorOnceMessages = new Map();
-/** Clears module-scoped dedupe state when a new Adaptable instance starts (e.g. Playwright retries). */
 export const resetLayoutHelperErrorOnceMessages = () => {
     errorOnceMessages.clear();
 };
@@ -173,7 +164,6 @@ export const checkForDuplicateColumns = (layout) => {
             layout.TableColumns.reduce((acc, col) => {
                 acc[col] = (acc[col] || 0) + 1;
                 if (acc[col] > 1) {
-                    // if you change this message, make sure you also update the associated test error-on-duplicate-column.spec.ts
                     errorOnce(`Duplicate column "${col}" in layout "${layout.Name}"!`);
                 }
                 return acc;
@@ -392,9 +382,7 @@ export const tableLayoutModelToTableLayout = (layoutModel) => {
     if (layoutModel.Ignore_OpenCharts) {
         tableLayout.OpenCharts = layoutModel.Ignore_OpenCharts;
     }
-    // if (layoutModel.RowGroupDisplayType) {
     tableLayout.RowGroupDisplayType = layoutModel.RowGroupDisplayType ?? 'single';
-    // }
     if (layoutModel.RowGroupedColumns) {
         tableLayout.RowGroupedColumns = layoutModel.RowGroupedColumns;
     }
@@ -420,7 +408,6 @@ export const tableLayoutModelToTableLayout = (layoutModel) => {
         if ((layoutModel.ColumnGroupValues.ColumnGroupDisplay === 'collapsed' ||
             layoutModel.ColumnGroupValues.ColumnGroupDisplay === 'expanded') &&
             layoutModel.ColumnGroupValues.Values) {
-            // @ts-ignore
             tableLayout.ColumnGroupValues.ExceptionGroupKeys = layoutModel.ColumnGroupValues.Values;
         }
     }
@@ -556,13 +543,10 @@ export const pivotLayoutModelToPivotLayout = (layoutModel) => {
         if ((layoutModel.ColumnGroupValues.ColumnGroupDisplay === 'collapsed' ||
             layoutModel.ColumnGroupValues.ColumnGroupDisplay === 'expanded') &&
             layoutModel.ColumnGroupValues.Values) {
-            // @ts-ignore
             pivotLayout.ColumnGroupValues.ExceptionGroupKeys = layoutModel.ColumnGroupValues.Values;
         }
     }
-    // if (layoutModel.RowGroupDisplayType) {
     pivotLayout.RowGroupDisplayType = layoutModel.RowGroupDisplayType ?? 'single';
-    // }
     if (layoutModel.RowSelection != null) {
         pivotLayout.RowSelection = layoutModel.RowSelection;
     }

package/src/Api/Implementation/NamedQueryApiImpl.js

@@ -28,7 +28,6 @@ export class NamedQueryApiImpl extends ApiBase {
         if (IsNullOrEmptyOrWhiteSpace(namedQuery?.Name)) {
             return { valid: false, message: 'A name is required for the Named Query.' };
         }
-        // check that there is no other existing named query with the same name
         const duplicate = this.getNamedQueries().find((q) => q.Name === namedQuery.Name && q.Uuid !== namedQuery.Uuid);
         if (duplicate) {
             return {

package/src/Api/Implementation/PredicateApiImpl.js

@@ -106,7 +106,6 @@ export class PredicateApiImpl extends ApiBase {
         if (!adaptablePredicateDef) {
             return false;
         }
-        // not perfect but if there are no inputs - and there should be some - we treat the filter as invalid
         if (adaptablePredicateDef.inputs) {
             if (ArrayExtensions.IsNullOrEmpty(predicate.Inputs) ||
                 StringExtensions.IsNullOrEmpty(predicate.Inputs[0])) {
@@ -126,7 +125,6 @@ export class PredicateApiImpl extends ApiBase {
         const paramsForColumn = {
             ...context,
             column: this.getColumnApi().getColumnWithColumnId(predicate.ColumnId),
-            // value: params?.node?.data?.[predicate.ColumnId],
             value: this.getGridApi().getRawValueFromRowNode(context.node, predicate.ColumnId),
         };
         if ('oldValue' in paramsForColumn) {
@@ -141,9 +139,6 @@ export class PredicateApiImpl extends ApiBase {
         if (!predicate) {
             return defaultReturn;
         }
-        // memoization to avoid repeated lookups
-        // during filtering of large datasets
-        // see #column_filter_predicates_evaluation
         let predicateDef = predicateDefMap_MEMO.get(predicate.PredicateId);
         if (predicateDef === undefined) {
             predicateDef = this.getPredicateDefById(predicate.PredicateId);

package/src/Api/Implementation/RowFormApiImpl.js

@@ -17,7 +17,6 @@ export class RowFormApiImpl extends ApiBase {
         this.dispatchAction(PopupShowForm({
             Id: 'edit_row_form',
             Form: editForm,
-            // formProps are added to the formContext
             FormProps: {
                 rowNode,
             },
@@ -40,7 +39,6 @@ export class RowFormApiImpl extends ApiBase {
         this.dispatchAction(PopupShowForm({
             Id: 'create_row_form',
             Form: createForm,
-            // formProps are added to the formContext
             FormProps: {
                 clonedRowNode: rowNode,
             },