WordPress.org
Get WordPress

Make WordPress Core

Opened 10 years ago

Closed 10 years ago

#36427 closed enhancement (fixed)

Edit_Comment Should Pass CommentData

Reported by: dshanske Owned by: dshanske
Priority: normal Milestone: 4.6
Component: Comments Version:
Severity: normal Keywords: needs-patch needs-refresh
Cc: Focuses:

Description

Action 'edit_comment', which fires on comment update, should pass $commentdata along to it.

For one, 'comment_post' now does this. This would allow an action to perform additional work...such as in the comment meta, passed by whatever triggered the update function.

Attachments (3)

36427.diff​ (682 bytes ) - added by dshanske 10 years ago.
36427.2.diff​ (705 bytes ) - added by dshanske 10 years ago.
Fix Document
36427.3.diff​ (762 bytes ) - added by rachelbaker 10 years ago.
Minor docs formatting fixes

Download all attachments as: .zip

Change History (9)

#1 @rachelbaker
10 years ago

  • Keywords needs-patch added
  • Milestone Awaiting Review → Future Release

@dshanske sounds reasonable. Want to create a patch for 4.6?

@dshanske
10 years ago

This ticket was mentioned in ​Slack in #core-comments by dshanske. ​View the logs.
10 years ago

This ticket was mentioned in ​Slack in #core-comments by dshanske. ​View the logs.
10 years ago

#4 @rachelbaker
10 years ago

  • Keywords needs-refresh added
  • Milestone Future Release → 4.6
  • Owner set to dshanske
  • Status new → assigned

36427.diff​ makes sense, but the inline docs need a refresh to match the standards.

Assigned to @dshanske to refresh.

Version 0, edited 10 years ago by rachelbaker (next)

@dshanske
10 years ago

Fix Document

#5 @dshanske
10 years ago

@rachelbaker Think that should fix it.

@rachelbaker
10 years ago

Minor docs formatting fixes

#6 @rachelbaker
10 years ago

  • Resolution → fixed
  • Status assigned → closed

In 37423:

Comments: Add $data parameter to include the comment data in the edit_comment action.

Props dshanske.

Fixes #36427.

Note: See TracTickets for help on using tickets.