fix: fixed consistency of idpUrls among all idpUrls .filter

Noovolari · Apr 29, 2021 · 2472f83 · 2472f83
1 parent 5f7a988
commit 2472f83
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 8 deletions.
diff --git a/src/app/app.component.ts b/src/app/app.component.ts
@@ -72,9 +72,9 @@ export class AppComponent implements OnInit {
           if (session.account.type === AccountType.AWS || session.account.type === AccountType.AWS_TRUSTER) {
             if (session.account.parent === undefined) {
               if (session.account.idpUrl === '' || session.account.idpUrl === null || session.account.idpUrl === undefined) {
-                session.account.idpUrl = workspace.idpUrl[0].id; // We force the first
+                session.account.idpUrl = workspace.idpUrl.filter(u => (u !== null && u !== undefined))[0].id; // We force the first
               } else {
-                const found = workspace.idpUrl.filter(u => u.url === session.account.idpUrl)[0];
+                const found = workspace.idpUrl.filter(u => u && u.url === session.account.idpUrl)[0];
                 if (found) {
                   session.account.idpUrl = found.id;
                 }

diff --git a/src/app/services/federated-account.service.ts b/src/app/services/federated-account.service.ts
@@ -70,7 +70,7 @@ export class FederatedAccountService extends NativeService {
 
       console.log('idpurl in workspace', workspace.idpUrl);
 
-      if (workspace.idpUrl.findIndex(i => i.id === idpUrl.id) === -1) {
+      if (workspace.idpUrl.findIndex(i => i && i.id === idpUrl.id) === -1) {
         workspace.idpUrl.push(idpUrl);
       }
 

diff --git a/src/app/services/workspace.service.ts b/src/app/services/workspace.service.ts
@@ -147,10 +147,10 @@ export class WorkspaceService extends NativeService {
     const workspace = this.configurationService.getDefaultWorkspaceSync();
     let idpUrl;
     if (session.account.parent === undefined) {
-      idpUrl = workspace.idpUrl.filter(u => u.id === session.account.idpUrl)[0].url;
+      idpUrl = workspace.idpUrl.filter(u => u && u.id === session.account.idpUrl)[0].url;
     } else {
       const parentSession = this.sessionService.getSession(session.account.parent);
-      idpUrl = workspace.idpUrl.filter(u => u.id === parentSession.account.idpUrl)[0].url;
+      idpUrl = workspace.idpUrl.filter(u => u && u.id === parentSession.account.idpUrl)[0].url;
     }
 
     // TODO: probably here will need to clean the partition area to force a windows refresh

diff --git a/src/app/shared/profile-page/profile-page.component.html b/src/app/shared/profile-page/profile-page.component.html
@@ -83,13 +83,15 @@ <h2 style="margin-top: 10px;">IdP URL list</h2>
             </thead>
             <tbody>
             <ng-container *ngFor="let idpUrl of workspaceData.idpUrl">
+              <ng-container *ngIf="idpUrl">
               <tr>
                 <td>{{idpUrl.url}}</td>
                 <td>
                   <button class="xs-btn" type="button" (click)="editIdpUrl(idpUrl.id);"><i class="fa fa-edit"></i></button>
                   <button class="xs-btn" type="button" (click)="deleteIdpUrl(idpUrl.id);"><i class="fa fa-trash"></i></button>
                 </td>
               </tr>
+              </ng-container>
             </ng-container>
             </tbody>
           </table>

diff --git a/src/app/shared/profile-page/profile-page.component.ts b/src/app/shared/profile-page/profile-page.component.ts
@@ -153,7 +153,7 @@ export class ProfilePageComponent extends AntiMemLeak implements OnInit {
   }
 
   manageIdpUrl(id) {
-    const idpUrl = this.workspaceData.idpUrl.findIndex(u => u.id === id);
+    const idpUrl = this.workspaceData.idpUrl.findIndex(u => u && u.id === id);
     if (this.form.get('idpUrl').value !== '') {
       if (idpUrl === -1) {
         this.workspaceData.idpUrl.push({ id: uuid.v4(), url: this.form.get('idpUrl').value });
@@ -168,7 +168,7 @@ export class ProfilePageComponent extends AntiMemLeak implements OnInit {
   }
 
   editIdpUrl(id) {
-    const idpUrl = this.workspaceData.idpUrl.filter(u => u.id === id)[0];
+    const idpUrl = this.workspaceData.idpUrl.filter(u => u && u.id === id)[0];
     this.idpUrlValue = idpUrl;
     this.form.get('idpUrl').setValue(idpUrl.url);
     this.editingIdpUrl = true;
@@ -197,7 +197,7 @@ export class ProfilePageComponent extends AntiMemLeak implements OnInit {
     this.appService.confirmDialog(`Deleting this Idp url will also remove these sessions: <br><ul>${sessionsNames.join('')}</ul>Do you want to proceed?`, (res) => {
       if (res !== constants.CONFIRM_CLOSED) {
         this.appService.logger(`Removing idp url with id: ${id}`, LoggerLevel.INFO, this);
-        const idpUrl = this.workspaceData.idpUrl.findIndex(u => u.id === id);
+        const idpUrl = this.workspaceData.idpUrl.findIndex(u => u && u.id === id);
         this.workspaceData.idpUrl.splice(idpUrl, 1);
         this.configurationService.updateWorkspaceSync(this.workspaceData);
         sessions.forEach(s => {