[wp-trac] [WordPress Trac] #55105: Introduce a polyfill for `array_is_list()`.

WordPress Trac noreply at wordpress.org
Sat Mar 12 20:41:32 UTC 2022


#55105: Introduce a polyfill for `array_is_list()`.
---------------------------------------------------+---------------------
 Reporter:  costdev                                |       Owner:  (none)
     Type:  feature request                        |      Status:  new
 Priority:  normal                                 |   Milestone:  6.0
Component:  General                                |     Version:  5.9
 Severity:  minor                                  |  Resolution:
 Keywords:  has-patch has-unit-tests dev-feedback  |     Focuses:
---------------------------------------------------+---------------------

Comment (by costdev):

 @jrf I tried to find a use case prior to writing the polyfill. I couldn't
 find one. Examples online seem to literally just reiterate what the
 function does, without providing use cases, and vaguely referring to "PHP
 security vulnerabilities caused by arrays that are not in order", without
 mentioning `array_values()` once.

 **''Why did I write it then?''**

 I'd just reviewed and written tests for some of the polyfills introduced
 in 5.9, and I wasn't quite done with my polyfilling-spree. It's there,
 with unit tests, if use cases are found.

 As we're still early enough in the release cycle, I'll leave this open in
 case someone sees value in introducing this polyfill to Core. If there are
 none provided as we approach feature freeze, I'll go ahead and close as
 `maybelater`, or punt to 6.1 if use cases are actively being discussed.

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


More information about the wp-trac mailing list