fix: add sub to explorer

artemnih · Mar 3, 2022 · c1dcb6d · c1dcb6d
1 parent 9ea408e
commit c1dcb6d
Showing 1 changed file with 10 additions and 5 deletions.
diff --git a/projects/ngx-explorer/src/lib/components/explorer/explorer.component.ts b/projects/ngx-explorer/src/lib/components/explorer/explorer.component.ts
@@ -1,5 +1,5 @@
-import { Component, Inject, ViewEncapsulation } from '@angular/core';
-import { BehaviorSubject } from 'rxjs';
+import { Component, Inject, OnDestroy, ViewEncapsulation } from '@angular/core';
+import { BehaviorSubject, Subscription } from 'rxjs';
 import { AvialableView } from '../../common/types';
 import { CURRENT_VIEW } from '../../injection-tokens/current-view.token';
 
@@ -9,15 +9,20 @@ import { CURRENT_VIEW } from '../../injection-tokens/current-view.token';
     styleUrls: ['./explorer.component.scss'],
     encapsulation: ViewEncapsulation.None
 })
-export class ExplorerComponent {
+export class ExplorerComponent implements OnDestroy {
 
     public avialableView = AvialableView;
     public view: string;
+    private sub = new Subscription();
 
     constructor(@Inject(CURRENT_VIEW) private currentView: BehaviorSubject<AvialableView>) {
-        this.currentView.subscribe(view => {
+        this.sub.add(this.currentView.subscribe(view => {
             this.view = view;
-        })
+        }));
     }
 
+    ngOnDestroy() {
+        this.sub.unsubscribe();
+    }
+
 }