[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:
     Type:  enhancement                          |  boonebgorges
 Priority:  normal                               |      Status:  reviewing
Component:  Pings/Trackbacks                     |   Milestone:  Future
 Severity:  normal                               |  Release
 Keywords:  has-unit-tests 2nd-opinion needs-    |     Version:  2.1
  patch                                          |  Resolution:
                                                 |     Focuses:
                                                 |  performance
-------------------------------------------------+-------------------------

Comment (by dshanske):

 This is odd. I looked at the patch and I see that we added 'fields=ids'
 for only one. But I tested another version of this in plugin form as part
 of my experiments in pingback improvement and I just checked that code and
 I did add it in in all places. I don't know how I missed this.

 I will try to refresh the patch to address the _pingme and _encloseme
 issues. I would suggest we use this ticket to address those specific
 areas.

 Specifically with the to_ping notation, I think we need to reconsider a
 few things re trackbacks before we start mirroring to post meta. So, I
 would suggest that that, and the WP_Query issue be set aside for future
 and this address the lower hanging fruit.

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