fraport/fradrive-old
fraport/fradrive-old
Archived
3
1
Fork 0
Code Issues 65 Pull Requests 5 Actions Packages Projects Releases 197 Wiki Activity

chore(alert): removing the interceptor when destroying the util

Browse Source
This commit is contained in:
Johannes Eder 2021-08-03 17:35:10 +02:00 committed by Sarah Vaupel
parent 0823df33b5
commit fc7d5dc94e
2 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
frontend/src/utils/alerts/alerts.js
View File

@ -34,6 +34,7 @@ export class Alerts {
_app;
_eventManager;
_boundResponseInterceptor;
constructor(element, app) {
if (!element) {
@ -44,6 +45,7 @@ export class Alerts {
this._app = app;
this._eventManager = new EventManager();
this._boundResponseInterceptor = this._responseInterceptor.bind(this);
if (this._element.classList.contains(ALERTS_INITIALIZED_CLASS)) {
return false;
@ -68,6 +70,7 @@ export class Alerts {
destroy() {
this._eventManager.removeAllEventListenersFromUtil();
this._app.httpClient.removeResponseInterceptor(this._boundResponseInterceptor);
if(this._alertElements) {
this._alertElements.forEach(element => element.remove());
@ -135,7 +138,7 @@ export class Alerts {
}
_setupHttpInterceptor() {
this._app.httpClient.addResponseInterceptor(this._responseInterceptor.bind(this));
this._app.httpClient.addResponseInterceptor(this._boundResponseInterceptor);
}
_elevateAlerts() {

1
frontend/src/utils/alerts/alerts.spec.js
View File

@ -3,6 +3,7 @@ import { Alerts, ALERTS_INITIALIZED_CLASS } from './alerts';
const MOCK_APP = {
httpClient: {
addResponseInterceptor: () => {},
removeResponseInterceptor: () => {},
},
};