feat: only query for local users

Automattic · Feb 21, 2024 · 111a7b4 · 111a7b4
1 parent b74a619
commit 111a7b4
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/includes/cli/class-data-backfill.php b/includes/cli/class-data-backfill.php
@@ -7,7 +7,6 @@
 
 namespace Newspack_Network;
 
-use Automattic\Jetpack\VideoPress\Data;
 use WP_CLI;
 
 /**
@@ -112,6 +111,12 @@ private static function process_reader_registered( $start, $end, $live, $verbose
 				],
 				'fields'     => [ 'id', 'user_email', 'user_registered' ],
 				'number'     => -1,
+				'meta_query' => [ // phpcs:ignore WordPress.DB.SlowDBQuery.slow_db_query_meta_query
+					[
+						'key'     => \Newspack_Network\Utils\Users::USER_META_REMOTE_SITE,
+						'compare' => 'NOT EXISTS',
+					],
+				],
 			]
 		);
 		if ( ! $verbose ) {
@@ -357,6 +362,4 @@ public static function data_backfill( $args, $assoc_args ) { // phpcs:ignore Gen
 		}
 		WP_CLI::line( '' );
 	}
-
-	// TODO: a script to run the above on all nodes of the network.
 }
diff --git a/includes/utils/class-users.php b/includes/utils/class-users.php
@@ -13,6 +13,8 @@
  * Class to watch the user for updates and trigger events
  */
 class Users {
+	const USER_META_REMOTE_SITE = 'newspack_remote_site';
+	const USER_META_REMOTE_ID = 'newspack_remote_id';
 
 	/**
 	 * Gets an existing user or creates a user propagated from another site in the Network
@@ -48,8 +50,8 @@ public static function get_or_create_user_by_email( $email, $remote_site_url, $r
 			return $user_id;
 		}
 
-		update_user_meta( $user_id, 'newspack_remote_site', $remote_site_url );
-		update_user_meta( $user_id, 'newspack_remote_id', $remote_id );
+		update_user_meta( $user_id, self::USER_META_REMOTE_SITE, $remote_site_url );
+		update_user_meta( $user_id, self::USER_META_REMOTE_ID, $remote_id );
 
 		return get_user_by( 'id', $user_id );
 	}