[wp-trac] [WordPress Trac] #58129: Rename the $myHTML parameter in htmlentities2()

WordPress Trac noreply at wordpress.org
Thu Apr 13 16:37:54 UTC 2023


#58129: Rename the $myHTML parameter in htmlentities2()
------------------------------+----------------------------------------
 Reporter:  SergeyBiryukov    |      Owner:  (none)
     Type:  defect (bug)      |     Status:  new
 Priority:  normal            |  Milestone:  6.3
Component:  Formatting        |    Version:
 Severity:  normal            |   Keywords:  needs-patch good-first-bug
  Focuses:  coding-standards  |
------------------------------+----------------------------------------
 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.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/58129>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list