Skip to content

Commit

Permalink
Fix Sonar issues
Browse files Browse the repository at this point in the history
  • Loading branch information
bassages committed Oct 9, 2023
1 parent f16e4a0 commit 86254a0
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 13 deletions.
13 changes: 5 additions & 8 deletions src/app/auth/authorization-interceptor.ts
Original file line number Diff line number Diff line change
Expand Up @@ -23,16 +23,13 @@ export class AuthorizationInterceptor implements HttpInterceptor {
// noinspection JSMethodCanBeStatic
private handleError(request: HttpRequest<any>, 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));
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -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;
}
Expand Down

0 comments on commit 86254a0

Please sign in to comment.