Changeset 53548 for trunk/tests/phpunit/tests/sitemaps/wpSitemapsPosts.php

Timestamp:

06/21/2022 01:16:10 PM (3 years ago)

Author:

audrasjb

Message:

Sitemaps: Remove duplicate sticky Posts from Sitemap Post Query.

This changeset sets the ignore_sticky_posts parameter to true in the default arguments passed to wp_sitemaps_posts_query_args.

Props RavanH, pbiron, swissspidy, audrasjb, SergeyBiryukov.
Fixes #55633.

File:

• trunk/tests/phpunit/tests/sitemaps/wpSitemapsPosts.php (modified) (1 diff)

trunk/tests/phpunit/tests/sitemaps/wpSitemapsPosts.php

@@ -64 +64 @@
 
     /**
+     * Tests that sticky posts are not moved to the front of the first page of the post sitemap.
+     *
+     * @ticket 55633
+     */
+    public function test_posts_sticky_posts() {
+        $factory = self::factory();
+
+        // Create 4 posts, and stick the last one.
+        $post_ids     = $factory->post->create_many( 4 );
+        $last_post_id = end( $post_ids );
+        stick_post( $last_post_id );
+
+        $posts_provider = new WP_Sitemaps_Posts();
+
+        $url_list = $posts_provider->get_url_list( 1, 'post' );
+
+        $this->assertCount( count( $post_ids ), $url_list );
+        // Check that the URL list is still in the order of the post IDs (i.e., sticky post wasn't moved to the front).
+        foreach ( $post_ids as $idx => $post_id ) {
+            $this->assertSame( array( 'loc' => home_url( "?p={$post_id}" ) ), $url_list[ $idx ] );
+        }
+    }
+
+    /**
      * Callback for 'wp_sitemaps_posts_show_on_front_entry' filter.
      */