[wp-trac] [WordPress Trac] #41524: Remove usage of each() from WP_Upgrader

[WordPress Trac]

#41524: Remove usage of each() from WP_Upgrader
-----------------------------+-------------------------
 Reporter:  johnbillion      |      Owner:
     Type:  task (blessed)   |     Status:  new
 Priority:  normal           |  Milestone:  4.9
Component:  Upgrade/Install  |    Version:
 Severity:  major            |   Keywords:  needs-patch
  Focuses:                   |
-----------------------------+-------------------------
 Ticket split out from #40109.

 `WP_Upgrader::clear_destination()` uses `each()`, which is deprecated in
 PHP 7.2 and therefore should be removed.

 The `each()` function exhibits two pieces of behaviour which mean it can't
 simply be replaced with `foreach()`:

 1. After `each()` has executed, the array cursor will be left on the next
 element of the array, or past the last element if it hits the end of the
 array. This means you can break out of an each loop, and continue the
 iteration at a later point. `Text_Diff_Engine_native::_diag()` does just
 this.
 2. With the `while ( each( $array ) )` pattern, it's possible to alter the
 elements in the array during iteration. `WP_Upgrader::clear_destination()`
 does this.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/41524>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform