diff --git a/src/app/pages/sharing/smb/smb-list/smb-list.component.ts b/src/app/pages/sharing/smb/smb-list/smb-list.component.ts index 45b3c9448c1..8ae7d815577 100644 --- a/src/app/pages/sharing/smb/smb-list/smb-list.component.ts +++ b/src/app/pages/sharing/smb/smb-list/smb-list.component.ts @@ -107,20 +107,7 @@ export class SmbListComponent implements OnInit { title: this.translate.instant('Enabled'), propertyName: 'enabled', requiredRoles: this.requiredRoles, - onRowToggle: (row) => { - this.ws.call('sharing.smb.update', [row.id, { enabled: row.enabled }]).pipe( - this.appLoader.withLoader(), - untilDestroyed(this), - ).subscribe({ - next: (share) => { - row.enabled = share.enabled; - }, - error: (error: unknown) => { - this.dataProvider.load(); - this.dialog.error(this.errorHandler.parseError(error)); - }, - }); - }, + onRowToggle: (row) => this.onChangeEnabledState(row), }), yesNoColumn({ title: this.translate.instant('Audit Logging'), @@ -273,4 +260,19 @@ export class SmbListComponent implements OnInit { message: this.translate.instant('The path {path} is in a locked dataset.', { path }), }); } + + private onChangeEnabledState(row: SmbShare): void { + this.ws.call('sharing.smb.update', [row.id, { enabled: !row.enabled }]).pipe( + this.appLoader.withLoader(), + untilDestroyed(this), + ).subscribe({ + next: () => { + this.dataProvider.load(); + }, + error: (error: unknown) => { + this.dataProvider.load(); + this.dialog.error(this.errorHandler.parseError(error)); + }, + }); + } }