From 86254a04b4f9712df635f79dd0c97803f9919df9 Mon Sep 17 00:00:00 2001 From: Rob Date: Mon, 9 Oct 2023 19:00:42 +0200 Subject: [PATCH] Fix Sonar issues --- src/app/auth/authorization-interceptor.ts | 13 +++++-------- .../energie-verbruik-base-chart.service.ts | 8 +++----- 2 files changed, 8 insertions(+), 13 deletions(-) diff --git a/src/app/auth/authorization-interceptor.ts b/src/app/auth/authorization-interceptor.ts index 1b1035b..7a97a82 100644 --- a/src/app/auth/authorization-interceptor.ts +++ b/src/app/auth/authorization-interceptor.ts @@ -23,16 +23,13 @@ export class AuthorizationInterceptor implements HttpInterceptor { // noinspection JSMethodCanBeStatic private handleError(request: HttpRequest, httpErrorResponse: HttpErrorResponse) { if (httpErrorResponse.error instanceof ErrorEvent) { - // A client-side or network error occurred. Handle it accordingly. + // A client-side or network error occurred console.error('An error occurred:', httpErrorResponse.error.message); - } else { - - if (httpErrorResponse.status === 401) { - if (request.url !== '/api/user') { - this.authService.loggedOut(); - } - return EMPTY; + } else if (httpErrorResponse.status === 401) { + if (request.url !== '/api/user') { + this.authService.loggedOut(); } + return EMPTY; } return throwError(() => new Error(httpErrorResponse.message)); } diff --git a/src/app/energie-verbruik/energie-verbruik-base-chart.service.ts b/src/app/energie-verbruik/energie-verbruik-base-chart.service.ts index 04426e7..b71443d 100644 --- a/src/app/energie-verbruik/energie-verbruik-base-chart.service.ts +++ b/src/app/energie-verbruik/energie-verbruik-base-chart.service.ts @@ -159,11 +159,9 @@ export abstract class AbstractEnergieVerbruikHistorieService extends ChartServic } else { newEnergiesoorten.splice(indexOfToggledEnergieSoort, 1); } - } else { - if (newEnergiesoorten[0] !== energiesoortToToggle) { - newEnergiesoorten.splice(0, newEnergiesoorten.length); - newEnergiesoorten.push(energiesoortToToggle); - } + } else if (newEnergiesoorten[0] !== energiesoortToToggle) { + newEnergiesoorten.splice(0, newEnergiesoorten.length); + newEnergiesoorten.push(energiesoortToToggle); } return newEnergiesoorten; }