Automattic / Automattic/WP-Job-Manager
Move all actions and filters outside constructor functions
- Dominant language
- PHP
- Stars
- 899
- Forks
- 369
- Avg merge
- 11h 37m
- Merged PRs (30d)
- 12
Description
We have a lot of actions and filters inside constructor functions. Example:
https://github.com/Automattic/WP-Job-Manager/blob/master/includes/class-wp-job-manager-ajax.php#L14
This makes it hard to, say, remove an action and use your own instead. We should move them all out of `__construct`.
Contributor guide
Research direction
Start by reviewing includes/class-wp-job-manager-ajax.php, especially the constructor around the linked example, and identify the actions and filters registered there. Trace the other constructor functions that register hooks, then verify that hook registration remains available for replacement or removal without changing existing behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- php
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100