Skip to content

Commit 7c628cf

Browse files
committed
Add guard for checking matching site after installing Jetpack
1 parent 5e1bdac commit 7c628cf

File tree

1 file changed

+12
-13
lines changed

1 file changed

+12
-13
lines changed

WooCommerce/Classes/Authentication/AuthenticationManager.swift

Lines changed: 12 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -566,20 +566,19 @@ private extension AuthenticationManager {
566566
ServiceLocator.stores.synchronizeEntities { [weak self] in
567567
guard let self = self else { return }
568568
matcher.refreshStoredSites()
569-
if let matchedSite = matcher.matchedSite(originalURL: siteURL) {
570-
// checks if the site has woo
571-
if matchedSite.isWooCommerceActive == false {
572-
let noWooUI = self.noWooUI(for: matchedSite,
573-
with: matcher,
574-
navigationController: navigationController,
575-
onStorePickerDismiss: {})
576-
navigationController.show(noWooUI, sender: nil)
577-
} else {
578-
self.startStorePicker(with: matchedSite.siteID, in: navigationController, onDismiss: {})
579-
}
580-
} else {
569+
guard let matchedSite = matcher.matchedSite(originalURL: siteURL) else {
581570
DDLogWarn("⚠️ Could not find \(siteURL) connected to the account")
582-
// TODO: what now?
571+
return
572+
}
573+
// checks if the site has woo
574+
if matchedSite.isWooCommerceActive == false {
575+
let noWooUI = self.noWooUI(for: matchedSite,
576+
with: matcher,
577+
navigationController: navigationController,
578+
onStorePickerDismiss: {})
579+
navigationController.show(noWooUI, sender: nil)
580+
} else {
581+
self.startStorePicker(with: matchedSite.siteID, in: navigationController, onDismiss: {})
583582
}
584583
}
585584
})

0 commit comments

Comments
 (0)