has_filter() can return 0, and remove_filter() doesn't use $accepted_args
|Reported by:||nacin||Owned by:||nacin|
|Severity:||normal||Keywords:||has-patch commit early|
has_filter() returns priority, which means it can return 0. But core doesn't check for it being identical to false. We should audit all has_filter() calls and make sure 'false !==' or 'false ===' depending on what we want.
Additionally remove_filter() has an $accepted_args argument, but doesn't use it. We should eliminate it from the function definition and docs.
Change History (9)
- Keywords early added
- Milestone changed from Awaiting Review to Future Release