@c8y/ngx-components 1023.97.3 → 1023.97.7

package/fesm2022/c8y-ngx-components-widgets-implementations-asset-table.mjs

@@ -9,7 +9,7 @@ import * as i1 from '@c8y/ngx-components';
 import { DatePipe, BaseColumn, CustomColumn, IconDirective, AlertService, CellRendererContext, CommonModule, IntervalBasedReload, DEFAULT_INTERVAL_VALUES, WIDGET_TYPE_VALUES, CoreModule, CountdownIntervalComponent, ManagedObjectRealtimeService, WidgetGlobalAutoRefreshService, globalAutoRefreshLoading, WidgetActionWrapperComponent, ModalModule, SelectModule, DateTimePickerModule, C8yTranslateModule, BottomDrawerService, DropdownFocusTrapDirective, EmptyStateComponent, ListGroupModule, C8yTranslatePipe, HumanizePipe } from '@c8y/ngx-components';
 import { AssetSelectorModule } from '@c8y/ngx-components/assets-navigator';
 import { WidgetConfigService, ContextDashboardService } from '@c8y/ngx-components/context-dashboard';
-import { from, switchMap, isObservable, of, map, firstValueFrom, tap, BehaviorSubject, Subject, merge, startWith, scan, distinctUntilChanged, shareReplay, takeUntil, skip } from 'rxjs';
+import { from, switchMap, isObservable, of, map, firstValueFrom, tap, BehaviorSubject, Subject, merge, startWith, scan, distinctUntilChanged, takeUntil, shareReplay, skip } from 'rxjs';
 import * as i2$1 from '@c8y/ngx-components/asset-properties';
 import { AssetPropertySelectorDrawerComponent } from '@c8y/ngx-components/asset-properties';
 import { transform, identity, get, assign } from 'lodash';
@@ -1409,7 +1409,11 @@ class AssetTableWidgetConfigComponent {
     ngOnInit() {
         this.initializeForm();
         this.setupGridHeaderFilterDependency();
-        const mergedConfig$ = merge(this.widgetConfigService.currentConfig$, this.formGroup.valueChanges).pipe(startWith(this.formGroup.getRawValue()), scan((acc, value) => ({ ...acc, ...value }), {}), distinctUntilChanged((a, b) => isEqual(a, b)), shareReplay(1));
+        const mergedConfig$ = merge(this.widgetConfigService.currentConfig$, this.formGroup.valueChanges).pipe(startWith(this.formGroup.getRawValue()), scan((acc, value) => ({ ...acc, ...value }), {}), distinctUntilChanged((a, b) => isEqual(a, b)), 
+        // takeUntil must precede shareReplay: a non-refCount shareReplay never unsubscribes from its
+        // source, so without this the merge would keep the root currentConfig$ BehaviorSubject
+        // subscribed (and the buffered config + formGroup graph alive) forever after destroy.
+        takeUntil(this.destroy$), shareReplay(1));
         mergedConfig$.pipe(takeUntil(this.destroy$)).subscribe(async (config) => {
             if (!config.device && config.settings?.context && !this.initializedContext) {
                 const updatedDevice = { ...config.settings.context };
@@ -1522,9 +1526,12 @@ class AssetTableWidgetConfigComponent {
     updateConfigBasedOnRefreshOption() {
         const isInterval = this.refreshOption === 'interval';
         this.formGroup.controls.isAutoRefreshEnabled.setValue(isInterval);
-        isInterval
-            ? this.formGroup.get('refreshInterval').enable()
-            : this.formGroup.get('refreshInterval').disable();
+        if (isInterval) {
+            this.formGroup.get('refreshInterval').enable();
+        }
+        else {
+            this.formGroup.get('refreshInterval').disable();
+        }
     }
     updateFormDisabledState() {
         const selectedProperties = this.formGroup.get('selectedProperties')?.value;