Applying patch from 'tkykmw'. Behavior mapped methods are no longer r…

…un through strtolower(). Mapped methods are now case sensitive. Fixes #318
cakephp · Sep 6, 2010 · 5789aaf · 5789aaf
1 parent 2e46d2f
commit 5789aaf
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/cake/libs/model/behavior_collection.php b/cake/libs/model/behavior_collection.php
@@ -191,10 +191,6 @@ public function detach($name) {
  */
 	public function dispatchMethod(&$model, $method, $params = array(), $strict = false) {
 		$methods = array_keys($this->__methods);
-		foreach ($methods as $key => $value) {
-			$methods[$key] = strtolower($value);
-		}
-		$method = strtolower($method);
 		$check = array_flip($methods);
 		$found = isset($check[$method]);
 		$call = null;

diff --git a/cake/tests/cases/libs/model/model_behavior.test.php b/cake/tests/cases/libs/model/model_behavior.test.php
@@ -1024,6 +1024,10 @@ function testBehaviorMethodDispatching() {
 		$result = $Apple->{'look for the remote'}('in the couch');
 		$expected = "Item.name = 'the remote' AND Location.name = 'the couch'";
 		$this->assertEqual($result, $expected);
+
+		$result = $Apple->{'look for THE REMOTE'}('in the couch');
+		$expected = "Item.name = 'THE REMOTE' AND Location.name = 'the couch'";
+		$this->assertEqual($result, $expected, 'Mapped method was lowercased.');
 	}
 
 /**