Release kMXKAccountAPNSActivityDidChangeNotification observer when de…

…allocation controller Signed-off-by: Nicolas Buquet <[email protected]>
element-hq · Oct 10, 2023 · 6b8add7 · 6b8add7
1 parent 94dc313
commit 6b8add7
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/Riot/Modules/Settings/SettingsViewController.m b/Riot/Modules/Settings/SettingsViewController.m
@@ -340,6 +340,15 @@ - (void)finalizeInit
     self.screenTracker = [[AnalyticsScreenTracker alloc] initWithScreen:AnalyticsScreenSettings];
 }
 
+- (void)dealloc {
+    // Remove observers
+    if (pushInfoUpdateObserver)
+    {
+        [[NSNotificationCenter defaultCenter] removeObserver:pushInfoUpdateObserver name:kMXKAccountAPNSActivityDidChangeNotification object:nil];
+        pushInfoUpdateObserver = nil;
+    }
+}
+
 - (void)updateSections
 {
     NSMutableArray<Section*> *tmpSections = [NSMutableArray arrayWithCapacity:SECTION_TAG_DEACTIVATE_ACCOUNT + 1];

diff --git a/changelog.d/Settings-controller-dealloc-observer.bugfix b/changelog.d/Settings-controller-dealloc-observer.bugfix
@@ -0,0 +1 @@
+Remove registered observer kMXKAccountAPNSActivityDidChangeNotification in dealloc of SettingsViewController