[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