From ab972ddea9faeee136eb9b73af4c3cc6385cd79e Mon Sep 17 00:00:00 2001
From: Michael Telatynski <7t3chguy@gmail.com>
Date: Mon, 10 Jul 2023 16:01:59 +0100
Subject: [PATCH] PosthogAnalytics unwatch settings on logout (#11207)

* PosthogAnalytics unwatch settings on logout

* Tidy
---
 src/Lifecycle.ts        | 6 +++---
 src/PosthogAnalytics.ts | 4 +++-
 2 files changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/Lifecycle.ts b/src/Lifecycle.ts
index 6d1be2240d6..9c4a1cfd33f 100644
--- a/src/Lifecycle.ts
+++ b/src/Lifecycle.ts
@@ -748,11 +748,12 @@ let _isLoggingOut = false;
  * Logs the current session out and transitions to the logged-out state
  */
 export function logout(): void {
-    if (!MatrixClientPeg.get()) return;
+    const client = MatrixClientPeg.get();
+    if (!client) return;
 
     PosthogAnalytics.instance.logout();
 
-    if (MatrixClientPeg.get()!.isGuest()) {
+    if (client.isGuest()) {
         // logout doesn't work for guest sessions
         // Also we sometimes want to re-log in a guest session if we abort the login.
         // defer until next tick because it calls a synchronous dispatch, and we are likely here from a dispatch.
@@ -761,7 +762,6 @@ export function logout(): void {
     }
 
     _isLoggingOut = true;
-    const client = MatrixClientPeg.get()!;
     PlatformPeg.get()?.destroyPickleKey(client.getSafeUserId(), client.getDeviceId() ?? "");
     client.logout(true).then(onLoggedOut, (err) => {
         // Just throwing an error here is going to be very unhelpful
diff --git a/src/PosthogAnalytics.ts b/src/PosthogAnalytics.ts
index 995559b1ee2..dc4e4837819 100644
--- a/src/PosthogAnalytics.ts
+++ b/src/PosthogAnalytics.ts
@@ -138,6 +138,7 @@ export class PosthogAnalytics {
     private propertiesForNextEvent: Partial<Record<"$set" | "$set_once", UserProperties>> = {};
     private userPropertyCache: UserProperties = {};
     private authenticationType: Signup["authenticationType"] = "Other";
+    private watchSettingRef?: string;
 
     public static get instance(): PosthogAnalytics {
         if (!this._instance) {
@@ -337,6 +338,7 @@ export class PosthogAnalytics {
         if (this.enabled) {
             this.posthog.reset();
         }
+        if (this.watchSettingRef) SettingsStore.unwatchSetting(this.watchSettingRef);
         this.setAnonymity(Anonymity.Disabled);
     }
 
@@ -400,7 +402,7 @@ export class PosthogAnalytics {
         //  * When the user changes their preferences on this device
         // Note that for new accounts, pseudonymousAnalyticsOptIn won't be set, so updateAnonymityFromSettings
         // won't be called (i.e. this.anonymity will be left as the default, until the setting changes)
-        SettingsStore.watchSetting(
+        this.watchSettingRef = SettingsStore.watchSetting(
             "pseudonymousAnalyticsOptIn",
             null,
             (originalSettingName, changedInRoomId, atLevel, newValueAtLevel, newValue) => {