Introduce abandoned status

this should help us with edge cases to permit onboarding re-entry so it's not a black and white case as previous

Author: wpalani

includes/Services/AppService.php

@@ -56,6 +56,9 @@ public function complete( string $selected_sitegen_homepage ): void {
 		// Trash Preview pages.
 		PreviewsService::trash_preview_pages();
 
+		// Mark onboarding as completed.
+		StatusService::handle_completed();
+
 		// Purge all caches.
 		container()->get( 'cachePurger' )->purge_all();
 

includes/Services/StatusService.php

@@ -28,6 +28,17 @@ public static function handle_started(): bool {
 		return false;
 	}
 
+	/**
+	 * Handles Onboarding abandoned event.
+	 *
+	 * @return void
+	 */
+	public static function handle_abandoned(): void {
+		if ( 'started' === get_option( Options::get_option_name( 'status' ) ) ) {
+			update_option( Options::get_option_name( 'status' ), 'abandoned' );
+		}
+	}
+
 	/**
 	 * Handles Onboarding completed event.
 	 *
@@ -164,22 +175,17 @@ public static function update_onboarding_restart_status(): void {
 	 * @return void
 	 */
 	public static function track(): void {
-		global $pagenow;
-
+		// Ignore if the request is an AJAX request.
 		if ( defined( 'DOING_AJAX' ) && DOING_AJAX ) {
 			return;
 		}
 
-		switch ( $pagenow ) {
-			case 'index.php':
-				// If the page is not nfd-onboarding.
-				//phpcs:ignore
-				if ( isset( $_GET['page'] ) && WP_Admin::$slug !== \sanitize_text_field( $_GET['page'] ) ) {
-					self::handle_completed();
-				}
-				break;
-			default:
-				self::handle_completed();
+		// Ignore if the request is not for the onboarding page.
+		if ( isset( $_GET['page'] ) && WP_Admin::$slug === \sanitize_text_field( $_GET['page'] ) ) {
+			return;
 		}
+
+		// Handle abandoned event.
+		self::handle_abandoned();
 	}
 }