[wp-trac] [WordPress Trac] #56504: `sanitize_html_class()` is both too restrictive, and too permissive so it may return an invalid class name

WordPress Trac noreply at wordpress.org
Mon Sep 5 11:25:41 UTC 2022


#56504: `sanitize_html_class()` is both too restrictive, and too permissive so it
may return an invalid class name
----------------------------------+------------------------------
 Reporter:  anrghg                |       Owner:  (none)
     Type:  defect (bug)          |      Status:  closed
 Priority:  normal                |   Milestone:  Awaiting Review
Component:  General               |     Version:
 Severity:  normal                |  Resolution:  duplicate
 Keywords:  close needs-dev-note  |     Focuses:
----------------------------------+------------------------------
Changes (by anrghg):

 * keywords:  close changes-requested needs-testing needs-dev-note needs-
     I18N-review => close needs-dev-note
 * status:  new => closed
 * resolution:   => duplicate


Comment:

 Replying to [comment:5 peterwilsoncc]:
 > > all scripts should be equal
 >
 > I do agree that the function ought to be more permissive for valid
 characters, there's an existing ticket for that #33924

 Thank you! I wasn’t aware.

 > CSS is more permissive than it once was;

 Well done.

 > Are you happy to continue this discussion on #33924 and close this
 ticket as a duplicate?

 Indeed as a duplicate it needs to be closed. I’ll replicate the
 potentially useful stuff on #33924.

 > Raising the issue of non-latin alphabets is an excellent point. If you
 post it to the original ticket, it will ensure you get props for
 contributing to the discussion.

 Thank you very much. I’m happy that the issue of Non-Latin is worked on,
 and I’ll strive to “weigh in” :)

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/56504#comment:6>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list