From 7babce4a66fa4f96446f7bb59c31e2cedd32f8b9 Mon Sep 17 00:00:00 2001
From: Sam Minnee <sam@silverstripe.com>
Date: Tue, 19 Feb 2008 00:37:50 +0000
Subject: [PATCH] Merged revisions 45750 via svnmerge from
 svn://svn.silverstripe.com/silverstripe/modules/sapphire/branches/2.1.0-rc3oriwave2

........
  r45750 | sminnee | 2007-11-27 17:45:36 +1300 (Tue, 27 Nov 2007) | 1 line

  Fixed DataObjectSet::insertFirst() - it now uses a numeric key rather than null
........


git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/sapphire/trunk@49791 467b73ca-7a2a-4603-9d3b-597d59a354a9
---
 core/model/DataObjectSet.php | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/core/model/DataObjectSet.php b/core/model/DataObjectSet.php
index 1ed0c176e..7da57579a 100644
--- a/core/model/DataObjectSet.php
+++ b/core/model/DataObjectSet.php
@@ -390,8 +390,8 @@ class DataObjectSet extends ViewableData implements Iterator {
 	 * @param string $key Key to index this DataObject by.
 	 */
 	public function insertFirst($item, $key = null) {
-		if($key != null) {
-			array_shift($this->items, $item);
+		if($key == null) {
+			array_unshift($this->items, $item);
 		} else {
 			// Not very efficient :-(
 			$newItems = array();