Skip to content

Commit 00df73c

Browse files
committed
Make the UPGRADING note about ArrayObject more explicit.
We noticed the BC break when testing `curl_setopt()`, and hadn't interpreted this note as affecting internal functions in general, so this adds a sub-bullet to note that. This overlaps a bit with the previous bullet, but since I don't know exactly which operations were in mind at the time, I've elected to preserve them both. Bits are cheap.
1 parent bcb8ab3 commit 00df73c

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

UPGRADING

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -120,6 +120,9 @@ PHP 7.4 UPGRADE NOTES
120120
* ReflectionObject::getProperties()
121121
* reset(), current(), etc. Use Iterator methods instead.
122122
* Potentially others working on object properties as a list.
123+
* Other internal functions that iterate over an array, but which
124+
previously silently accepted an ArrayObject as well; eg curl_setopt()
125+
when used with an option that expects an array.
123126

124127
(array) casts are *not* affected. They will continue to return either the
125128
wrapped array, or the ArrayObject properties, depending on whether the

0 commit comments

Comments
 (0)