[wp-trac] [WordPress Trac] #64538: memoize wp_normalize_path

[WordPress Trac]

#64538: memoize wp_normalize_path
--------------------------------------+--------------------------
 Reporter:  josephscott               |       Owner:  dmsnell
     Type:  defect (bug)              |      Status:  reopened
 Priority:  normal                    |   Milestone:  7.0
Component:  General                   |     Version:  3.9
 Severity:  normal                    |  Resolution:
 Keywords:  has-patch has-unit-tests  |     Focuses:  performance
--------------------------------------+--------------------------

Comment (by dmsnell):

 @josephscott wild.

 this appears then to be another case of //tests failing//.

 perhaps instead of splitting the tests, which seems a bit awkward, can we
 think of a way to assert the end behavior rather than the the method by
 which the function accomplishes it?

 ----

 As an aside, when I follow the reproduction steps with PHP 8.1.33 and also
 with PHP 8.2.29 it works appropriately, so it probably doesn’t affect all
 of the PHP 8.1 and 8.2 lines. Maybe some hosts haven’t updated to the
 latest minor releases.


 {{{
 array(1) {
   ["key"]=>
   string(5) "value"
 }
 }}}


 Were you successful in reproducing this with the proposed snippet?

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