Fix reload loop bug on shared folder ACL page.

Signed-off-by: Volker Theile <votdev@gmx.de>
openmediavault · Jan 19, 2023 · bd4f957 · bd4f957
1 parent 7f286e6
commit bd4f957
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/deb/openmediavault/debian/changelog b/deb/openmediavault/debian/changelog
@@ -1,5 +1,6 @@
 openmediavault (6.1.7-1) stable; urgency=low
 
+  * Fix reload loop bug on shared folder ACL page.
   * Issue #1310: Store UI settings on the server as well instead
     of the browser only. Note, existing settings in the browser
     are not adopted. Thanks to jpspringall for his contribution.

diff --git a/...t/workbench/src/app/pages/storage/shared-folders/shared-folder-acl-form-page.component.ts b/...t/workbench/src/app/pages/storage/shared-folders/shared-folder-acl-form-page.component.ts
@@ -20,7 +20,7 @@ import { marker as gettext } from '@ngneat/transloco-keys-manager/marker';
 import * as _ from 'lodash';
 import { BlockUI, NgBlockUI } from 'ng-block-ui';
 import { EMPTY } from 'rxjs';
-import { catchError, finalize } from 'rxjs/operators';
+import { catchError, distinctUntilChanged, finalize } from 'rxjs/operators';
 
 import {
   FormPageButtonConfig,
@@ -295,7 +295,7 @@ export class SharedFolderAclFormPageComponent extends BaseFormPageComponent impl
     self.loadData = () => this.loadData('/');
     self.afterViewInitEvent.subscribe(() => {
       const control = self.form.formGroup.get('file');
-      control.valueChanges.subscribe((value) => this.loadData(value));
+      control.valueChanges.pipe(distinctUntilChanged()).subscribe((value) => this.loadData(value));
     });
   }