Page MenuHomePhabricator

In "array_mergev()", guarantee the "call_user_func_array()" parameter list is a natrual list
ClosedPublic

Authored by epriestley on Feb 8 2021, 6:17 PM.
Tags
None
Referenced Files
F18986613: D21551.diff
Nov 17 2025, 3:28 PM
F18955511: D21551.id.diff
Nov 12 2025, 9:17 PM
F18873600: D21551.id51298.diff
Nov 5 2025, 10:28 AM
F18836283: D21551.id51297.diff
Oct 26 2025, 11:43 PM
F18791324: D21551.id51297.diff
Oct 16 2025, 9:16 AM
F18773324: D21551.diff
Oct 9 2025, 9:02 AM
F18578076: D21551.id.diff
Sep 10 2025, 8:55 PM
F18571578: D21551.id51298.diff
Sep 10 2025, 5:10 AM
Subscribers
None

Details

Summary

Ref T13588. The behavior of "call_user_func_array()" has changed in PHP8, and the function now attempts to use array keys as argument names.

This always fails when calling "array_merge()" (which does not accept named parameters), and may cause misbehavior in the general case.

Guarantee the argument is a natural list (with keys "0", "1", "2", ...).

Test Plan

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Feb 8 2021, 6:18 PM
epriestley requested review of this revision.
This revision was automatically updated to reflect the committed changes.