BUGFIX SS_Map::keys() and SS_Map::values() are identical, keys() should return the *keys* not the values silverstripe#6818

Author: halkyon

model/Map.php

@@ -47,28 +47,29 @@ function toArray() {
 		}
 		return $array;
 	}
-	
+
 	/**
 	 * Return all the keys of this map
 	 */
 	function keys() {
-		$array = array();
+		$output = array();
 		foreach($this as $k => $v) {
-			$array[] = $v;
+			$output[] = $k;
 		}
-		return $array;
+		return $output;
 	}
+
 	/**
 	 * Return all the values of this map
 	 */
 	function values() {
-		$array = array();
+		$output = array();
 		foreach($this as $k => $v) {
-			$array[] = $v;
+			$output[] = $v;
 		}
-		return $array;
+		return $output;
 	}
-	
+
 	/**
 	 * Unshift an item onto the start of the map
 	 */

tests/model/MapTest.php

@@ -57,6 +57,26 @@ function testToArray() {
 			"Phil" => "Phil is a unique guy, and comments on team2"), $map->toArray());
 	}
 
+	function testKeys() {
+		$list = DataList::create('DataObjectTest_TeamComment');
+		$map = new SS_Map($list, 'Name', 'Comment');
+		$this->assertEquals(array(
+			'Joe',
+			'Bob',
+			'Phil'
+		), $map->keys());
+	}
+
+	function testValues() {
+		$list = DataList::create('DataObjectTest_TeamComment');
+		$map = new SS_Map($list, 'Name', 'Comment');
+		$this->assertEquals(array(
+			'This is a team comment by Joe',
+			'This is a team comment by Bob',
+			'Phil is a unique guy, and comments on team2'
+		), $map->values());
+	}
+
 	function testUnshift() {
 		$list = DataList::create("DataObjectTest_TeamComment");
 		$map = new SS_Map($list, 'Name', 'Comment');