From 68fcd8c94f29a81328adcfb06b4f681782efde5b Mon Sep 17 00:00:00 2001 From: Ingo Schommer Date: Fri, 15 Oct 2010 02:28:12 +0000 Subject: [PATCH] MINOR Reverted r105264, breaks CompositeFieldTest, FieldSetTest, TranslatableTest (from r105274) git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/sapphire/trunk@112474 467b73ca-7a2a-4603-9d3b-597d59a354a9 --- forms/FieldSet.php | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/forms/FieldSet.php b/forms/FieldSet.php index 632a61aee70..f37d91e5893 100755 --- a/forms/FieldSet.php +++ b/forms/FieldSet.php @@ -335,8 +335,7 @@ public function insertBefore($item, $name) { $i = 0; foreach($this->items as $child) { if($name == $child->Name() || $name == $child->id) { - // On PHP 5.2.6, using $this->items directly in array_splice can segfault - $itms = $this->items; array_splice($itms, $i, 0, array($item)); + array_splice($this->items, $i, 0, array($item)); return $item; } elseif($child->isComposite()) { $ret = $child->insertBefore($item, $name);