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

WordPress Trac noreply at wordpress.org
Thu Jul 21 21:10:13 UTC 2022


#24661: remove_accents is not removing combining accents
---------------------------------------------+-----------------------
 Reporter:  NumidWasNotAvailable             |       Owner:  audrasjb
     Type:  defect (bug)                     |      Status:  closed
 Priority:  normal                           |   Milestone:  6.1
Component:  Formatting                       |     Version:  1.2.1
 Severity:  normal                           |  Resolution:  fixed
 Keywords:  has-patch has-unit-tests commit  |     Focuses:
---------------------------------------------+-----------------------
Changes (by audrasjb):

 * owner:  (none) => audrasjb
 * status:  new => closed
 * resolution:   => fixed


Comment:

 In [changeset:"53754" 53754]:
 {{{
 #!CommitTicketReference repository="" revision="53754"
 Formatting: Normalize to Unicode NFC encoding before converting accent
 characters in `remove_accents()`.

 This changeset adds Unicode sequence normalization from NFD to NFC, via
 the `normalizer_normalize()` PHP function which is available with the
 recommended `intl` PHP extension.

 This fixes an issue where NFD characters were not properly sanitized. It
 also provides a unit test for NFD sequences (alternate Unicode
 representations of the same characters).

 Props NumidWasNotAvailable, targz, nacin, nunomorgadinho, p_enrique,
 gitlost, SergeyBiryukov, markoheijnen, mikeschroder, ocean90, pento,
 helen, rodrigosevero, zodiac1978, ironprogrammer, audrasjb, azaozz,
 laboiteare, nuryko, virgar, dxd5001, onnimonni, johnbillion.
 Fixes #24661, #47763, #35951.
 See #30130, #52654.
 }}}

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


More information about the wp-trac mailing list