From f6c58f17ed0cc93a5c8521fa5a20ecda60218967 Mon Sep 17 00:00:00 2001
From: Alain Schlesser <alain.schlesser@gmail.com>
Date: Sun, 25 Jul 2021 16:46:22 +0200
Subject: [PATCH] Ensure post attachment meta is exported correctly

---
 composer.json           |  1 +
 features/export.feature | 56 +++++++++++++++++++++++++----------------
 src/WP_Export_Query.php |  6 ++---
 3 files changed, 38 insertions(+), 25 deletions(-)

diff --git a/composer.json b/composer.json
index 70bff8c4..bc366571 100644
--- a/composer.json
+++ b/composer.json
@@ -19,6 +19,7 @@
         "wp-cli/db-command": "^1.3 || ^2",
         "wp-cli/entity-command": "^1.3 || ^2",
         "wp-cli/extension-command": "^1.2 || ^2",
+        "wp-cli/media-command": "^1 || ^2",
         "wp-cli/import-command": "^1 || ^2",
         "wp-cli/wp-cli-tests": "^3.0.11"
     },
diff --git a/features/export.feature b/features/export.feature
index 45a2bbbc..2df5d467 100644
--- a/features/export.feature
+++ b/features/export.feature
@@ -736,52 +736,36 @@ Feature: Export content.
 
   Scenario: Export individual post with attachments
     Given a WP install
-
-    When I run `wp plugin install wordpress-importer --activate`
-    Then STDOUT should contain:
-      """
-      Success:
-      """
+    And I run `wp plugin install wordpress-importer --activate`
+    And I run `wp site empty --yes`
 
     When I run `wp post generate --count=10`
     And I run `wp post list --format=count`
     Then STDOUT should be:
       """
-      11
+      10
       """
 
     When I run `wp post create --post_title='Post with attachment to export' --porcelain`
     Then STDOUT should be a number
     And save STDOUT as {EXPORT_ATTACHMENT_POST_ID}
 
-    When I run `wp post create --post_type=attachment --porcelain`
+    When I run `wp media import 'http://wp-cli.org/behat-data/codeispoetry.png' --post_id={EXPORT_ATTACHMENT_POST_ID} --porcelain`
     Then STDOUT should be a number
     And save STDOUT as {EXPORT_ATTACHMENT_ID}
 
-    When I run `wp post update {EXPORT_ATTACHMENT_ID} --post_parent={EXPORT_ATTACHMENT_POST_ID} --porcelain`
-    Then STDOUT should contain:
-      """
-      Success: Updated post {EXPORT_ATTACHMENT_ID}
-      """
-
     When I run `wp post create --post_title='Post with attachment to ignore' --porcelain`
     Then STDOUT should be a number
     And save STDOUT as {IGNORE_ATTACHMENT_POST_ID}
 
-    When I run `wp post create --post_type=attachment --porcelain`
+    When I run `wp media import 'http://wp-cli.org/behat-data/white-150-square.jpg' --post_id={IGNORE_ATTACHMENT_POST_ID} --porcelain`
     Then STDOUT should be a number
     And save STDOUT as {IGNORE_ATTACHMENT_ID}
 
-    When I run `wp post update {IGNORE_ATTACHMENT_ID} --post_parent={IGNORE_ATTACHMENT_POST_ID} --porcelain`
-    Then STDOUT should contain:
-      """
-      Success: Updated post {IGNORE_ATTACHMENT_ID}
-      """
-
     When I run `wp post list --post_type=post --format=count`
     Then STDOUT should be:
       """
-      13
+      12
       """
 
     When I run `wp post list --post_type=attachment --format=count`
@@ -804,6 +788,26 @@ Feature: Export content.
       """
       <wp:post_id>{EXPORT_ATTACHMENT_ID}</wp:post_id>
       """
+    And the {EXPORT_FILE} file should contain:
+      """
+      <wp:meta_key>_wp_attachment_metadata</wp:meta_key>
+      """
+    And the {EXPORT_FILE} file should contain:
+      """
+      codeispoetry.png";s:5:"sizes"
+      """
+    And the {EXPORT_FILE} file should contain:
+      """
+      <wp:meta_key>_wp_attached_file</wp:meta_key>
+      """
+    And the {EXPORT_FILE} file should contain:
+      """
+      codeispoetry.png]]></wp:meta_value>
+      """
+    And the {EXPORT_FILE} file should not contain:
+      """
+      <wp:meta_key>_edit_lock</wp:meta_key>
+      """
     And the {EXPORT_FILE} file should not contain:
       """
       <wp:post_id>{IGNORE_ATTACHMENT_POST_ID}</wp:post_id>
@@ -812,6 +816,14 @@ Feature: Export content.
       """
       <wp:post_id>{IGNORE_ATTACHMENT_ID}</wp:post_id>
       """
+    And the {EXPORT_FILE} file should not contain:
+      """
+      white-150-square.jpg]]></wp:meta_value>
+      """
+    And the {EXPORT_FILE} file should not contain:
+      """
+      white-150-square.jpg";s:5:"sizes"
+      """
 
   Scenario: Export categories, tags and terms
     Given a WP install
diff --git a/src/WP_Export_Query.php b/src/WP_Export_Query.php
index 727a3091..262f6e6e 100644
--- a/src/WP_Export_Query.php
+++ b/src/WP_Export_Query.php
@@ -387,11 +387,11 @@ private static function get_meta_for_post( $post ) {
 		$meta_for_export = [];
 		$meta_from_db    = $wpdb->get_results( $wpdb->prepare( "SELECT * FROM $wpdb->postmeta WHERE post_id = %d", $post->ID ) );
 		foreach ( $meta_from_db as $meta ) {
-			// phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedHooknameFound -- Calling native WordPress hook.
-			if ( apply_filters( 'wxr_export_skip_postmeta', false, $meta->meta_key, $meta ) ) {
+			if ( '_edit_lock' === $meta->meta_key ) {
 				continue;
 			}
-			if ( in_array( $meta->meta_key, [ '_edit_lock', '_wp_attachment_metadata', '_wp_attached_file' ], true ) ) {
+			// phpcs:ignore WordPress.NamingConventions.PrefixAllGlobals.NonPrefixedHooknameFound -- Calling native WordPress hook.
+			if ( apply_filters( 'wxr_export_skip_postmeta', false, $meta->meta_key, $meta ) ) {
 				continue;
 			}
 			$meta_for_export[] = $meta;