Changeset 1830 for trunk/bb-includes/functions.bb-core.php

Timestamp:

12/08/2008 05:02:16 PM (18 years ago)

Author:

sambauers

Message:

Allow bb_paginate_links() to deal with ?page=# format correctly without relying on bb_repermalink() to fix the URL first. This helps plugins that want to use the function. Props _ck_ for investigations, fixes #1003

File:

• trunk/bb-includes/functions.bb-core.php (modified) (2 diffs)

trunk/bb-includes/functions.bb-core.php

@@ -219 +219 @@
         $dots = false;
 
+        $empty_format = '';
+        if ( strpos( $format, '?' ) === 0 ) {
+                $empty_format = '?';
+        }
+
         if ( $prev_next && $current && 1 < $current ) {
-                $link = str_replace( '%_%', 2 == $current ? '' : $format, $base );
+                $link = str_replace( '%_%', 2 == $current ? $empty_format : $format, $base );
                 $link = str_replace( '%#%', $current - 1, $link );
+                $link = str_replace( '?&', '?', $link );
                 if ( $add_args )
                         $link = add_query_arg( $add_args, $link );
@@ -236 +242 @@
                 } else {
                         if ( $show_all || ( $n <= $end_size || ( $current && $n >= $current - $mid_size && $n <= $current + $mid_size ) || $n > $total - $end_size ) ) {
-                                $link = str_replace( '%_%', 1 == $n ? '' : $format, $base );
+                                $link = str_replace( '%_%', 1 == $n ? $empty_format : $format, $base );
                                 $link = str_replace( '%#%', $n, $link );
+                                $link = str_replace( '?&', '?', $link );
                                 if ( $add_args )
                                         $link = add_query_arg( $add_args, $link );