Changeset 56939 for trunk/src/wp-includes/link-template.php

Timestamp:

10/16/2023 12:05:28 AM (2 years ago)

Author:

costdev

Message:

Posts, Post Types: Don't force trailing slash in get_pagenum_link().

Previously, a trailing slash was appended to the link returned from get_pagenum_link(). If the permalink structure didn't contain a trailing slash, this link could fail.

This change removes trailing slashes and only appends one if the site is set for adding trailing slashes.

This adds a new test file for the accompanying tests, tests/phpunit/tests/link/getPagenumLink.php, and moves an existing test for get_pagenum_link() to the same file.

Props davemad-davenet, darkfate, Nazgul, scribu, nacin, obenland, chriscct7, jesin, matthewppelsheimer, audrasjb, petitphp, mukesh27, oglekler, mai21, webtechpooja, tejwanihemant, nicolefurlan, hellofromTonya, costdev.
Fixes #2877.

File:

• trunk/src/wp-includes/link-template.php (modified) (2 diffs)

trunk/src/wp-includes/link-template.php

@@ -2432 +2432 @@
         preg_match( $qs_regex, $request, $qs_match );
 
+        $parts   = array();
+        $parts[] = untrailingslashit( get_bloginfo( 'url' ) );
+
         if ( ! empty( $qs_match[0] ) ) {
             $query_string = $qs_match[0];
@@ -2443 +2446 @@
         $request = ltrim( $request, '/' );
 
-        $base = trailingslashit( get_bloginfo( 'url' ) );
-
         if ( $wp_rewrite->using_index_permalinks() && ( $pagenum > 1 || '' !== $request ) ) {
-            $base .= $wp_rewrite->index . '/';
-        }
+            $parts[] = $wp_rewrite->index;
+        }
+
+        $parts[] = untrailingslashit( $request );
 
         if ( $pagenum > 1 ) {
-            $request = ( ( ! empty( $request ) ) ? trailingslashit( $request ) : $request ) . user_trailingslashit( $wp_rewrite->pagination_base . '/' . $pagenum, 'paged' );
-        }
-
-        $result = $base . $request . $query_string;
+            $parts[] = $wp_rewrite->pagination_base;
+            $parts[] = $pagenum;
+        }
+
+        $result = user_trailingslashit( implode( '/', array_filter( $parts ) ), 'paged' );
+        if ( ! empty( $query_string ) ) {
+            $result .= $query_string;
+        }
     }