Solve sonar issues regarding window. to replace with globalThis.

Signed-off-by: freddidierRTE <[email protected]>

Author: freddidierRTE

frontend/src/app/authentication/AuthHandler.ts

@@ -63,7 +63,7 @@ export abstract class AuthHandler {
     }
 
     public getOpfabRouteAfterLogin(): string {
-        const hash = window.location.hash;
+        const hash = globalThis.location.hash;
         const hashWithoutAuthenticationFields = hash.split('&')[0].split('?')[0];
         const hashLength = hash.length;
         const routeAfterLogin = hashLength > 2 ? hashWithoutAuthenticationFields.substring(1, hashLength) : '/';

frontend/src/app/authentication/AuthService.ts

@@ -118,7 +118,7 @@ export class AuthService {
         this.removeUserFromStorage();
         this.authHandler.logout();
         if (this.mode !== AuthenticationMode.IMPLICIT)
-            window.location.href = ConfigService.getConfigValue('security.logout-url', 'https://opfab.github.io');
+            globalThis.location.href = ConfigService.getConfigValue('security.logout-url', 'https://opfab.github.io');
     }
 
     private goBackToLoginPage() {

frontend/src/app/authentication/CodeAuthenticationHandler.ts

@@ -16,9 +16,9 @@ export class CodeAuthenticationHandler extends AuthHandler {
     initializeAuthentication() {
         let authCode;
         const searchCodeString = 'code=';
-        const foundIndex = window.location.href.indexOf(searchCodeString);
+        const foundIndex = globalThis.location.href.indexOf(searchCodeString);
         if (foundIndex !== -1) {
-            authCode = window.location.href.substring(foundIndex + searchCodeString.length);
+            authCode = globalThis.location.href.substring(foundIndex + searchCodeString.length);
         } else this.saveOpfabRoute();
         this.checkAuthentication().subscribe((token) => {
             // no token stored or token invalid
@@ -46,14 +46,14 @@ export class CodeAuthenticationHandler extends AuthHandler {
     // so we need to save in the session storage the route before login.
     // We do not use the local storage because the local storage is shared between tabs
     private saveOpfabRoute() {
-        const hash = window.location.hash;
+        const hash = globalThis.location.hash;
         const hashLength = hash.length;
         const routeAfterLogin = hashLength > 2 ? hash.substring(1, hashLength) : '/';
-        window.sessionStorage.setItem('route_after_login_for_code_flow', routeAfterLogin);
+        globalThis.sessionStorage.setItem('route_after_login_for_code_flow', routeAfterLogin);
     }
 
     public getOpfabRouteAfterLogin(): string {
-        return window.sessionStorage.getItem('route_after_login_for_code_flow');
+        return globalThis.sessionStorage.getItem('route_after_login_for_code_flow');
     }
 
     private askToken(code: string): Observable<HttpAuthInfo> {
@@ -69,9 +69,9 @@ export class CodeAuthenticationHandler extends AuthHandler {
 
     private moveToLoginPage() {
         if (!this.delegateUrl) {
-            window.location.href = `${environment.url}auth/code/redirect_uri=${this.getRedirectUri()}`;
+            globalThis.location.href = `${environment.url}auth/code/redirect_uri=${this.getRedirectUri()}`;
         } else {
-            window.location.href = `${this.delegateUrl}&redirect_uri=${this.getRedirectUri()}`;
+            globalThis.location.href = `${this.delegateUrl}&redirect_uri=${this.getRedirectUri()}`;
         }
     }
 

frontend/src/app/components/core/application-loading/ApplicationLoadingComponent.ts

@@ -161,7 +161,7 @@ export class ApplicationLoadingComponent implements OnInit {
     // To have a cleaner code , we need to refactor the code
     // regarding authentication
     private waitForEmptyTokenInStorageToShowLoginForm() {
-        if (!window.localStorage.getItem('token')) {
+        if (!globalThis.localStorage.getItem('token')) {
             this.showLoginScreen = true;
         } else if (!this.applicationLoaded) {
             setTimeout(() => this.waitForEmptyTokenInStorageToShowLoginForm(), 100);

frontend/src/app/components/core/application-loading/app-loaded-in-another-tab/AppLoadedInAnotherTabComponent.ts

@@ -50,7 +50,7 @@ export class AppLoadedInAnotherTabComponent extends ApplicationLoadingComponent
     private isApplicationActive = false;
 
     public async execute(): Promise<boolean> {
-        this.opfabUrl = window.location.href;
+        this.opfabUrl = globalThis.location.href;
         setTimeout(() => this.checkIfAppLoadedInAnotherTab(), 0);
         this.createListenerForDisconnectSignal();
         return super.execute();

frontend/src/app/components/core/application-loading/app-loaded-in-another-tab/UrlLockService.ts

@@ -37,7 +37,7 @@ export class UrlLockService {
 
     public setDisconnectSignalListener(listener: Function): void {
         this.disconnectSignalListener = listener;
-        window.addEventListener('storage', this.listenForDisconnectSignal.bind(this), false);
+        globalThis.addEventListener('storage', this.listenForDisconnectSignal.bind(this), false);
     }
 
     private listenForDisconnectSignal(event): void {

frontend/src/app/components/core/application-loading/loading-in-progress/LoadingInProgressComponent.ts

@@ -32,7 +32,7 @@ export class LoadingInProgressComponent implements OnInit, OnDestroy {
     }
 
     reloadPage() {
-        window.location.reload();
+        globalThis.location.reload();
     }
 
     ngOnDestroy() {

frontend/src/app/components/externalappiframe/view/ExternalAppIFrameView.spec.ts

@@ -83,7 +83,7 @@ describe('ExternalAppIFrame view ', () => {
     });
 
     // If a business application is called form a card, it can be called with parameters
-    // To do that, in the card the window.location is set with the url #/businessconfigparty/menu_id/menuItem_id/
+    // To do that, in the card the globalThis.location is set with the url #/businessconfigparty/menu_id/menuItem_id/
     // then is is possible to add params to the url
     // For example: #/businessconfigparty/menu_id/menuItem_id/?myparam=param1&myotherparam=param2
     //
@@ -99,7 +99,7 @@ describe('ExternalAppIFrame view ', () => {
     });
 
     // If a business application is called form a card, it can be called with parameters
-    // To do that, in the card the window.location is set with the url #/businessconfigparty/menu_id/menuItem_id
+    // To do that, in the card the globalThis.location is set with the url #/businessconfigparty/menu_id/menuItem_id
     // then is is possible to add params to the url
     // For example: #/businessconfigparty/menu_id/menuItem_id?myparam=param1&myotherparam=param2
     //

frontend/src/app/components/feed/components/time-line/custom-timeline-chart/CustomTimeLineChartComponent.ts

@@ -68,7 +68,7 @@ export class CustomTimelineChartComponent implements OnInit, OnDestroy, OnChange
     ngOnInit(): void {
         this.computeDimensions();
         this.updateRealtime();
-        window.addEventListener('resize', this.onWindowResize);
+        globalThis.addEventListener('resize', this.onWindowResize);
     }
 
     ngOnChanges(): void {
@@ -180,6 +180,6 @@ export class CustomTimelineChartComponent implements OnInit, OnDestroy, OnChange
     ngOnDestroy() {
         this.timeLineView.destroy();
         this.isDestroyed = true;
-        window.removeEventListener('resize', this.onWindowResize);
+        globalThis.removeEventListener('resize', this.onWindowResize);
     }
 }

frontend/src/app/services/logs/RemoteLoggerService.ts

@@ -30,7 +30,7 @@ export class RemoteLoggerService {
                 RemoteLoggerService.isActive = true;
                 RemoteLoggerService.regularlyFlush();
                 RemoteLoggerService.postLog(
-                    'Remote log activated - ' + packageInfo.opfabVersion + ' - ' + window.navigator.userAgent
+                    'Remote log activated - ' + packageInfo.opfabVersion + ' - ' + globalThis.navigator.userAgent
                 );
             }
         } else {