✨ New WordPress.WP.OptionAutoload sniff#2520

Open

rodrigoprimo wants to merge 26 commits intoWordPress:developfrom

rodrigoprimo:new-option-autoload

Commits on Jul 4, 2025

✨ New WordPress.WP.OptionAutoload sniff
rodrigoprimo
committed
Apply suggestions from code review

rodrigoprimo
and
jrfnl
committed
Attemp to improve the XML documentation by combining standards
rodrigoprimo
committed
Fix unintentional coding standard violation in a test case
rodrigoprimo
committed
Update some tests to have more function name case variations
rodrigoprimo
committed
Update some tests to have more whitespaces, comments and annotations
rodrigoprimo
committed
Use `false === isset()` instead of `! isset()` for consistency with the rest of the sniff code
rodrigoprimo
committed
Improve sniff performance by changing the format of some properties
rodrigoprimo
committed
Apply suggestions from code review

rodrigoprimo
and
jrfnl
committed
Remove unused parameter
rodrigoprimo
committed
Rename test case file to be able to add more test case files with syntax errors
rodrigoprimo
committed
Handle case when $array_token is false + add a test for it
rodrigoprimo
committed
Improve sniff XML doc following suggestions from code review
rodrigoprimo
committed
Add link to external blog post about when to use the autoload flag
rodrigoprimo
committed
Add backticks to all function calls in the docblocks
rodrigoprimo
committed
Rename property `valid_values_other_functions` to `valid_values_wp_set_option_functions`
rodrigoprimo
committed
Explain in the docblock the array properties where all values are `true`
rodrigoprimo
committed
Ensure all `@param array` tags explain the structure of the array
rodrigoprimo
committed
Only record `param missing` metric when the autoload parameter is optional
rodrigoprimo
committed

Commits on Jul 15, 2025

Commits on Jul 16, 2025

Move the third block of the documentation up as suggestion during code review
rodrigoprimo
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

✨ New WordPress.WP.OptionAutoload sniff#2520

✨ New WordPress.WP.OptionAutoload sniff#2520
rodrigoprimo wants to merge 26 commits intoWordPress:developfrom
rodrigoprimo:new-option-autoload

Commits on Jul 4, 2025

✨ New WordPress.WP.OptionAutoload sniff

Apply suggestions from code review

Attemp to improve the XML documentation by combining standards

Fix unintentional coding standard violation in a test case

Update some tests to have more function name case variations

Update some tests to have more whitespaces, comments and annotations

Use `false === isset()` instead of `! isset()` for consistency with the rest of the sniff code

Improve sniff performance by changing the format of some properties

Apply suggestions from code review

Remove unused parameter

Rename test case file to be able to add more test case files with syntax errors

Handle case when $array_token is false + add a test for it

Improve sniff XML doc following suggestions from code review

Add link to external blog post about when to use the autoload flag

Add backticks to all function calls in the docblocks

Rename property `valid_values_other_functions` to `valid_values_wp_set_option_functions`

Explain in the docblock the array properties where all values are `true`

Ensure all `@param array` tags explain the structure of the array

Only record `param missing` metric when the autoload parameter is optional

Commits on Jul 15, 2025

Use array union instead of `array_merge()` to create the list of known discouraged values

Use one error code for fixable internal values and another for non-fixable internal values

Update if condition to explicitly check what is expected and add a test to reach that condition

Add another test without whitespaces surrounding the parameter to test the condition on OptionAutoloadSniff.php:247 correctly

Add a comment explaining why `null` needs special treatment on line 361

Explicitly check if $param_second_token is an integer

Commits on Jul 16, 2025

Move the third block of the documentation up as suggestion during code review

Uh oh!

✨ New WordPress.WP.OptionAutoload sniff#2520rodrigoprimo wants to merge 26 commits intoWordPress:developWordPress/WordPress-Coding-Standards:developfrom rodrigoprimo:new-option-autoloadrodrigoprimo/WordPress-Coding-Standards:new-option-autoloadCopy head branch name to clipboard

Commits

Commits on Jul 4, 2025

Commits on Jul 15, 2025

Commits on Jul 16, 2025

✨ New WordPress.WP.OptionAutoload sniff#2520
rodrigoprimo wants to merge 26 commits intoWordPress:developfrom
rodrigoprimo:new-option-autoload