[wp-trac] [WordPress Trac] #56980: Check that the Normalizer class exists in remove_accents()

WordPress Trac noreply at wordpress.org
Fri Nov 11 15:27:16 UTC 2022


#56980: Check that the Normalizer class exists in remove_accents()
------------------------------+-----------------------------
 Reporter:  SergeyBiryukov    |       Owner:  SergeyBiryukov
     Type:  defect (bug)      |      Status:  closed
 Priority:  normal            |   Milestone:  6.1.1
Component:  Formatting        |     Version:
 Severity:  normal            |  Resolution:  fixed
 Keywords:  has-patch commit  |     Focuses:
------------------------------+-----------------------------
Changes (by SergeyBiryukov):

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


Comment:

 In [changeset:"54813" 54813]:
 {{{
 #!CommitTicketReference repository="" revision="54813"
 Formatting: Check that both `normalizer_*` functions exist in
 `remove_accents()`.

 This applies to:
 * `normalizer_is_normalized()`
 * `normalizer_normalize()`

 Includes removing the `Normalizer::FORM_C` constant as a parameter, since
 it is the default value for both functions and does not need to be
 explicitly passed. This avoids a fatal error if a plugin includes polyfill
 for any of the functions but the `Normalizer` class has a different
 namespace, for example when using the Symfony polyfill.

 Follow-up to [53754].

 Props hellofromTonya, costdev, desrosj, mukesh27, zodiac1978, jchambo,
 gisgeo, SergeyBiryukov.
 Fixes #56980.
 }}}

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


More information about the wp-trac mailing list