Ticket #5741 (reviewing enhancement)

Opened 4 years ago

Last modified 15 months ago

No Edit Comment Form Hook

Reported by: DavidMeade Owned by: westi
Priority: normal Milestone: Future Release
Component: Administration Version:
Severity: normal Keywords: close
Cc: bbeoj

Description

Hooks are provided to allow plugins to add custom fields to comment forms, however there is no hook to allow plugins to give admins to ability to adjust any such fields in the comment edit form.

To fix this, I added the following at line 51 of wp-admin/edit-form-comment.php (just above the advanced area of the form)

<?php do_action('edit_comment_form', $comment->comment_ID); ?>

Attachments

edit-form-comment.php Download (4.1 KB) - added by DavidMeade 4 years ago.
patched file
5741.r6772.diff Download (360 bytes) - added by hansengel 4 years ago.
patch-ified #5741
5741.diff Download (623 bytes) - added by scohoust 3 years ago.

Change History

  • Milestone changed from 2.6 to 2.3.3
  • Milestone changed from 2.3.3 to 2.6

DavidMeade, please leave updating the milestone to a core contributor. Please also attach a patch.

patched file

ok I uploaded the changed version of wp-admin/edit-form-comment.php

Sorry about the milestone .. it was a field I was presented with to select a value for so I thought I was supposed to select a value :-P

patch-ified #5741

  • Keywords close added
  • Type changed from defect (bug) to enhancement
  • Component changed from Administration to Themes
  • Milestone changed from 2.9 to Future Release
  • Component changed from Themes to Administration

This isn't to do with themes, this is for plugins to modify the admin comment editing form.

Patch refresh, not sure how many plugins would benefit from this.

  • Owner changed from anonymous to westi
  • Status changed from new to reviewing

This looks like a reasonable idea.

Could you possible knock together a proof-of-concept plugin.

i.e. one that adds an extra field to the comment form, processes it and displays it here

This is to make sure we are not missing any other necessary hooks

Thanks :-)

  • Cc bbeoj added

I think this is made obsolete by #13017 ??

Note: See TracTickets for help on using tickets.