Ticket #4844 (closed enhancement: fixed)

Opened 4 years ago

Last modified 2 years ago

Need set_post_type API

Reported by: filosofo Owned by: anonymous
Priority: normal Milestone: 2.5
Component: General Version: 2.3
Severity: normal Keywords: set_post_type post_type has-patch
Cc: filosofo

Description

Currently there is a get_post_type function, which with other functions allows plugins to use custom "post_type"s. There should be API to set the post_type without having to access the db directly.

Attachments

set_post_type.diff Download (579 bytes) - added by filosofo 4 years ago.
set_post_type.2.diff Download (656 bytes) - added by filosofo 4 years ago.

Change History

The second patch sanitizes post_type per rboren's suggestion at comment:ticket:4991:1

comment:2 follow-up: ↓ 3   ryan4 years ago

I think this needs to use wp_update_post() so that all of the right actions are triggered.

comment:3 in reply to: ↑ 2   filosofo4 years ago

Replying to ryan:

I think this needs to use wp_update_post() so that all of the right actions are triggered.

One advantage to a stand-alone function instead of just using wp_update_post() is that you don't have to know all of the post data first.

Also, other aspects of a post can be changed without going through wp_update_post, such as categories, tags, ping URLs, and post meta-data.

And you keep the API parallel: where there's a get_* there should be a set_* or update_*

comment:4   ryan4 years ago

  • Status changed from new to closed
  • Resolution set to fixed

(In [6537]) set_post_type from filosofo. fixes #4844

comment:5   ryan4 years ago

I added db prepare and some phpdoc, hopefully didn't break anything.

Note: See TracTickets for help on using tickets.