Fix: add a default value for siteID

Momo Ozawa · Momo Ozawa · commit 0e5d002acf0c · 2022-02-01T12:22:19.000Z
This fixes an issue where the promptWasDismissedKey was returning nil for self-hosted sites not connected to Jetpack.
diff --git a/WordPress/Classes/ViewRelated/Blog/QuickStartSettings.swift b/WordPress/Classes/ViewRelated/Blog/QuickStartSettings.swift
@@ -35,9 +35,7 @@ final class QuickStartSettings {
     }
 
     private func promptWasDismissedKey(for blog: Blog) -> String? {
-        guard let siteID = blog.dotComID?.intValue else {
-            return nil
-        }
+        let siteID = blog.dotComID?.intValue ?? 0
         return "QuickStartPromptWasDismissed-\(siteID)"
     }
 

Original file line number	Diff line number	Diff line change
`@@ -35,9 +35,7 @@ final class QuickStartSettings {`
`35`	`35`	`}`
`36`	`36`
`37`	`37`	`private func promptWasDismissedKey(for blog: Blog) -> String? {`
`38`		`- guard let siteID = blog.dotComID?.intValue else {`
`39`		`- return nil`
`40`		`- }`
	`38`	`+ let siteID = blog.dotComID?.intValue ?? 0`
`41`	`39`	`return "QuickStartPromptWasDismissed-\(siteID)"`
`42`	`40`	`}`
`43`	`41`