Changeset 13089 for sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-markdown/inc/class-importer.php

Timestamp:

01/05/2024 01:25:28 AM (3 years ago)

Author:

dd32

Message:

Markdown Importer: Don't constantly update posts which have fields derrived from the manifest.

This avoids calling wp_update_post() with no changes, which causes the post_modified dates to be bumped every 15 minutes.

See ​https://github.com/WordPress/wporg-developer/issues/456

File:

• sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-markdown/inc/class-importer.php (modified) (1 diff)

sites/trunk/wordpress.org/public_html/wp-content/plugins/wporg-markdown/inc/class-importer.php

@@ -397 +397 @@
                 }
 
+                // Check to see if this differs from the actual post.
+                if ( $post_data ) {
+                        $post = get_post( $post_id );
+                        foreach ( $post_data as $field => $value ) {
+                                if ( $value == $post->$field ) {
+                                        // Unset it to prevent calling `wp_update_post()` with this and bumping the last_modified date.
+                                        unset( $post_data[ $field ] );
+                                }
+                        }
+                }
+
                 if ( $do_update && $post_data ) {
                         $post_data['ID'] = $post_id;