[wp-trac] [WordPress Trac] #36765: Remove Legacy Code from pingback_ping

WordPress Trac noreply at wordpress.org
Mon Jul 18 19:38:16 UTC 2016


#36765: Remove Legacy Code from pingback_ping
-------------------------------------------------+-------------------------
 Reporter:  dshanske                             |       Owner:
     Type:  defect (bug)                         |      Status:  new
 Priority:  normal                               |   Milestone:  Future
Component:  Pings/Trackbacks                     |  Release
 Severity:  normal                               |     Version:  4.1
 Keywords:  has-patch dev-feedback needs-unit-   |  Resolution:
  tests                                          |     Focuses:
-------------------------------------------------+-------------------------
Changes (by rachelbaker):

 * keywords:  has-patch dev-feedback => has-patch dev-feedback needs-unit-
     tests


Comment:

 @dshanske this is looking good so far.  Are there any conditions covered
 by the duplicate logic you are removing in this ticket that are not
 covered by `url_to_post_id()`?  Would be good to check the tests in
 [source:/trunk/tests/phpunit/tests/rewrite.php], and add any conditions
 that are missing.

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


More information about the wp-trac mailing list