[wp-meta] [Making WordPress.org] #6381: Filter docblock not parsed / doesn't exist on DevHub
Making WordPress.org
noreply at wordpress.org
Thu Jun 30 00:22:57 UTC 2022
#6381: Filter docblock not parsed / doesn't exist on DevHub
---------------------------+---------------------
Reporter: audrasjb | Owner: (none)
Type: defect (bug) | Status: new
Priority: normal | Milestone:
Component: Developer Hub | Resolution:
Keywords: 2nd-opinion |
---------------------------+---------------------
Comment (by SergeyBiryukov):
Replying to [ticket:6381 audrasjb]:
> It looks like the `xmlrpc_element_limit` filter doesn't exist on DevHub.
> I can't figure out what's wrong with the docblock declaration… or maybe
it's just because of the conditional?
Thanks for catching this! The same applies to the
[https://core.trac.wordpress.org/browser/tags/6.0/src/wp-includes/IXR
/class-IXR-message.php?marks=103-110#L102 xmlrpc_chunk_parsing_size]
filter.
These two are the only filters in the `wp-includes/IXR` directory. I think
the conditional is not relevant here, but the directory is simply excluded
from parsing. Looking at the source,
[source:sites/trunk/wordpress.org/public_html/wp-content/themes/pub/wporg-
developer/inc/parser.php?rev=9184&marks=21#L11 that is indeed the case].
> If this filter can't be parsed because of the conditional, should we
simply address the issue by removing the `@see` mention from the
`xmlrpc_enabled` function DocBlock?
I think the solution would be to remove the exclusion, so that the
directory is parsed too. Apparently it was considered an external library
without any filters, but that is not the case, it's an
[https://core.trac.wordpress.org/ticket/48267#comment:2 adopted library]
and does include some hooks.
--
Ticket URL: <https://meta.trac.wordpress.org/ticket/6381#comment:1>
Making WordPress.org <https://meta.trac.wordpress.org/>
Making WordPress.org
More information about the wp-meta
mailing list