Make WordPress Core

Opened 11 years ago

Closed 11 years ago

#31747 closed defect (bug) (fixed)

wp_insert_comment better docstrings

Reported by: lamosty Owned by: DrewAPicture
Priority: normal Milestone: 4.3
Component: Comments Version:
Severity: normal Keywords: has-patch
Cc: Focuses: docs

Description

Function wp_insert_comment has incomplete docstring documentation. Its docstring says

The available comment data key names are 'comment_author_IP', 'comment_date',
'comment_date_gmt', 'comment_parent', 'comment_approved', and 'user_id'.

but the function accepts additional arguments through $commentdata assoc array, such as comment_type or comment_content.

Docstring could be enhanced to have a format like in the case of wp_insert_post function (=> argument type with a short description of what the argument does).

Attachments (2)

docs_update.31747.diff (2.5 KB ) - added by lamosty 11 years ago.
Update docs of wp_insert_comment function
31747.patch (2.8 KB ) - added by rachelbaker 11 years ago.
Minor formatting and description adjustments

Download all attachments as: .zip

Change History (9)

#1 @SergeyBiryukov
11 years ago

  • Component Text ChangesComments

#2 @SergeyBiryukov
11 years ago

  • Keywords needs-patch added

@lamosty
11 years ago

Update docs of wp_insert_comment function

#3 @iseulde
11 years ago

  • Keywords has-patch added; needs-patch removed
  • Version trunk

#4 @DrewAPicture
11 years ago

  • Owner set to DrewAPicture
  • Status newreviewing

@rachelbaker
11 years ago

Minor formatting and description adjustments

#5 @DrewAPicture
11 years ago

  • Milestone Awaiting Review4.3
  • Type enhancementdefect (bug)

#6 @DrewAPicture
11 years ago

In 32499:

Document the default arguments for wp_insert_comment() as a hash notation.

Props lamosty, rachelbaker.
See #31747.

#7 @DrewAPicture
11 years ago

  • Resolutionfixed
  • Status reviewingclosed

In 32500:

Update the DocBlock for wp_new_comment() to reference the wp_insert_comment() documentation for argument information.

Fixes #31747.

Note: See TracTickets for help on using tickets.