vhad/wordpress-develop
1
0
Fork 0
mirror of https://github.com/gosticks/wordpress-develop.git synced 2026-08-11 20:30:23 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

In paginate_links(), don't override custom format arguments when setting up default 'add_args'.

Browse Source 
Since 4.1 [29780], the default value of the 'add_args' argument in
`paginate_links()` has been determined by parsing the current URL. This change
had the side effect of overriding custom values of 'format' that changed the
pagination query var, with the result that plugins using `paginate_links()`
with a custom format generated the incorrect links unless explicitly
declaring 'add_args=false' to prevent the default values from overriding. We
fix this behavior by parsing URL query vars into the 'add_args' array only
after the explicit function params have been parsed, and by skipping the
current page's pagination query var when doing this parsing (to avoid the
override).

Props obenland.
Fixes #30831 for trunk.

git-svn-id: https://develop.svn.wordpress.org/trunk@31203 602fd350-edb4-49c9-b593-d223f7449a82
This commit is contained in:
Boone Gorges
2015-01-16 15:48:24 +00:00
parent 7c707c6633
commit af146fef8c
2 changed files with 81 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/phpunit/tests/general/paginateLinks.php
+62
View File
@@ -229,4 +229,66 @@ EXPECTED;
$this->assertEquals( $expected_href, $href );
}
}
/**
* @ticket 30831
*/
function test_paginate_links_with_custom_query_args() {
add_filter( 'get_pagenum_link', array( $this, 'add_query_arg' ) );
$links = paginate_links( array(
'current' => 2,
'total' => 5,
'end_size' => 1,
'mid_size' => 1,
'type' => 'array',
'add_args' => array(
'baz' => 'qux',
),
) );
remove_filter( 'get_pagenum_link', array( $this, 'add_query_arg' ) );
$document = new DOMDocument();
$document->preserveWhiteSpace = false;
$data = array(
0 => home_url( '/?baz=qux&foo=bar&s=search+term' ),
1 => home_url( '/?baz=qux&foo=bar&s=search+term' ),
3 => home_url( '/?paged=3&baz=qux&foo=bar&s=search+term' ),
5 => home_url( '/?paged=5&baz=qux&foo=bar&s=search+term' ),
6 => home_url( '/?paged=3&baz=qux&foo=bar&s=search+term' ),
);
foreach ( $data as $index => $expected_href ) {
$document->loadHTML( $links[ $index ] );
$tag = $document->getElementsByTagName( 'a' )->item( 0 );
$this->assertNotNull( $tag );
$href = $tag->attributes->getNamedItem( 'href' )->value;
$this->assertEquals( $expected_href, $href );
}
}
/**
* @ticket 30831
*/
public function test_paginate_links_should_allow_non_default_format_without_add_args() {
// Fake the query params.
$request_uri = $_SERVER['REQUEST_URI'];
$_SERVER['REQUEST_URI'] = add_query_arg( 'foo', 3, home_url() );
$links = paginate_links( array(
'base' => add_query_arg( 'foo', '%#%' ),
'format' => '',
'total' => 5,
'current' => 3,
'type' => 'array',
) );
$this->assertContains( '?foo=1', $links[1] );
$this->assertContains( '?foo=2', $links[2] );
$this->assertContains( '?foo=4', $links[4] );
$this->assertContains( '?foo=5', $links[5] );
$_SERVER['REQUEST_URI'] = $request_uri;
}
}