[wp-trac] [WordPress Trac] #36576: Split do_all_pings

WordPress Trac noreply at wordpress.org
Mon Oct 12 18:41:29 UTC 2020


#36576: Split do_all_pings
-------------------------------------------------+-------------------------
 Reporter:  dshanske                             |       Owner:  dshanske
     Type:  enhancement                          |      Status:  assigned
 Priority:  normal                               |   Milestone:  5.6
Component:  Pings/Trackbacks                     |     Version:  2.1
 Severity:  trivial                              |  Resolution:
 Keywords:  needs-dev-note has-patch needs-      |     Focuses:
  testing                                        |
-------------------------------------------------+-------------------------
Changes (by garrett-eclipse):

 * keywords:  needs-dev-note dev-feedback needs-patch => needs-dev-note has-
     patch needs-testing


Comment:

 I've put together an initial patch
 [https://core.trac.wordpress.org/attachment/ticket/36576/36576.diff
 36576.diff] in this direction, please test and review.

 In the patch I introduce a new action hook `perform_pings` used to hook in
 the other new functions. I did rename the new methods from `do_` as
 there's already a `do_trackbacks` so wanted to avoid a duplicate use of
 the function name. And I removed the first comments inside the new
 functions as their function docblock already handles the description and
 they're mostly duplicated there.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/36576#comment:19>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list