tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-10-22 14:05:37 +02:00
Code Issues Projects Releases Wiki Activity

FIX Bug when specifying 0 in ArrayList::offsetSet

Browse Source
This commit is contained in:
Steve Boyd 2020-12-15 14:50:10 +13:00
parent 43dbcfd001
commit 5be045f9a2
2 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/ORM/ArrayList.php
View File

@ -844,7 +844,7 @@ class ArrayList extends ViewableData implements SS_List, Filterable, Sortable, L
*/
public function offsetSet($offset, $value)
{
if ($offset == null) {
if ($offset === null) {
$this->items[] = $value;
} else {
$this->items[$offset] = $value;

10
tests/php/ORM/ArrayListTest.php
View File

@ -1251,4 +1251,14 @@ class ArrayListTest extends SapphireTest
$this->assertNotEquals(range(1, $upperLimit), $list->toArray());
}
public function testOffsetSet()
{
$list = new ArrayList(['first value', 'second value']);
$this->assertSame(2, $list->count());
$list->offsetSet(0, 'new value');
$this->assertSame(2, $list->count());
$this->assertSame('new value', $list->offsetGet(0));
$this->assertSame('second value', $list->offsetGet(1));
}
}