Context Navigation

← Previous Ticket
Next Ticket →

#20723 closed defect (bug) (fixed)

wp.getPosts xmlrpc using "number" when it should be using "numberposts"

Reported by:	djcp	Owned by:	ryan
Milestone:	3.4	Priority:	normal
Severity:	normal	Version:	3.4
Component:	XML-RPC	Keywords:	api has-patch commit
Focuses:		Cc:

Description

See $SUBJECT. Using "number" to limit the number of posts returned is incorrect, as wp_get_recent_posts() expects this parameter to be called "numberposts".

A patch is attached to fix the parameter name to be "numberposts" - though we could have the API externally expose "number" yet give the parameter "numberposts" to wp_get_recent_posts().

Attachments (2)

20723.patch (1020 bytes) - added by SergeyBiryukov 13 years ago.: Also corrects PHPDocs
class-wp-xmlrpc-server.php.patch (570 bytes) - added by djcp 13 years ago.

Download all attachments as: .zip

Change History (10)

#1 @djcp
13 years ago

Keywords has-patch added

#2 @djcp
13 years ago

To clarify, this means setting the number of posts that wp.getPosts returns is busted.

@SergeyBiryukov
13 years ago

Attachment 20723.patch added

Also corrects PHPDocs

#3 @SergeyBiryukov
13 years ago

Milestone changed from Awaiting Review to 3.4

Related: #18433

The change happened in ticket:18433:wp.getPosts.6.patch , the earlier patches used numberposts.

#4 @ryan
13 years ago

numberposts is ancient cruft. I'd rather not bubble it up. How about exposing number in wp.getPosts and passing it along as numberposts per the alternative suggested in the description.