NAS-132227: Fix enable change in SMB table (#11001)

Co-authored-by: Boris Vasilenko <[email protected]>
truenas · Nov 11, 2024 · 915b8f6 · 915b8f6
1 parent 9d2424c
commit 915b8f6
Showing 1 changed file with 16 additions and 14 deletions.
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
@@ -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 <i>{path}</i> 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));
+      },
+    });
+  }
 }