Fix logged-out logic to de-register device from Push Notifications

fdocr · fdocr · commit c611fa1a7cd9 · 2020-11-23T09:02:01.000-06:00
diff --git a/DEV-Simple/ViewController.swift b/DEV-Simple/ViewController.swift
@@ -169,7 +169,7 @@ class ViewController: UIViewController {
                 // Clear out the subscription because it failed so it can try again next time
                 pushNotificationSubscription = ""
             }
-        } else if !pushNotificationSubscription.isEmpty {
+        } else if webView.userData?.userID == nil && !pushNotificationSubscription.isEmpty {
             // This means we had already subscribed to an interest for the logged-in user but
             // since `webView.userData` is now nil the user has just logged out.
             do {

Original file line number	Diff line number	Diff line change
`@@ -169,7 +169,7 @@ class ViewController: UIViewController {`
`169`	`169`	`// Clear out the subscription because it failed so it can try again next time`
`170`	`170`	`pushNotificationSubscription = ""`
`171`	`171`	`}`
`172`		`- } else if !pushNotificationSubscription.isEmpty {`
	`172`	`+ } else if webView.userData?.userID == nil && !pushNotificationSubscription.isEmpty {`
`173`	`173`	`// This means we had already subscribed to an interest for the logged-in user but`
`174`	`174`	// since `webView.userData` is now nil the user has just logged out.
`175`	`175`	`do {`