Changeset 47760 for trunk/tests/phpunit/tests/canonical/paged.php

Timestamp:

05/04/2020 10:40:06 AM (5 years ago)

Author:

SergeyBiryukov

Message:

Canonical: Redirect paged requests for a static page assigned as the "Posts page".

This avoids displaying duplicate content of the home page under different URLs with appended page numbers.

This change only affects the <!--nextpage--> pagination (page query variable) and not the regular multiple posts pagination (paged query variable).

The posts page does not support the <!--nextpage--> pagination, so requests for invalid page numbers should be redirected to the page permalink, applying the logic previously implemented for single posts or pages.

Follow-up to [34492], [47727].

Props jeremyfelt, sachit.tandukar, SergeyBiryukov.
Fixes #45337. See #40773, #28081, #11694.

File:

• trunk/tests/phpunit/tests/canonical/paged.php (modified) (2 diffs)

trunk/tests/phpunit/tests/canonical/paged.php

@@ -7 +7 @@
 class Tests_Canonical_Paged extends WP_Canonical_UnitTestCase {
 
-    function test_nextpage() {
+    function test_redirect_canonical_with_nextpage_pagination() {
         $para = 'This is a paragraph.
             This is a paragraph.
@@ -20 +20 @@
         );
 
-        $link  = parse_url( get_permalink( $post_id ), PHP_URL_PATH );
-        $paged = $link . '4/';
+        $link = parse_url( get_permalink( $post_id ), PHP_URL_PATH );
 
-        $this->assertCanonical( $paged, $link );
+        // Existing page should be displayed as is.
+        $this->assertCanonical( $link . '3/', $link . '3/' );
+        // Non-existing page should redirect to the permalink.
+        $this->assertCanonical( $link . '4/', $link );
     }
+
 }