#31359 closed defect (bug) (fixed)
wp_insert_post documentation missing post_title
Reported by: | atimmer | Owned by: | DrewAPicture |
---|---|---|---|
Milestone: | 4.2 | Priority: | normal |
Severity: | trivial | Version: | 4.2 |
Component: | Posts, Post Types | Keywords: | has-patch commit |
Focuses: | docs | Cc: |
Description
The documentation for the $postarr for wp_insert_post is missing post_title as an argument.
Attachments (4)
Change History (18)
#2
@
10 years ago
- Component changed from General to Posts, Post Types
- Milestone changed from Awaiting Review to 4.2
#4
@
10 years ago
We should probably synchronize that block with the list of fields in the wp_posts
table:
ID post_author post_date post_date_gmt post_content post_title post_excerpt post_status comment_status ping_status post_password post_name to_ping pinged post_modified post_modified_gmt post_content_filtered post_parent guid menu_order post_type post_mime_type comment_count
#5
@
10 years ago
- Keywords needs-patch added; has-patch removed
Great idea, I will look further into this with more time.
#6
follow-up:
↓ 7
@
10 years ago
- Keywords has-patch added; needs-patch removed
31359.2.diff adds all missing possibilities. I would love feedback on the textual content, I tried to mimic the pre-existing documentation.
I wonder if we should order all the variables the same way they are ordered in the database. Thoughts?
#7
in reply to:
↑ 6
@
10 years ago
Replying to atimmer:
I wonder if we should order all the variables the same way they are ordered in the database. Thoughts?
I would expect that, yes.
#8
@
10 years ago
31359.3.diff is the same as 31359.2.diff except that the documentation is ordered the same way as the database.
#9
@
10 years ago
$comment_status
and $ping_status
should be changed to string
, they can be 'open'
or 'closed'
. Apparently this was an oversight in [25628].
Looks good to me otherwise.
#10
@
10 years ago
Done, see 31359.4.diff
31359.diff adds post_title to the documentation.