Fixed some sonar issues

COS301-SE-2023 · Sep 21, 2023 · c299e5d · c299e5d
1 parent 5a87d33
commit c299e5d
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/frontend/src/app/edit/edit.component.html b/frontend/src/app/edit/edit.component.html
@@ -259,6 +259,7 @@ <h4 *ngIf="sideBarTab" class="sidebarHeaderText">History</h4>
           class="spinner"
           src="assets/Spinner/loading2.gif"
           title="Loading..."
+          alt="Loading..."
         />
       </div>
 
@@ -275,6 +276,7 @@ <h4 *ngIf="sideBarTab" class="sidebarHeaderText">History</h4>
             class="spinner"
             src="assets/Spinner/loading2.gif"
             title="Loading..."
+            alt="Loading..."
           />
         </div>
         <div

diff --git a/frontend/src/app/edit/edit.component.ts b/frontend/src/app/edit/edit.component.ts
@@ -632,7 +632,7 @@ export class EditComponent implements AfterViewInit, OnInit {
             : 0;
         });
 
-        snapshot.map((a, i) => {
+        snapshot.forEach((a, i) => {
           a.LastModifiedString = this.formatDate(a.LastModified);
           a.OrderNumber = i + 1;
           a.Name = 'Snapshot ' + a.OrderNumber;
@@ -647,14 +647,14 @@ export class EditComponent implements AfterViewInit, OnInit {
             }
           }
         });
-        snapshot.map((a) =>
+        snapshot.forEach((a) =>
           a.ChildDiffs.sort((a, b) => {
             return a.LastModified < b.LastModified
               ? 1
               : a.LastModified > b.LastModified
               ? -1
               : 0;
-          }).map((a, i, arr) => {
+          }).forEach((a, i, arr) => {
             a.VersionNumber = arr.length - i + 1;
             a.Name = 'Version ' + a.VersionNumber;
           })
@@ -669,7 +669,7 @@ export class EditComponent implements AfterViewInit, OnInit {
             this.history.push(diff[i]);
           }
         }
-        snapshot.map((a) => this.history.push(a));
+        snapshot.forEach((a) => this.history.push(a));
 
         console.log(this.history);
       });

diff --git a/frontend/src/app/home/home.component.html b/frontend/src/app/home/home.component.html
@@ -1,6 +1,6 @@
 
 <div *ngIf="loading" class="loading-overlay">
-  <img class="loadingIcon" src="assets/Spinner/loading2.gif" alt="Loading..." />
+  <img class="loadingIcon" src="assets/Spinner/loading2.gif" title="Loading..." alt="Loading..." />
 </div>
 
 <div class="container" data-testid="home-page" (click)="unselectAll()">