Import: setting WP_IMPORTING to true when doing an import (#40563)

* Setting WP_IMPORTING while import
Automattic · Dec 11, 2024 · fac4d21 · fac4d21
1 parent cb69d68
commit fac4d21
Show file tree

Hide file tree

Showing 12 changed files with 41 additions and 0 deletions.
diff --git a/projects/packages/import/changelog/fix-wp-import-issue b/projects/packages/import/changelog/fix-wp-import-issue
@@ -0,0 +1,4 @@
+Significance: patch
+Type: fixed
+
+Import: setting WP_IMPORTING when doing an import
diff --git a/projects/packages/import/src/endpoints/class-attachment.php b/projects/packages/import/src/endpoints/class-attachment.php
@@ -105,6 +105,8 @@ public function register_routes() {
 	 * @return WP_REST_Response|WP_Error Response object on success, WP_Error object on failure.
 	 */
 	public function create_item( $request ) {
+		// Set the WP_IMPORTING constant to prevent sync notifications
+		$this->set_importing();
 		$file_info  = $this->get_file_info( $request );
 		$attachment = $this->get_attachment_by_file_info( $file_info );
 		if ( $attachment ) {

diff --git a/projects/packages/import/src/endpoints/class-comment.php b/projects/packages/import/src/endpoints/class-comment.php
@@ -50,6 +50,8 @@ public function __construct() {
 	 * @return WP_REST_Response|WP_Error Response object on success, or error object on failure.
 	 */
 	public function create_item( $request ) {
+		// Set the WP_IMPORTING constant to prevent sync notifications
+		$this->set_importing();
 		// Resolve comment post ID.
 		if ( ! empty( $request['post'] ) ) {
 			$posts = \get_posts( $this->get_import_db_query( $request['post'] ) );

diff --git a/projects/packages/import/src/endpoints/class-custom-css.php b/projects/packages/import/src/endpoints/class-custom-css.php
@@ -51,6 +51,8 @@ public function __construct() {
 	 * @return WP_REST_Response|WP_Error Response object on success, or WP_Error object on failure.
 	 */
 	public function create_item( $request ) {
+		// Set the WP_IMPORTING constant to prevent sync notifications
+		$this->set_importing();
 		$args = array(
 			'stylesheet' => $request['title'],
 		);

diff --git a/projects/packages/import/src/endpoints/class-global-style.php b/projects/packages/import/src/endpoints/class-global-style.php
@@ -71,6 +71,9 @@ public function add_additional_fields_schema( $schema ) {
 	 * @return WP_REST_Response|WP_Error Response object on success, or WP_Error object on failure.
 	 */
 	public function create_item( $request ) {
+		// Set the WP_IMPORTING constant to prevent sync notifications
+		$this->set_importing();
+
 		if ( ! class_exists( 'WP_Theme_JSON_Resolver' ) ) {
 			require_once ABSPATH . 'wp-includes/class-wp-theme-json-resolver.php';
 		}

diff --git a/projects/packages/import/src/endpoints/class-menu-item.php b/projects/packages/import/src/endpoints/class-menu-item.php
@@ -67,6 +67,9 @@ public function add_additional_fields_schema( $schema ) {
 	 * @return WP_REST_Response|WP_Error Response object on success, or WP_Error object on failure.
 	 */
 	public function create_item( $request ) {
+		// Set the WP_IMPORTING constant to prevent sync notifications
+		$this->set_importing();
+
 		if ( ! empty( $request['menus'] ) ) {
 			$menu_id = \term_exists( $request['menus'], 'nav_menu' );
 

diff --git a/projects/packages/import/src/endpoints/class-menu.php b/projects/packages/import/src/endpoints/class-menu.php
@@ -50,6 +50,9 @@ public function __construct() {
 	 * @return WP_REST_Response|WP_Error Response object on success, or WP_Error object on failure.
 	 */
 	public function create_item( $request ) {
+		// Set the WP_IMPORTING constant to prevent sync notifications
+		$this->set_importing();
+
 		$response = parent::create_item( $request );
 
 		// Ensure that the HTTP status is a valid one.

diff --git a/projects/packages/import/src/endpoints/class-page.php b/projects/packages/import/src/endpoints/class-page.php
@@ -30,6 +30,9 @@ public function __construct() {
 	 * @return WP_REST_Response|WP_Error Response object on success, or WP_Error object on failure.
 	 */
 	public function create_item( $request ) {
+		// Set the WP_IMPORTING constant to prevent sync notifications
+		$this->set_importing();
+
 		if ( ! empty( $request['parent'] ) ) {
 			$pages = \get_pages( $this->get_import_db_query( $request['parent'] ) );
 

diff --git a/projects/packages/import/src/endpoints/class-post.php b/projects/packages/import/src/endpoints/class-post.php
@@ -74,6 +74,9 @@ public function add_additional_fields_schema( $schema ) {
 	 * @return WP_REST_Response|WP_Error Response object on success, or WP_Error object on failure.
 	 */
 	public function create_item( $request ) {
+		// Set the WP_IMPORTING constant to prevent sync notifications
+		$this->set_importing();
+
 		// Skip if the post already exists.
 		$post_id = \post_exists(
 			$request['title'],

diff --git a/projects/packages/import/src/endpoints/class-template.php b/projects/packages/import/src/endpoints/class-template.php
@@ -47,6 +47,9 @@ public function __construct() {
 	 * @return WP_REST_Response|WP_Error Response object on success, or WP_Error object on failure.
 	 */
 	public function create_item( $request ) {
+		// Set the WP_IMPORTING constant to prevent sync notifications
+		$this->set_importing();
+
 		$response = parent::create_item( $request );
 
 		return $this->add_import_id_metadata( $request, $response );

diff --git a/projects/packages/import/src/endpoints/trait-import.php b/projects/packages/import/src/endpoints/trait-import.php
@@ -94,4 +94,13 @@ protected function ensure_http_status( $response, $error_code, $status ) {
 
 		return $response;
 	}
+
+	/**
+	 * Set the importing constant.
+	 */
+	public function set_importing() {
+		if ( ! defined( 'WP_IMPORTING' ) ) {
+			define( 'WP_IMPORTING', true );
+		}
+	}
 }
diff --git a/projects/plugins/jetpack/changelog/fix-wp-import-issue b/projects/plugins/jetpack/changelog/fix-wp-import-issue
@@ -0,0 +1,4 @@
+Significance: patch
+Type: bugfix
+
+Import: setting WP_IMPORTING when doing an import