bad variable name in get_previous_posts_page_link()

[dalirajab]

Hello !

While i was reading get_previous_posts_page_link() function code:

<?php
function get_previous_posts_page_link() {
        global $paged;

        if ( ! is_single() ) {
                $nextpage = (int) $paged - 1;
                if ( $nextpage < 1 ) {
                        $nextpage = 1;
                }
                return get_pagenum_link( $nextpage );
        }
}

my brain crashed for a while because of this line :

$nextpage = (int) $paged - 1;

I'm pretty sure that the right variable name is $previouspage (instead of $nextpage)

Hello !
While i was reading get_previous_posts_page_link() function code:

{{{php
<?php
function get_previous_posts_page_link() {

global $paged;

if ( ! is_single() ) {

$nextpage = (int) $paged - 1;
if ( $nextpage < 1 ) {

$nextpage = 1;

}
return get_pagenum_link( $nextpage );

}

}
}}}
my brain crashed for a while because of this line :
$nextpage = (int) $paged - 1;

I'm pretty sure that the right variable name is $previouspage (instead of $nextpage)

Trac ticket: https://core.trac.wordpress.org/ticket/57746#ticket

[SergeyBiryukov]

Hi there, welcome to WordPress Trac! Thanks for the ticket.

get_previous_posts_page_link() was introduced in [5045], but the variable appears to be even older than that, it was brought over along with previous_posts()from b2/cafelog in [8].

Indeed, the suggested name makes more sense.

[SergeyBiryukov]

In 55364:

Coding Standards: Improve variables names in post and comment page link functions.

This commit renames a few internal variables for better clarity and consistency:

• $nextpage to $next_page in:
  • get_next_posts_page_link()
  • get_next_posts_link()
  • get_next_comments_link()
• $nextpage to $previous_page in:
  • get_previous_posts_page_link()
• $prevpage to $previous_page in:
  • get_previous_comments_link()

Includes minor code layout fixes for better readability.

Follow-up to [5045], [8502], [8961], [28111].

Props dalirajab, SergeyBiryukov.
Fixes #57746.

Thanks for the PR! Merged in r55364 as part of a few other similar edits.