[wp-trac] [WordPress Trac] #36824: do_all_pings function queries all posts
WordPress Trac
noreply at wordpress.org
Tue Sep 17 19:31:29 UTC 2019
#36824: do_all_pings function queries all posts
--------------------------------------+---------------------------
Reporter: spacedmonkey | Owner: boonebgorges
Type: enhancement | Status: reviewing
Priority: normal | Milestone: 5.3
Component: Pings/Trackbacks | Version: 2.1
Severity: normal | Resolution:
Keywords: has-patch has-unit-tests | Focuses: performance
--------------------------------------+---------------------------
Comment (by boonebgorges):
[attachment:"36824-pings.diff"] includes only the change to the `pingback`
query. It corrects the `meta_key` / `meta_value` error in the most recent
patches. Looking at the SQL generated, there are two potentially breaking
changes. I'd like feedback from others with an understanding of this issue
before moving forward with it.
1. Currently, the _pingme query doesn't look at post type at all. Now
we're limiting it to those post types with `post_type_supports(
'trackbacks' )`. ([comment:38 Related concern].) This seems like it will
break expected behavior. What problem is it meant to solve? In part, it
addresses the problems with `'post_type' => 'any'` ([comment:10
reference]). But I wonder if we might instead pass a list of all
registered post types, for better backward compatibility. This issue could
then possibly be revisited after #39960.
2. Currently, the pingback query is post_status-agnostic. Now, it's
limited to `publish`. Could this have unintended consequences? In theory,
only published posts should have `_pingme`
https://core.trac.wordpress.org/browser/tags/5.2.3/src/wp-
includes/post.php#L6551 but I'd like others to weigh in.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/36824#comment:52>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list