[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:  4.9.7
Component:  Pings/Trackbacks  |     Version:  2.1
 Severity:  normal            |  Resolution:
 Keywords:  needs-patch       |     Focuses:  performance
------------------------------+---------------------------

Comment (by birgire):

 The patch in [attachment:36824.10.diff]:

 - Adds unit tests for {{{do_enclose()}}} to help with the ticket. First
 pass.
 - Adds {{{@return}}} for {{{do_enclode()}}}.
 - Adjusts the {{{! empty( get_to_ping( $post_id ) )}}} from
 [attachment:36824.9.diff] to avoid {{{Fatal error: Can't use function
 return value in write context}}} for PHP < 5.5.
 - Removes global {{{$wpdb}}} in {{{do_all_pings()}}}.
 - Removes doc block for {{{$wpdb}}} in {{{do_all_pings()}}}.
 - Added end-of-line dots for inline comments in {{{do_all_pings()}}}.

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