Context Navigation

#58129 closed defect (bug) (fixed)

There is currently a WPCS warning in wp-includes/formatting.php:

Variable "$myHTML" is not in valid snake_case format, try "$my_h_t_m_l"

Renaming the $myHTML parameter to $text in htmlentities2() should fix the warning.

It would also match other formatting functions with the $text parameter, e.g. esc_html(), ent2ncr(), etc.

Rename parameter $myHTML to $text to follow the standard snake_case. Now matches other formating functions like esc_js, esc_html, etc.

Coding Standards: Rename the $myHTML parameter to $text in htmlentities2().

This resolves a WPCS warning:

Variable "$myHTML" is not in valid snake_case format, try "$my_h_t_m_l"

It also matches other formatting functions with the $text parameter, e.g. esc_html(), ent2ncr(), etc.

Props victoranto, audrasjb, jrf.
Fixes #58129.

Thanks for the PR! Merged in r55651.

Note: See TracTickets for help on using tickets.