Add limit parameter to get_pages

[DD32]

add 'numberposts' and 'offset' to get_pages() to allow for limiting of returned posts.

[DD32]

Adds numberpages and offset to get_pages

[DD32]

'Doh!
Sorry, I meant "adds 'numberpages'" in the Description :)

[rob1n]

Why is it 'numberposts'? Why not 'numberpages'?

[foolswisdom]

Replying to rob1n:

Why is it 'numberposts'? Why not 'numberpages'?

DD32 commented on wp-hackers about that typo that is in the ticket, but not in the patch.

[ryan]

I think we use "number" in other places. Let's keep it consistent.

The number needs to be cast to an int for security. See how number is used in get_terms().

[Speedboxer]

Add offset and number parameters to get_pages()

[Speedboxer]

Patch 4531-add-limit-parameters-to-get_pages.diff adds offset and number parameters to get_pages().

[DD32]

Replying to foolswisdom:

Replying to rob1n:

Why is it 'numberposts'? Why not 'numberpages'?

DD32 commented on wp-hackers about that typo that is in the ticket, but not in the patch.

See comment 1:

DD32:

'Doh! Sorry, I meant "adds 'numberpages'" in the Description :)

If we want to standarise on 'number' and 'offset' for all functions, then it might be an idea to depreciate "numberposts" in get_posts too:

function get_posts($args) {
	global $wpdb;

	$defaults = array(
		'numberposts' => 5, 'offset' => 0, 

[westi]

marking as an enhancement

moving to 2.6 for resolving.

Fix with #4711

[Denis-de-Bernardy]

still current?

[Speedboxer]

Updated patch

[Speedboxer]

Replying to Denis-de-Bernardy:

still current?

It probably is, updated my patch.

[Denis-de-Bernardy]

patch is harmless

[Denis-de-Bernardy]

I *am* wondering how this is useful, though. :D

[westi]

In [11251] - Add paging support to get_pages(). Fixes #4531 props DD32.