Merge pull request #7697 from NicolasBuquet/develop

Release kMXKAccountAPNSActivityDidChangeNotification observer when de…
element-hq · Oct 25, 2023 · 709f078 · 709f078
2 parents 94dc313 + d50b82a
commit 709f078
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/Riot/Modules/Settings/SettingsViewController.m b/Riot/Modules/Settings/SettingsViewController.m
@@ -340,6 +340,11 @@ - (void)finalizeInit
     self.screenTracker = [[AnalyticsScreenTracker alloc] initWithScreen:AnalyticsScreenSettings];
 }
 
+- (void)dealloc {
+    // Fix for destroy not being called
+    [self destroy];
+}
+
 - (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 @@
+Call destroy in dealloc to remove all observers of SettingsViewController
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Call destroy in dealloc to remove all observers of SettingsViewController