Make WordPress Core

Opened 4 months ago

Last modified 5 weeks ago

#49905 new defect (bug)

Correct return type for xmlrpc_getposttitle() and xmlrpc_getpostcategory()

Reported by: SergeyBiryukov Owned by:
Milestone: 5.6 Priority: normal
Severity: normal Version:
Component: XML-RPC Keywords: has-patch
Focuses: Cc:


There are a few issues in xmlrpc_getposttitle() and xmlrpc_getpostcategory():

  • xmlrpc_getposttitle() uses the $post_default_title global as a fallback title, however the global is only set in xmlrpc.php and not in other contexts where the function might be called, e.g. wp-mail.php. In those cases, the function returns null instead of a string, causing issues like #49853.
  • xmlrpc_getpostcategory() uses the $post_default_category global, however it was removed in [2680]. Because of that, the function can also return null.
  • xmlrpc_getpostcategory() documentation suggests that it returns either a string or an array of strings, however that is incorrect. It's supposed to be used for the post_category parameter of wp_insert_post(), which receives an array of numeric category IDs.

Let's bring some consistency and unit tests here.

Attachments (1)

49905.diff (3.8 KB) - added by SergeyBiryukov 4 months ago.

Download all attachments as: .zip

Change History (4)

#2 @Presskopp
2 months ago

  • Keywords has-patch added

#3 @SergeyBiryukov
5 weeks ago

  • Milestone changed from 5.5 to 5.6
Note: See TracTickets for help on using tickets.