refactor: align presentLoginEpilogue behavior

Since the Jetpack app allows site creation, we don’t need to override presentLoginEpilogue anymore.

Author: Momo Ozawa

WordPress/Classes/Utility/App Configuration/AuthenticationHandler.swift

@@ -2,14 +2,6 @@ import WordPressAuthenticator
 
 protocol AuthenticationHandler {
 
-    /// Whether or not the AuthenticationHandler will override or handle the `presentLoginEpilogue` method.
-    /// If this returns true, the `AuthenticationHandler.presentLoginEpilogue` method is called
-    /// If not, then the default implementation will be called instead
-    /// - Returns: Bool, true if we should override the functionality, false if we should not
-    func willHandlePresentLoginEpilogue(in navigationController: UINavigationController, for credentials: AuthenticatorCredentials) -> Bool
-
-    func presentLoginEpilogue(in navigationController: UINavigationController, for credentials: AuthenticatorCredentials, windowManager: WindowManager, onDismiss: @escaping () -> Void) -> Bool
-
     /// Whether or not the AuthenticationHandler will override or handle the `presentSignupEpilogue` method.
     /// If this returns true, the `AuthenticationHandler.presentSignupEpilogue` method is called
     /// If not, then the default implementation will be called instead

WordPress/Classes/ViewRelated/NUX/WordPressAuthenticationManager.swift

@@ -317,11 +317,6 @@ extension WordPressAuthenticationManager: WordPressAuthenticatorDelegate {
     /// Presents the Login Epilogue, in the specified NavigationController.
     ///
     func presentLoginEpilogue(in navigationController: UINavigationController, for credentials: AuthenticatorCredentials, onDismiss: @escaping () -> Void) {
-        if let authenticationHandler = authenticationHandler,
-           authenticationHandler.willHandlePresentLoginEpilogue(in: navigationController, for: credentials),
-           authenticationHandler.presentLoginEpilogue(in: navigationController, for: credentials, windowManager: windowManager, onDismiss: onDismiss) {
-            return
-        }
 
         // If adding a self-hosted site, skip the Epilogue
         if let wporg = credentials.wporg,

WordPress/Jetpack/Classes/JetpackAuthenticationManager.swift

@@ -9,28 +9,6 @@ struct JetpackAuthenticationManager: AuthenticationHandler {
     var prologuePrimaryButtonStyle: NUXButtonStyle? = JetpackPrologueStyleGuide.continueButtonStyle
     var prologueSecondaryButtonStyle: NUXButtonStyle? = JetpackPrologueStyleGuide.siteAddressButtonStyle
 
-    func willHandlePresentLoginEpilogue(in navigationController: UINavigationController, for credentials: AuthenticatorCredentials) -> Bool {
-        // Don't display the "no sites" epilogue if we allow site creation
-        return !AppConfiguration.allowSiteCreation
-    }
-
-    func presentLoginEpilogue(in navigationController: UINavigationController, for credentials: AuthenticatorCredentials, windowManager: WindowManager, onDismiss: @escaping () -> Void) -> Bool {
-        if AccountHelper.hasBlogs {
-            return false
-        }
-
-        // Exit out of the sign in process, if we don't do this we later can't
-        // display the sign in again
-        windowManager.dismissFullscreenSignIn()
-
-        // Display the no sites view
-        let viewModel = JetpackNoSitesErrorViewModel()
-        let controller = errorViewController(with: viewModel)
-        windowManager.show(controller, completion: nil)
-
-        return true
-    }
-
     func willHandlePresentSignupEpilogue(in navigationController: UINavigationController, for credentials: AuthenticatorCredentials, service: SocialService?) -> Bool {
         // Don't display the "no sites" epilogue if we allow site creation
         return !AppConfiguration.allowSiteCreation