Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[POC] resolving hooks in workpress code #3067

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jaapio
Copy link
Member

@jaapio jaapio commented Nov 13, 2021

This is a POC using the new metadata concept in phpDocumentor to add
extra information in your docs. It is far from usefull as many changes are
needed in the core code to support this case.
This code can be used as reference code to see what we need to change in phpDocumentor
and it reveals missing extension points in our application.

This is a POC using the new metadata concept in phpDocumentor to add
extra information in your docs. It is far from usefull as many changes are
needed in the core code to support this case.
This code can be used as reference code to see what we need to change in phpDocumentor
and it reveals missing extension points in our application.

final class Action extends AbstractFactory
{
private const ACTION_NAMES = [
Copy link

@Chrico Chrico Nov 15, 2021

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually actions are filters which do nothing return in WordPress. So it's maybe better to call this either Filter or introduce a second class which is called Filter. In WordPress there are following functions which are using internally the Hook API of WP_Hook:

'apply_filters',
'apply_filters_ref_array',
'apply_filters_deprecated',
'do_action',
'do_action_ref_array',
'do_action_deprecated',

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants