Skip to content

🔧 Fix filter to action for admin head in setup#3276

Open
frugan-dev wants to merge 1 commit intoroots:mainfrom
frugan-dev:main
Open

🔧 Fix filter to action for admin head in setup#3276
frugan-dev wants to merge 1 commit intoroots:mainfrom
frugan-dev:main

Conversation

@frugan-dev
Copy link

I understand that admin_head only exists as an action, not as a filter, or am I wrong?

@github-actions github-actions bot added the php Pull requests that update PHP code label Nov 15, 2025
@dsturm
Copy link
Contributor

dsturm commented Nov 17, 2025

@frugan-dev Since add_action is merely a wrapper for add_filter, this is not a critical issue but rather a minor improvement.

@frugan-dev
Copy link
Author

@dsturm Ok thy, I didn't know about this wrapper..

@QWp6t
Copy link
Member

QWp6t commented Nov 28, 2025

I agree with this change since it clarifies intent as add_action() and add_filter() are semantically distinct. 🙂

I also agree this isn't a bugfix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

php Pull requests that update PHP code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants