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

package/src/Strategy/StyledColumnModule.js

@@ -28,8 +28,7 @@ const StyledColumnSparklinePreview_1 = require("../View/StyledColumn/Wizard/Styl
 const getObjectTagsViewItems_1 = require("../Utilities/getObjectTagsViewItems");
 class StyledColumnModule extends AdaptableModuleBase_1.AdaptableModuleBase {
     constructor(api) {
-        super(ModuleConstants.StyledColumnModuleId, ModuleConstants.StyledColumnFriendlyName, 'brush', 'StyledColumnPopup', // to change
-        'Create a Special Column Style e.g Gradient, Percent Bar or Badge', api);
+        super(ModuleConstants.StyledColumnModuleId, ModuleConstants.StyledColumnFriendlyName, 'brush', 'StyledColumnPopup', 'Create a Special Column Style e.g Gradient, Percent Bar or Badge', api);
     }
     getModuleAdaptableObjects(config) {
         return this.api.styledColumnApi.getStyledColumns(config);
@@ -42,17 +41,13 @@ class StyledColumnModule extends AdaptableModuleBase_1.AdaptableModuleBase {
     }
     createColumnMenuItems(column) {
         let returnColumnMenuItems = [];
-        // Need a Column and an editable Module
         if (column && this.isModuleEditable() && !column.isTreeColumn) {
             let styledColumn = this.api.styledColumnApi.getStyledColumnForColumnId(column.columnId);
-            // First look at Sparkline Columns - needs to be editable array type
             if (this.api.columnApi.hasNumericArrayDataType(column.columnId) &&
                 column.IsReadOnly !== true &&
                 this.api.styledColumnApi.canDisplaySparklines()) {
                 let sparklineColumnExists = column.isSparkline && Boolean(styledColumn);
                 if (column.isSparkline && !sparklineColumnExists) {
-                    // do nothing as we only want to continue if there is no sparkline column or if its in state
-                    // because if just one then it was created in AG Grid and we dont want to edit it
                 }
                 else {
                     const hasSparklineStyle = styledColumn != null && styledColumn.SparklineStyle != null;
@@ -72,17 +67,13 @@ class StyledColumnModule extends AdaptableModuleBase_1.AdaptableModuleBase {
                         source: 'ColumnMenu',
                         value: styledColumn,
                         config: {
-                            // New Styled Columns must land on the Settings step so the user
-                            // can enter the (mandatory) Name. Edit jumps straight to Style.
                             defaultCurrentSectionName: hasSparklineStyle ? 'Style' : 'Settings',
                         },
                     };
                     returnColumnMenuItems.push(this.createMenuItemShowPopup(name, label + 'Sparkline Column', this.moduleInfo.Popup, this.getGlyphForStyledColumn(styledColumn), popupParam));
                 }
             }
-            // Next do non sparkline
             else if (!column.isSparkline) {
-                // Edit is common
                 if (styledColumn) {
                     if (!styledColumn.IsReadOnly) {
                         const defaultCurrentSectionName = (0, StyledColumnWizard_1.getStyledColumnEditDefaultSectionName)(styledColumn);
@@ -125,8 +116,6 @@ class StyledColumnModule extends AdaptableModuleBase_1.AdaptableModuleBase {
                             defaultCurrentSectionName: 'Settings',
                         },
                     });
-                    // Icon Style resolves one key → glyph per cell; array columns are
-                    // unsupported (use Badge for per-element visuals).
                     const newIconButton = this.createMenuItemShowPopup('styled-column-icon-add', 'Create Icon Column', this.moduleInfo.Popup, 'icon-style', {
                         ...popupParam,
                         value: {
@@ -140,8 +129,6 @@ class StyledColumnModule extends AdaptableModuleBase_1.AdaptableModuleBase {
                         },
                     });
                     switch (column.dataType) {
-                        // Number
-                        // Create Styled Column - Gradient | Percent Bar | Badge
                         case 'number':
                             const emptyRanges = this.api.columnScopeApi.createCellColorRangesForScope({
                                 ColumnIds: [column.columnId],
@@ -196,8 +183,6 @@ class StyledColumnModule extends AdaptableModuleBase_1.AdaptableModuleBase {
                                 ...popupParam,
                                 value: {
                                     ...newStyledColumn,
-                                    // Seed with column-wide bounds so the Style step opens
-                                    // with a valid (renderable) configuration.
                                     RangeBarStyle: {
                                         Min: 'Col-Min',
                                         Max: 'Col-Max',

package/src/Strategy/TeamSharingModule.js

@@ -30,7 +30,6 @@ class TeamSharingModule extends AdaptableModuleBase_1.AdaptableModuleBase {
     }
     onAdaptableReady() {
         if (this.api.teamSharingApi.isTeamSharingAvailable()) {
-            // make sure there is no zombie import process remaining (in case a previous import crashed)
             this.api.internalApi.dispatchReduxAction(TeamSharingRedux.TeamSharingCommitImport());
             this.api.teamSharingApi.refreshTeamSharing();
         }
@@ -49,25 +48,20 @@ class TeamSharingModule extends AdaptableModuleBase_1.AdaptableModuleBase {
         return 1000;
     }
     handleStateChanged(adaptableStateChangedInfo) {
-        // check if TeamSharing is activated
         if (!this.api.teamSharingApi.isTeamSharingAvailable()) {
             return;
         }
-        // skip all teamsharing actions, otherwise we could get in an infinite loop
         if (this.SKIP_TEAMSHARING_UPDATE_ACTIONS.includes(adaptableStateChangedInfo.action.type)) {
             return;
         }
-        // check if there is any changed adaptable object
         const changedAdaptableObject = this.extractAdaptableObjectFromAction(adaptableStateChangedInfo.action);
         if (!changedAdaptableObject) {
             return;
         }
-        // check if the changed adaptable object is active in TeamSharing
         const activeSharedEntity = this.api.internalApi.getState().TeamSharing.ActiveSharedEntityMap[changedAdaptableObject.Uuid];
         if (!activeSharedEntity) {
             return;
         }
-        // handle special case of LAYOUT_SAVE which might be called redundantly without actually changing the state
         const isLayoutStateUnchanged = adaptableStateChangedInfo.action.type === LayoutRedux_1.LAYOUT_SAVE &&
             (0, ObjectExtensions_1.isEqual)(adaptableStateChangedInfo.oldState.Layout, adaptableStateChangedInfo.newState.Layout);
         if (isLayoutStateUnchanged) {
@@ -76,8 +70,6 @@ class TeamSharingModule extends AdaptableModuleBase_1.AdaptableModuleBase {
         this.api.internalApi.dispatchReduxAction((0, TeamSharingRedux_1.TeamSharingUpdateItem)(changedAdaptableObject, adaptableStateChangedInfo.userName));
     }
     extractAdaptableObjectFromAction(action) {
-        // this implementation is based on the assumption that any action which mutates an AdaptableObject has
-        // at most one(1) property which has an 'Uuid:TypeUuid' defined (and that is the AdaptableObject, of course)
         if (!action) {
             return;
         }
@@ -161,7 +153,6 @@ class TeamSharingModule extends AdaptableModuleBase_1.AdaptableModuleBase {
     }
     toViewAll() {
         return ((this.api.teamSharingApi.getLoadedAdaptableSharedEntities() ?? [])
-            // only top level
             .filter((sharedEntity) => !this.isSharedEntityADependency(sharedEntity))
             .map((item) => this.toView(item)));
     }

package/src/Strategy/Utilities/Layout/columnsSummaryHelpers.js

@@ -103,12 +103,7 @@ function getTableColumnsSizingSummaryValues(layout, api) {
 function getTableColumnsSortingSummaryValues(layout, api) {
     return (layout.ColumnSorts ?? []).map((sort) => `[${api.columnApi.getFriendlyNameForColumnId(sort.ColumnId, layout)}: ${sort.SortOrder}]`);
 }
-/** Matches legacy Layout settings panel behaviour (`layoutViewOptions.maxColumnsToDisplay`). */
 exports.DEFAULT_LAYOUT_VIEW_MAX_COLUMNS = 10;
-/**
- * Collapse a long column list for compact views (e.g. Layout settings panel).
- * Keeps all items when the overflow is only 1 column; otherwise shows `+N more`.
- */
 function truncateColumnSummaryValues(values, maxToDisplay = exports.DEFAULT_LAYOUT_VIEW_MAX_COLUMNS) {
     if (maxToDisplay <= 0 || values.length <= maxToDisplay + 1) {
         return values;

package/src/Utilities/Constants/DocumentationLinkConstants.js

@@ -3,7 +3,6 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.AgGridRequiredModulesDocsLink = exports.FormatColumnPlaceholderDocsLink = exports.AlertMessageDocsLink = exports.AgGridModulesDocsLink = exports.AdaptableOptionsDocsLink = exports.LicenseDocsLink = exports.PrimaryKeyDocsLink = exports.PredicateDocsLink = exports.QuantileAggregatedScalarQueryDocsLink = exports.CumulativeAggregatedScalarQueryDocsLink = exports.AggregatedScalarQueryDocsLink = exports.AggregatedBooleanQueryDocsLink = exports.ObservableQueryDocsLink = exports.ScalarQueryDocsLink = exports.BooleanQueryDocsLink = exports.ExpressionEditorDocsLink = exports.HOST_URL_DOCS = exports.HOST_URL_ROOT = void 0;
 exports.HOST_URL_ROOT = 'https://www.adaptabletools.com';
 exports.HOST_URL_DOCS = `${exports.HOST_URL_ROOT}/docs`;
-//export const HOST_URL_DOCS = 'http://localhost:3000';
 exports.ExpressionEditorDocsLink = `${exports.HOST_URL_DOCS}/ui-expression-editor`;
 exports.BooleanQueryDocsLink = `${exports.HOST_URL_DOCS}/adaptable-ql-expression-standard`;
 exports.ScalarQueryDocsLink = `${exports.HOST_URL_DOCS}/adaptable-ql-expression-standard`;

package/src/Utilities/Constants/GeneralConstants.js

@@ -17,7 +17,6 @@ exports.ADAPTABLE = 'AdapTable';
 exports.ADAPTABLE_ID = 'adaptable_id';
 exports.ADAPTABLE_TOOLPANEL_ID = 'adaptable';
 exports.ADAPTABLE_TOOLPANEL_COMPONENT = 'AdaptableToolPanel';
-// from https://www.ag-grid.com/javascript-data-grid/side-bar/#string-configuration
 exports.AGGRID_TOOLPANEL_FILTERS = 'filters';
 exports.AGGRID_TOOLPANEL_COLUMNS = 'columns';
 exports.READ_ONLY_STYLE = 'ab_readonly';
@@ -33,7 +32,6 @@ exports.AG_GRID_PIVOT_GRAND_TOTAL_COLUMN = 'PivotRowTotal_pivot_';
 exports.AG_GRID_CHART_WINDOW = 'AG Grid Window';
 exports.AG_GRID_GRAND_TOTAL_ROW_ID = 'rowGroupFooter_ROOT_NODE_ID';
 exports.ADAPTABLE_FDC3_ACTION_COLUMN_FRIENDLY_NAME = '(FDC3ActionColumn)';
-// FIXME AFL - load this from DateInputOptions
 exports.DEFAULT_DATE_FORMAT_PATTERN = 'dd-MM-yyyy';
 exports.DEFAULT_DATE_FORMAT_PATTERN_WITH_TIME = 'dd-MM-yyyy HH:mm:ss';
 exports.DEFAULT_STRING_DISPLAY_VALUE = 'Hello World';
@@ -41,12 +39,9 @@ exports.DEFAULT_INTEGER_DISPLAY_VALUE = 12345;
 exports.DEFAULT_DOUBLE_DISPLAY_VALUE = 12345.6789;
 exports.QUICK_SEARCH_DEBOUNCE_TIME = 350;
 exports.MENU_SEPARATOR = '-';
-// Live Report Throttle Time (used in OpenFin, IPushPull etc.)
 exports.DEFAULT_LIVE_REPORT_THROTTLE_TIME = 2000;
-// Validation
 exports.SERVER_VALIDATION_HEADER = 'Server Validation Message';
 exports.SERVER_VALIDATION_MESSAGE_TYPE = 'Info';
-// System Report Name Constants
 exports.ALL_DATA_REPORT = 'All Data';
 exports.CURRENT_LAYOUT_REPORT = 'Current Layout';
 exports.SELECTED_DATA_REPORT = 'Selected Data';
@@ -55,7 +50,6 @@ exports.SYSTEM_REPORT_NAMES = [
     exports.ALL_DATA_REPORT,
     exports.SELECTED_DATA_REPORT,
 ];
-// Systen Report Format Constants
 exports.EXCEL_FORMAT_REPORT = 'Excel';
 exports.VISUAL_EXCEL_FORMAT_REPORT = 'VisualExcel';
 exports.CSV_FORMAT_REPORT = 'CSV';
@@ -74,9 +68,7 @@ exports.SYSTEM_EXPORT_DESTINATIONS = [
     exports.DOWNLOAD_EXPORT_DESTINATION,
     exports.CLIPBOARD_EXPORT_DESTINATION,
 ];
-// Theme Constants
 exports.THEME_STYLE = `ab--theme-`;
-// Layout UI Constants
 exports.LAYOUT_NEW_TABLE_TOOLTIP = 'Create a new Table Layout';
 exports.LAYOUT_NEW_PIVOT_TOOLTIP = 'Create a new Pivot Layout';
 exports.LAYOUT_NEW_TABLE_OR_PIVOT_TOOLTIP = 'Create a new Layout';
@@ -84,7 +76,6 @@ exports.LAYOUT_SAVE_TOOLTIP = 'Save Changes to Current Layout';
 exports.LAYOUT_EDIT_TOOLTIP = 'Edit Layout';
 exports.LAYOUT_CLONE_TOOLTIP = 'Clone Layout';
 exports.LAYOUT_DELETE_TOOLTIP = 'Delete Layout';
-// Standalone Popups
 exports.STANDALONE_MODULE_POPUPS = [
     'BulkUpdate',
     'SmartEdit',
@@ -92,7 +83,6 @@ exports.STANDALONE_MODULE_POPUPS = [
 ];
 exports.OBSERVABLE_EXPRESSION_ROW_ADDED = 'ROW_ADDED';
 exports.OBSERVABLE_EXPRESSION_ROW_REMOVED = 'ROW_REMOVED';
-// Entitlement Access Level Constants
 exports.ACCESS_LEVEL_FULL = 'Full';
 exports.ACCESS_LEVEL_READ_ONLY = 'ReadOnly';
 exports.ACCESS_LEVEL_HIDDEN = 'Hidden';

package/src/Utilities/Constants/ObjectDefaultConstants.js

@@ -1,9 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.ALERT_DEFAULT_SHOW_POPUP = exports.ALERT_DEFAULT_MESSAGE_TYPE = exports.PLUS_MINUS_DEFAULT_NUDGE_VALUE = void 0;
-/**
- * Object Factory
- */
 exports.PLUS_MINUS_DEFAULT_NUDGE_VALUE = 1;
 exports.ALERT_DEFAULT_MESSAGE_TYPE = 'Error';
 exports.ALERT_DEFAULT_SHOW_POPUP = true;

package/src/Utilities/Constants/ReduxConstants.js

@@ -4,14 +4,10 @@ exports.THEME_DEFAULT_CURRENT_THEME = exports.SYSTEM_THEMES = exports.QUICK_SEAR
 const Enums_1 = require("../../AdaptableState/Common/Enums");
 const UIHelper_1 = require("../../View/UIHelper");
 const GeneralConstants_1 = require("./GeneralConstants");
-// Cells Summary
 exports.CELL_SUMMARY_DEFAULT_OPERATION = Enums_1.SummaryOperation.Sum;
-// Quick Search — text match uses a stronger yellow; cell background uses a lighter shade
-exports.QUICK_SEARCH_DEFAULT_TEXT_BACK_COLOR = (0, UIHelper_1.getSwatchColorVar)(9); // #ffff00
-exports.QUICK_SEARCH_DEFAULT_CELL_BACK_COLOR = (0, UIHelper_1.getSwatchColorVar)(10); // #ffffe0
-exports.QUICK_SEARCH_DEFAULT_FORE_COLOR = (0, UIHelper_1.getSwatchColorVar)(1); // black
-exports.QUICK_SEARCH_DEFAULT_CURRENT_BACK_COLOR = (0, UIHelper_1.getSwatchColorVar)(20); // orange
-// theme
-// consts for Themes
+exports.QUICK_SEARCH_DEFAULT_TEXT_BACK_COLOR = (0, UIHelper_1.getSwatchColorVar)(9);
+exports.QUICK_SEARCH_DEFAULT_CELL_BACK_COLOR = (0, UIHelper_1.getSwatchColorVar)(10);
+exports.QUICK_SEARCH_DEFAULT_FORE_COLOR = (0, UIHelper_1.getSwatchColorVar)(1);
+exports.QUICK_SEARCH_DEFAULT_CURRENT_BACK_COLOR = (0, UIHelper_1.getSwatchColorVar)(20);
 exports.SYSTEM_THEMES = [GeneralConstants_1.LIGHT_THEME, GeneralConstants_1.DARK_THEME, GeneralConstants_1.OS_THEME];
 exports.THEME_DEFAULT_CURRENT_THEME = GeneralConstants_1.LIGHT_THEME;

package/src/Utilities/Emitter.js

@@ -1,11 +1,9 @@
 "use strict";
-/** based on emittery npm package, which is MIT */
 Object.defineProperty(exports, "__esModule", { value: true });
 const anyMap = new WeakMap();
 const eventsMap = new WeakMap();
 const triggeredMap = new WeakMap();
 const resolvedPromise = Promise.resolve();
-//type EmitterCallback = (data?: any) => any;
 function assertEventName(eventName) {
     if (typeof eventName !== 'string') {
         throw new TypeError('eventName must be a string');
@@ -19,8 +17,6 @@ function assertListener(listener) {
 function getListeners(instance, eventName) {
     const events = eventsMap.get(instance);
     if (!events) {
-        // we're hitting this case only when calling `off` after the instance has been destroyed (so extremely rarely)
-        // so we want to return an empty set in order not to make the code throw an error
         return new Set();
     }
     if (!events.has(eventName)) {
@@ -87,9 +83,6 @@ class Emittery {
     }
     destroy() {
         this.clearListeners();
-        // should not be needed, since those maps are weak
-        // but let's do it anyway - some of our users suggested this Emitter class
-        // could be a source of memory leaks, so let's clean this up
         anyMap.delete(this);
         eventsMap.delete(this);
         triggeredMap.delete(this);
@@ -180,7 +173,6 @@ class Emittery {
         const staticListeners = [...listeners];
         const staticAnyListeners = [...anyListeners];
         await resolvedPromise;
-        /* eslint-disable no-await-in-loop */
         for (const listener of staticListeners) {
             if (listeners.has(listener)) {
                 await listener(eventData);
@@ -191,7 +183,6 @@ class Emittery {
                 await listener(eventName, eventData);
             }
         }
-        /* eslint-enable no-await-in-loop */
     }
     onAny(listener) {
         assertListener(listener);

package/src/Utilities/ExpressionFunctions/aggregatedBooleanExpressionFunctions.js

@@ -20,9 +20,6 @@ const aggregationScalarOperandMap = {
     AVG: ['string', 'number'],
     COUNT: ['string', 'number'],
 };
-// 0. optional non-capturing block of empty spaces
-// 1. block containing a numeric value (any digits) (required)
-// 2. block containing a single 'K', 'M' or 'B' letter (optional) (case insensitive)
 const CRITERIA_REGEX = /^(?:\s*)(\d+)(K|M|B)?$/i;
 exports.aggregatedBooleanExpressionFunctions = {
     WHERE: {
@@ -144,9 +141,7 @@ exports.aggregatedBooleanExpressionFunctions = {
     },
 };
 const buildBooleanAggregationParameter = (args, context, comparisonOperator, conditionFn) => {
-    // LHS
     const aggregationOperand = extractScalarAggregationOperand(comparisonOperator, args[0]);
-    // RHS
     const scalarOperand = extractScalarOperand(comparisonOperator, args[1], aggregationScalarOperandMap[aggregationOperand.name]);
     return {
         type: 'aggregationBoolean',
@@ -172,7 +167,6 @@ const extractScalarOperand = (consumingFunctionName, value, allowedTypes) => {
     if (typeof value === 'number') {
         return value;
     }
-    // assertion is safe, we checked in the IF above
     const stringValue = value;
     const matchingResult = stringValue.match(CRITERIA_REGEX);
     const numericString = matchingResult?.[1];
@@ -180,14 +174,12 @@ const extractScalarOperand = (consumingFunctionName, value, allowedTypes) => {
     if (!numericString) {
         throw new ExpressionEvaluationError_1.ExpressionEvaluationError('Numeric abbreviation', `"${stringValue}" is invalid, expecting a number with a suffix for thousands(K), millions(M) or billions(B)"`);
     }
-    // unit -> number
     const numberUnitRatios = {
         number: 1,
         k: 1000,
         m: 1000000,
         b: 1000000000,
     };
-    // numeric value
     return (0, NumberExtensions_1.parseIntFn)(numericString) * numberUnitRatios[largeNumberUnit.toLowerCase()];
 };
 exports.aggregatedBooleanExpressionFunctionNames = (0, TypeExtensions_1.getTypedKeys)(exports.aggregatedBooleanExpressionFunctions);

package/src/Utilities/ExpressionFunctions/aggregatedScalarExpressionFunctions.js

@@ -64,8 +64,6 @@ exports.aggregatedScalarExpressionFunctions = {
                 isOptional: true,
             });
             if (!!groupByParameter) {
-                // ensure the GROUP_BY is NOT on the CUMUL function
-                // it should be on the AGGREGATION function
                 throw new ExpressionEvaluationError_1.ExpressionEvaluationError('GROUP_BY', `should be passed as an argument to the '${aggregationParameter.name}' aggregation function!`);
             }
             const cumulationExpressionEvaluation = mapAggregationToCumulation(aggregationParameter, overColumnParameter, context);
@@ -227,7 +225,6 @@ exports.aggregatedScalarExpressionFunctions = {
                                 }
                                 const numericRowValue = (0, expressionFunctionUtils_1.getNumericValue)(rowValue);
                                 if (weightParameter) {
-                                    // weighted average
                                     const weightValue = (0, expressionFunctionUtils_1.getNumericValue)(context.adaptableApi.gridApi.getRawValueFromRowNode(rowNode, weightParameter.value)) ?? 0;
                                     aggregatedValue.sumOfValues =
                                         aggregatedValue.sumOfValues + numericRowValue * weightValue;
@@ -517,7 +514,6 @@ exports.aggregatedScalarExpressionFunctions = {
                             field: columnName,
                             initialValue: { sumOfValues: 0, values: [] },
                             reducer: (aggregatedValue, rowValue, rowNode) => {
-                                // TODO: getNumericValue; allow numeric strings
                                 if (typeof rowValue !== 'number' || isNaN(rowValue)) {
                                     return aggregatedValue;
                                 }
@@ -587,7 +583,6 @@ exports.aggregatedScalarExpressionFunctions = {
                                     return numericRowValue < minValue ? numericRowValue : minValue;
                                 }
                                 else {
-                                    // Date
                                     return (0, date_fns_1.isAfter)((0, DateHelper_1.parseDateValue)(rowValue), (0, DateHelper_1.parseDateValue)(minValue))
                                         ? minValue
                                         : rowValue;
@@ -651,7 +646,6 @@ exports.aggregatedScalarExpressionFunctions = {
                                     return numericRowValue > maxValue ? numericRowValue : maxValue;
                                 }
                                 else {
-                                    // Date
                                     return (0, date_fns_1.isAfter)((0, DateHelper_1.parseDateValue)(rowValue), (0, DateHelper_1.parseDateValue)(maxValue))
                                         ? rowValue
                                         : maxValue;
@@ -704,7 +698,6 @@ exports.aggregatedScalarExpressionFunctions = {
             const groupByColumnNames = groupByOperand?.value;
             let quantReducer;
             if (groupByColumnNames?.length) {
-                // do the heavy-lifting in done()
                 quantReducer = {
                     initialValue: {},
                     reducer: (nodeValueMap, rowValue, rowNode, rowIndex) => {
@@ -747,7 +740,6 @@ exports.aggregatedScalarExpressionFunctions = {
                 };
             }
             else {
-                // in case of non-grouped quantiles, we are able to calculate the buckets beforehand (which is the most efficient)
                 const populationSize = context.adaptableApi.gridApi.getAllRowNodes()?.filter((rowNode) => {
                     const rowValue = context.adaptableApi.gridApi.getRawValueFromRowNode(rowNode, quantileColumnName);
                     return isDefinedValue(rowValue);
@@ -1162,8 +1154,6 @@ exports.aggregatedScalarExpressionFunctions = {
                 isOptional: true,
             });
             if (!!groupByParameter) {
-                // ensure the GROUP_BY is NOT on the ABSOLUTE_DIFF function
-                // it should be on the AGGREGATION function
                 throw new ExpressionEvaluationError_1.ExpressionEvaluationError('GROUP_BY', `should be passed as an argument to the '${relativeValueParameter.name}' aggregation function!`);
             }
             const absoluteDiffExpressionEvaluation = mapWrappedAggregationToDiff(context, 'ABSOLUTE_DIFF', relativeValueParameter, absoluteDiffColumnName, columnType);
@@ -1192,8 +1182,6 @@ exports.aggregatedScalarExpressionFunctions = {
                 isOptional: true,
             });
             if (!!groupByParameter) {
-                // ensure the GROUP_BY is NOT on the ABSOLUTE_DIFF function
-                // it should be on the AGGREGATION function
                 throw new ExpressionEvaluationError_1.ExpressionEvaluationError('GROUP_BY', `should be passed as an argument to the '${relativeValueParameter.name}' aggregation function!`);
             }
             const percentDiffExpressionEvaluation = mapWrappedAggregationToDiff(context, 'PERCENT_DIFF', relativeValueParameter, percentDiffColumnName, columnType);
@@ -1296,7 +1284,6 @@ const mapAggregationToCumulation = (aggregationParameter, overColumnParameter, c
                         reducer: (cumulationBag, rowValue, rowNode) => {
                             if (isDefinedValue(rowValue)) {
                                 if (aggregationEvaluation.context?.weightParam) {
-                                    // weighted average
                                     const weightValue = context.adaptableApi.gridApi.getRawValueFromRowNode(rowNode, aggregationEvaluation.context.weightParam.value) ?? 0;
                                     cumulationBag.currentValue =
                                         cumulationBag.currentValue + rowValue * weightValue;
@@ -1394,7 +1381,6 @@ const mapWrappedAggregationToDiff = (context, diffWrapperFunction, aggregationPa
             if (diffWrapperFunction === 'PERCENT_DIFF') {
                 return ((cellValue - aggregatedValue) / aggregatedValue) * 100;
             }
-            // should never happen
             return null;
         },
         rowFilterFn: aggregationEvaluation.rowFilterFn,
@@ -1406,7 +1392,6 @@ const mapWrappedAggregationToDiff = (context, diffWrapperFunction, aggregationPa
     return absoluteDiffExpressionEvaluation;
 };
 exports.aggregatedScalarExpressionFunctionNames = (0, TypeExtensions_1.getTypedKeys)(exports.aggregatedScalarExpressionFunctions);
-// !! mutates expressionEvaluation
 const addGroupByParams = (groupByColumnNames, expressionEvaluation) => {
     if (groupByColumnNames?.length) {
         expressionEvaluation.aggregationParams.groupBy = groupByColumnNames.map((columnName) => ({

package/src/Utilities/ExpressionFunctions/booleanExpressionFunctions.js

@@ -245,8 +245,6 @@ exports.booleanExpressionFunctions = {
         returnType: 'boolean',
         inputs: [
             ['number', 'number[]'],
-            // Date is currently not supported in select
-            // ['date', 'date[]'],
             ['text', 'text[]'],
         ],
     },
@@ -315,7 +313,6 @@ exports.booleanExpressionFunctions = {
     },
     REGEX: {
         handler(args) {
-            // dont need context as here we always want case insenstivity in the Regex
             const firstInput = String(args[0]);
             const regex = String(args[1]);
             return RegExp(regex).test(firstInput);

package/src/Utilities/ExpressionFunctions/deepMap.js

@@ -26,9 +26,6 @@ class DeepMap {
         let currentMap = this.map;
         if (!keys.length) {
             throw `Cannot set given keys - please provide a non-empty keys array.`;
-            // currentMap.set((EMPTY_KEY as unknown) as K, { value });
-            // this.length++;
-            // return this;
         }
         for (let i = 0, len = keys.length; i < len; i++) {
             const key = keys[i];
@@ -55,7 +52,6 @@ class DeepMap {
         let currentMap = this.map;
         if (!keys.length) {
             throw `Cannot get given keys - please provide a non-empty keys array.`;
-            // return currentMap.get((EMPTY_KEY as unknown) as K)?.value;
         }
         for (let i = 0, len = keys.length; i < len; i++) {
             const key = keys[i];
@@ -95,10 +91,6 @@ class DeepMap {
         let currentMap = this.map;
         if (!keys.length) {
             throw `Cannot delete given keys - please provide a non-empty keys array.`;
-            // if (currentMap.has(EMPTY_KEY as unknown as K)) {
-            //   this.length--;
-            // }
-            // return currentMap.delete((EMPTY_KEY as unknown) as K);
         }
         let maps = [currentMap];
         let result = false;
@@ -118,7 +110,6 @@ class DeepMap {
                         delete pair.map;
                     }
                     if (!pair.map) {
-                        // pair is empty, so we can remove it altogether
                         currentMap.delete(key);
                     }
                 }
@@ -141,10 +132,8 @@ class DeepMap {
                 let parentMap = maps[maps.length - 1];
                 let pair = parentMap?.get(key);
                 if (pair) {
-                    // pair.map === map ; which can be deleted
                     delete pair.map;
                     if (!pair.hasOwnProperty('value')) {
-                        // whole pair can be successfully deleted from parentMap
                         parentMap.delete(key);
                     }
                 }
@@ -156,10 +145,6 @@ class DeepMap {
         let currentMap = this.map;
         if (!keys.length) {
             throw `Cannot find existing given keys - please provide a non-empty keys array.`;
-            // return (
-            //   currentMap.has((EMPTY_KEY as unknown) as K) &&
-            //   currentMap.get((EMPTY_KEY as unknown) as K)?.value !== undefined
-            // );
         }
         for (let i = 0, len = keys.length; i < len; i++) {
             const key = keys[i];
@@ -195,7 +180,6 @@ class DeepMap {
         if (pair.hasOwnProperty('value')) {
             fn(pair, keys, next);
         }
-        // if it was called by fn, it won't be called again, as it's once-d
         next();
     }
     visit = (fn) => {

package/src/Utilities/ExpressionFunctions/expressionFunctionUtils.js

@@ -7,8 +7,6 @@ const StringExtensions_1 = tslib_1.__importDefault(require("../Extensions/String
 const operators_1 = require("rxjs/operators");
 const NumberExtensions_1 = require("../Extensions/NumberExtensions");
 const evaluator_1 = require("../../parser/src/evaluator");
-// string functions may be case (in)sensitive, see internalApi.isTextComparisonCaseSensitive()
-// if case is insensitive, we convert ALL involved strings to lower case
 const getStringValues = (context, ...stringArguments) => {
     return (0, exports.isTextSearchCaseInsensitive)(context)
         ? stringArguments.map((string) => string.toLowerCase())
@@ -21,13 +19,9 @@ const getStringValue = (context, stringArgument) => {
 exports.getStringValue = getStringValue;
 const isTextSearchCaseInsensitive = (context) => !context.adaptableApi.expressionApi.useCaseSensitivity();
 exports.isTextSearchCaseInsensitive = isTextSearchCaseInsensitive;
-// returns an observable which filters the source$ emissions and emits only if the CellDataChangedInfo relates to the given column
-// optionally, if a filter function (where clause) is provided, it is also evaluated
 const getDataChangeLog$ = (context, columnNameFilter) => {
     let dataChangeLog$ = context.adaptableApi.internalApi.getDataService().cellDataChangeLog$;
-    // filter only the given column changes
     dataChangeLog$ = dataChangeLog$.pipe((0, operators_1.filter)((dataChangeLog) => dataChangeLog.column.columnId === columnNameFilter));
-    // filter based on the WHERE clause
     if (context.filterFn) {
         dataChangeLog$ = dataChangeLog$.pipe((0, operators_1.filter)((value) => {
             return context.filterFn(value.rowNode);
@@ -39,7 +33,6 @@ exports.getDataChangeLog$ = getDataChangeLog$;
 const getGridChangeLog$ = (context, rowChangeType) => {
     const gridChangeLog$ = context.adaptableApi.internalApi.getDataService().rowDataChangeLog$;
     let gridChangeStream$ = gridChangeLog$.pipe((0, operators_1.filter)((rowDataChangedInfo) => rowDataChangedInfo.rowTrigger === rowChangeType));
-    // filter based on the WHERE clause
     if (context.filterFn) {
         gridChangeStream$ = gridChangeStream$.pipe((0, operators_1.filter)((value) => value.rowNodes.some((node) => context.filterFn(node))));
     }
@@ -50,7 +43,6 @@ const handleWhereFunction = (args, context) => {
     const reactiveExpressionNode = args[0];
     const whereClauseExpressionNode = args[1];
     const filterFn = (rowNode) => {
-        // whereClauseResult
         return (0, evaluator_1.evaluateNode)(whereClauseExpressionNode, {
             node: rowNode,
             functions: context.whereClauseFunctions,
@@ -58,7 +50,6 @@ const handleWhereFunction = (args, context) => {
             ...context.adaptableApi.internalApi.buildBaseContext(),
         });
     };
-    // observableExpressionResult
     return (0, exports.evaluateExpressionNode)(reactiveExpressionNode, context, filterFn);
 };
 exports.handleWhereFunction = handleWhereFunction;

package/src/Utilities/ExpressionFunctions/groupingMap.js

@@ -65,15 +65,6 @@ function initReducers(reducers) {
         }
     return result;
 }
-/**
- *
- * This fn mutates the reducerResults array!!!
- *
- * @param data data item
- * @param reducers an array of reducers
- * @param reducerResults the results on which to operate
- *
- */
 function computeReducersFor(data, reducers, reducerResults) {
     if (!reducers || !Object.keys(reducers).length) {
         return;