Updating Inflector::slug() handling of $map argument. Fixes #18.

When passing a $map to Inflector::slug(), the $map values will now
overwrite any default mappings that Inflector::slug defines.

Modified pre-existing test-case for Inflector::slug() when using a $map.

cake/libs/inflector.php

@@ -521,7 +521,6 @@ function slug($string, $replacement = '_', $map = array()) {
 			$map = $replacement;
 			$replacement = '_';
 		}
-
 		$quotedReplacement = preg_quote($replacement, '/');
 
 		$default = array(
@@ -544,7 +543,7 @@ function slug($string, $replacement = '_', $map = array()) {
 			sprintf('/^[%s]+|[%s]+$/', $quotedReplacement, $quotedReplacement) => '',
 		);
 
-		$map = array_merge($default, $map);
+		$map = array_merge($map, $default);
 		return preg_replace(array_keys($map), array_values($map), $string);
 	}
 }

cake/tests/cases/libs/inflector.test.php

@@ -236,12 +236,24 @@ function testInflectorSlug() {
  * @return void
  */
 	function testInflectorSlugWithMap() {
-		$result = Inflector::slug('replace every r', array('/r/' => '_'));
-		$expected = '_eplace_eve_y__';
+		$result = Inflector::slug('replace every r', array('/r/' => '1'));
+		$expected = '1eplace_eve1y_1';
 		$this->assertEqual($result, $expected);
 
-		$result = Inflector::slug('replace every r', '_', array('/r/' => '_'));
-		$expected = '_eplace_eve_y__';
+		$result = Inflector::slug('replace every r', '_', array('/r/' => '1'));
+		$expected = '1eplace_eve1y_1';
+		$this->assertEqual($result, $expected);
+	}
+
+/**
+ * testInflectorSlugWithMapOverridingDefault method
+ *
+ * @access public
+ * @return void
+ */
+	function testInflectorSlugWithMapOverridingDefault() {
+		$result = Inflector::slug('Testing æ ø å', '-', array('/å/' => 'aa', '/ø/' => 'oe'));
+		$expected = 'Testing-ae-oe-aa';
 		$this->assertEqual($result, $expected);
 	}