[wp-trac] [WordPress Trac] #36824: do_all_pings function queries all posts

[WordPress Trac]

#36824: do_all_pings function queries all posts
------------------------------+-----------------------------
 Reporter:  spacedmonkey      |       Owner:  boonebgorges
     Type:  enhancement       |      Status:  reviewing
 Priority:  normal            |   Milestone:  Future Release
Component:  Pings/Trackbacks  |     Version:  2.1
 Severity:  normal            |  Resolution:
 Keywords:  has-patch         |     Focuses:  performance
------------------------------+-----------------------------

Comment (by boonebgorges):

 Thanks for the new patch, @dshanske. A question about the
 `publicly_queryable` clause. Is there a situation in which a post that
 belongs to a post type which has `publicly_queryable=false` would need to
 send a pingback? Your patch would break this kind of use. But I'm not sure
 whether it's a valid or realistic use case. (Keep in mind that this is an
 old part of WP, and people do weird things.)

 > All pingbacks add the URLs they send to the pinged field. So, there
 should be a way to get some resource savings by only looking at those
 posts for trackback sending. But that requires some more substantial
 changes.

 Can you explain this more? `do_trackbacks()` seems to skip sending
 trackbacks to URLs that have already gotten pingbacks. Is this correct?

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