Paginate_links : First link and previous link are break

[nexurium]

I have found a little bug into paginate_link function.

The first link and the previous link ( when current is the second link ) are broken.

You can find my correction below :

Index: wp-includes/general-template.php
===================================================================
--- wp-includes/general-template.php	(3.9.1)
+++ wp-includes/general-template.php	(copie de travail)
@@ -2456,7 +2456,7 @@
 	$dots = false;
 
 	if ( $prev_next && $current && 1 < $current ) :
-		$link = str_replace('%_%', 2 == $current ? '' : $format, $base);
+		$link = str_replace('%_%', 1 == $current ? '' : $format, $base);
 		$link = str_replace('%#%', $current - 1, $link);
 		if ( $add_args )
 			$link = add_query_arg( $add_args, $link );
@@ -2477,7 +2477,7 @@
 			$dots = true;
 		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('%_%', $current == $n ? '' : $format, $base);
 				$link = str_replace('%#%', $n, $link);
 				if ( $add_args )
 					$link = add_query_arg( $add_args, $link );

[SergeyBiryukov]

Introduced in [4275], modified in [4276].

Could you describe what exactly is broken and provide the steps to reproduce the issue?

[kitchin]

Note your first change can never branch, because 1 < $current there always.

The condition 1 == $i is correct for deciding whether to skip adding "?page=$i" to archive links. Maybe your bug is that thing about a blank last page of archives that happens sometimes, I forget why, maybe because of private posts??

[SergeyBiryukov]

Replying to kitchin:

Maybe your bug is that thing about a blank last page of archives that happens sometimes, I forget why, maybe because of private posts??

Every pagination issue I've seen so far was caused by using ​query_posts(). If you need to modify the main query, ​pre_get_posts should be used instead.

[obenland]

Feels like a duplicate of #24606.

[SergeyBiryukov]

Duplicate of #24606.

[nexurium]

Replying to kitchin:

Note your first change can never branch, because 1 < $current there always.

The condition 1 == $i is correct for deciding whether to skip adding "?page=$i" to archive links. Maybe your bug is that thing about a blank last page of archives that happens sometimes, I forget why, maybe because of private posts??

I'm ok,

My first change is not good, but the condition 2 == $current is not good too. if current is the second page the previous link must be 1 dans not ""