[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
 /class-IXR-message.php?marks=103-110#L102 xmlrpc_chunk_parsing_size]

 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,
 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