Update onResume to only refresh screen when on the HomePage

RADAR-base · Jan 9, 2025 · b9a4d74 · b9a4d74
1 parent 6942339
commit b9a4d74
Showing 1 changed file with 15 additions and 5 deletions.
diff --git a/src/app/pages/home/containers/home-page.component.ts b/src/app/pages/home/containers/home-page.component.ts
@@ -56,10 +56,9 @@ export class HomePageComponent implements OnInit, OnDestroy {
     private platform: Platform,
     private usage: UsageService
   ) {
-    this.resumeListener = this.platform.resume.subscribe(() => this.onResume())
     this.changeDetectionListener =
       this.tasksService.changeDetectionEmitter.subscribe(() => {
-        console.log('Changes to task service detected')
+        console.log('Changes to task service detected..')
         this.navCtrl.navigateRoot('')
       })
   }
@@ -89,7 +88,10 @@ export class HomePageComponent implements OnInit, OnDestroy {
   }
 
   ngOnDestroy() {
-    this.resumeListener.unsubscribe()
+    // Unsubscribe to avoid memory leaks when the page is left
+    if (this.resumeListener) {
+      this.resumeListener.unsubscribe();
+    }
     this.changeDetectionListener.unsubscribe()
   }
 
@@ -100,6 +102,14 @@ export class HomePageComponent implements OnInit, OnDestroy {
     this.sortedTasks = this.tasksService.getValidTasksMap()
     this.tasks = this.tasksService.getTasksOfToday()
     this.showCalendar = false
+    this.resumeListener = this.platform.resume.subscribe(() => this.onResume())
+  }
+
+  ionViewWillLeave() {
+    // Unsubscribe to avoid memory leaks when the page is left
+    if (this.resumeListener) {
+      this.resumeListener.unsubscribe();
+    }
   }
 
   init() {
@@ -198,7 +208,7 @@ export class HomePageComponent implements OnInit, OnDestroy {
         buttons: [
           {
             text: this.localization.translateKey(LocKeys.BTN_OKAY),
-            handler: () => {}
+            handler: () => { }
           }
         ]
       })
@@ -216,7 +226,7 @@ export class HomePageComponent implements OnInit, OnDestroy {
       buttons: [
         {
           text: this.localization.translateKey(LocKeys.BTN_OKAY),
-          handler: () => {}
+          handler: () => { }
         }
       ]
     })