WordPress.org

Make WordPress Core

Opened 8 months ago

Last modified 2 weeks ago

#49905 accepted defect (bug)

Correct return type for xmlrpc_getposttitle() and xmlrpc_getpostcategory()

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

Description

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 8 months ago.

Download all attachments as: .zip

Change History (10)

#2 @Presskopp
6 months ago

  • Keywords has-patch added

#3 @SergeyBiryukov
5 months ago

  • Milestone changed from 5.5 to 5.6

This ticket was mentioned in Slack in #core by hellofromtonya. View the logs.


5 weeks ago

#5 @SergeyBiryukov
5 weeks ago

  • Owner set to SergeyBiryukov
  • Status changed from new to accepted

#6 @whyisjake
4 weeks ago

In 49376:

XML-RPC: Return error message if attachment ID is incorrect.

Throw an error for incorrect attachment IDs when requesting a media object to ensure return signature is correct and doesn't include incorrectly typed values within the object.

Props zieladam, peterwilsoncc, whyisjake.
See #49905.

#7 @desrosj
4 weeks ago

In 49385:

XML-RPC: Return error message if attachment ID is incorrect.

Throw an error for incorrect attachment IDs when requesting a media object to ensure return signature is correct and doesn't include incorrectly typed values within the object.

Props zieladam, peterwilsoncc, whyisjake.
Merges [49376] to trunk.
See #49905.

This ticket was mentioned in Slack in #core by helen. View the logs.


2 weeks ago

#9 @SergeyBiryukov
2 weeks ago

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