angular-slickgrid 4.2.2 → 4.2.3

package/fesm2020/angular-slickgrid.mjs

@@ -178,6 +178,9 @@ class ContainerService {
         }
         return null;
     }
+    dispose() {
+        this.dependencies = [];
+    }
     registerInstance(key, instance) {
         const dependency = this.dependencies.some(dep => dep.key === key);
         if (!dependency) {
@@ -930,6 +933,7 @@ class AngularSlickgridComponent {
         this.gridService = externalServices?.gridService ?? new GridService(this.gridStateService, this.filterService, this._eventPubSubService, this.paginationService, this.sharedService, this.sortService, this.treeDataService);
         this.groupingService = externalServices?.groupingAndColspanService ?? new GroupingAndColspanService(this.extensionUtility, this._eventPubSubService);
         this.serviceList = [
+            this.containerService,
             this.extensionService,
             this.filterService,
             this.gridEventService,
@@ -1129,6 +1133,7 @@ class AngularSlickgridComponent {
         this.datasetHierarchical = undefined;
         this._columnDefinitions = [];
         this._angularGridInstances = undefined;
+        this.slickGrid = undefined;
     }
     emptyGridContainerElm() {
         const gridContainerId = this.gridOptions?.gridContainerId ?? 'grid1';
@@ -1238,8 +1243,7 @@ class AngularSlickgridComponent {
             this.dataView.endUpdate();
             // if you don't want the items that are not visible (due to being filtered out or being on a different page)
             // to stay selected, pass 'false' to the second arg
-            const selectionModel = this.slickGrid?.getSelectionModel();
-            if (selectionModel && this.gridOptions && this.gridOptions.dataView && this.gridOptions.dataView.hasOwnProperty('syncGridSelection')) {
+            if (this.slickGrid?.getSelectionModel() && this.gridOptions && this.gridOptions.dataView && this.gridOptions.dataView.hasOwnProperty('syncGridSelection')) {
                 // if we are using a Backend Service, we will do an extra flag check, the reason is because it might have some unintended behaviors
                 // with the BackendServiceApi because technically the data in the page changes the DataView on every page change.
                 let preservedRowSelectionWithBackend = false;
@@ -1526,27 +1530,26 @@ class AngularSlickgridComponent {
                 // When data changes in the DataView, we need to refresh the metrics and/or display a warning if the dataset is empty
                 this._eventHandler.subscribe(dataView.onRowCountChanged, () => {
                     grid.invalidate();
-                    this.handleOnItemCountChanged(this.dataView.getFilteredItemCount() || 0, dataView.getItemCount());
+                    this.handleOnItemCountChanged(dataView.getFilteredItemCount() || 0, dataView.getItemCount() || 0);
                 });
                 this._eventHandler.subscribe(dataView.onSetItemsCalled, (_e, args) => {
-                    grid.invalidate();
-                    this.handleOnItemCountChanged(this.dataView.getFilteredItemCount(), args.itemCount);
+                    this.handleOnItemCountChanged(dataView.getFilteredItemCount() || 0, args.itemCount);
                     // when user has resize by content enabled, we'll force a full width calculation since we change our entire dataset
                     if (args.itemCount > 0 && (this.gridOptions.autosizeColumnsByCellContentOnFirstLoad || this.gridOptions.enableAutoResizeColumnsByCellContent)) {
                         this.resizerService.resizeColumnsByCellContent(!this.gridOptions?.resizeByContentOnlyOnFirstLoad);
                     }
                 });
-                this._eventHandler.subscribe(dataView.onRowsChanged, (_e, args) => {
-                    // filtering data with local dataset will not always show correctly unless we call this updateRow/render
-                    // also don't use "invalidateRows" since it destroys the entire row and as bad user experience when updating a row
-                    // see commit: https://github.com/ghiscoding/aurelia-slickgrid/commit/8c503a4d45fba11cbd8d8cc467fae8d177cc4f60
-                    if (gridOptions?.enableFiltering && !gridOptions.enableRowDetailView) {
+                if (gridOptions?.enableFiltering && !gridOptions.enableRowDetailView) {
+                    this._eventHandler.subscribe(dataView.onRowsChanged, (_e, args) => {
+                        // filtering data with local dataset will not always show correctly unless we call this updateRow/render
+                        // also don't use "invalidateRows" since it destroys the entire row and as bad user experience when updating a row
+                        // see commit: https://github.com/ghiscoding/aurelia-slickgrid/commit/8c503a4d45fba11cbd8d8cc467fae8d177cc4f60
                         if (args?.rows && Array.isArray(args.rows)) {
                             args.rows.forEach((row) => grid.updateRow(row));
                             grid.render();
                         }
-                    }
-                });
+                    });
+                }
             }
         }
         // did the user add a colspan callback? If so, hook it into the DataView getItemMetadata
@@ -1765,9 +1768,8 @@ class AngularSlickgridComponent {
     loadRowSelectionPresetWhenExists() {
         // if user entered some Row Selections "presets"
         const presets = this.gridOptions?.presets;
-        const selectionModel = this.slickGrid?.getSelectionModel();
         const enableRowSelection = this.gridOptions && (this.gridOptions.enableCheckboxSelector || this.gridOptions.enableRowSelection);
-        if (enableRowSelection && selectionModel && presets && presets.rowSelection && (Array.isArray(presets.rowSelection.gridRowIndexes) || Array.isArray(presets.rowSelection.dataContextIds))) {
+        if (enableRowSelection && this.slickGrid?.getSelectionModel() && presets?.rowSelection && (Array.isArray(presets.rowSelection.gridRowIndexes) || Array.isArray(presets.rowSelection.dataContextIds))) {
             let dataContextIds = presets.rowSelection.dataContextIds;
             let gridRowIndexes = presets.rowSelection.gridRowIndexes;
             // maps the IDs to the Grid Rows and vice versa, the "dataContextIds" has precedence over the other
@@ -1859,7 +1861,7 @@ class AngularSlickgridComponent {
         // register all services by executing their init method and providing them with the Grid object
         if (Array.isArray(this._registeredResources)) {
             for (const resource of this._registeredResources) {
-                if (typeof resource.init === 'function') {
+                if (this.slickGrid && typeof resource.init === 'function') {
                     resource.init(this.slickGrid, this.containerService);
                 }
             }