Skip to content

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

Merged
retlehs merged 1 commit into
roots:mainfrom
frugan-dev:main
Feb 27, 2026
Merged

🔧 Fix filter to action for admin head in setup#3276
retlehs merged 1 commit into
roots:mainfrom
frugan-dev:main

Conversation

@frugan-dev

Copy link
Copy Markdown
Contributor

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

dsturm commented Nov 17, 2025

Copy link
Copy Markdown
Contributor

@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
Copy Markdown
Contributor Author

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

@QWp6t

QWp6t commented Nov 28, 2025

Copy link
Copy Markdown
Member

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.

@retlehs retlehs merged commit ddcae01 into roots:main Feb 27, 2026
3 checks passed
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.

4 participants