[wp-trac] [WordPress Trac] #24661: remove_accents is not removing combining accents

WordPress Trac noreply at wordpress.org
Fri Sep 16 16:04:34 UTC 2016


#24661: remove_accents is not removing combining accents
----------------------------------+--------------------
 Reporter:  NumidWasNotAvailable  |       Owner:
     Type:  defect (bug)          |      Status:  new
 Priority:  normal                |   Milestone:  4.7
Component:  Formatting            |     Version:  1.2.1
 Severity:  normal                |  Resolution:
 Keywords:  has-patch             |     Focuses:
----------------------------------+--------------------
Changes (by gitlost):

 * keywords:  has-patch needs-refresh => has-patch


Comment:

 Hi, do you mean put the constant through an `apply_filters()`? If so, is
 there a use case?

 Could use just ASCII `[A-Za-z]` instead of `WP_LATIN_SCRIPT_REGEX_ALT`
 (and `\p{Latin}`), as the stripping here comes after the ASCII mapping,
 rather than before as in the ICU situation, but against that the `$chars`
 mapping isn't complete.

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


More information about the wp-trac mailing list