Automattic / Automattic/WP-Job-Manager

Move all actions and filters outside constructor functions

Open
#797 5 comments 1 reaction 0 assignees View on GitHub
[Pri] Low [Status] Accepted [Type] Maintenance
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

Open the contributing 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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.