Fix: Permission update when changing batch account (#1988)

Azure · Mar 21, 2019 · 7dbc678 · 7dbc678
1 parent ede432e
commit 7dbc678
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -3,6 +3,7 @@
 ### Bug fixes
 * Node counts graphs don't recover from errors(e.g. Connection died). [\#1989](https://github.com/Azure/BatchExplorer/issues/1989)
 * Null exception in resource files properties [\#1984](https://github.com/Azure/BatchExplorer/issues/1984)
+* Only loads permissions for first selected batch account [\#1987](https://github.com/Azure/BatchExplorer/issues/1987)
 
 # 2.0.1
 

diff --git a/src/app/services/authorization-http/authorization-http.service.ts b/src/app/services/authorization-http/authorization-http.service.ts
@@ -2,7 +2,7 @@ import { Injectable } from "@angular/core";
 import { HttpRequestOptions } from "@batch-flask/core";
 import { Permission } from "@batch-flask/ui/permission";
 import { Observable, empty, of } from "rxjs";
-import { expand, flatMap, map, reduce, shareReplay, take } from "rxjs/operators";
+import { expand, flatMap, map, reduce, shareReplay, switchMap } from "rxjs/operators";
 import { ArmBatchAccount } from "../../models";
 import { ArmHttpService } from "../arm-http.service";
 import { BatchAccountService } from "../batch-account";
@@ -31,8 +31,7 @@ export class AuthorizationHttpService {
         private armService: ArmHttpService) {
 
         this._permission = this.accountService.currentAccount.pipe(
-            take(1),
-            flatMap(account => {
+            switchMap((account) => {
                 if (account instanceof ArmBatchAccount) {
                     const resourceId = account && account.id;
                     return this.getPermission(resourceId);