ClassType: method names are case insensitive

Author: dg

src/PhpGenerator/ClassType.php

@@ -395,7 +395,7 @@ public function addMember($member): self
 			if ($this->isInterface()) {
 				$member->setBody(null);
 			}
-			$this->methods[$member->getName()] = $member;
+			$this->methods[strtolower($member->getName())] = $member;
 
 		} elseif ($member instanceof Property) {
 			$this->properties[$member->getName()] = $member;
@@ -563,8 +563,8 @@ public function setMethods(array $methods): self
 	{
 		(function (Method ...$methods) {})(...array_values($methods));
 		$this->methods = [];
-		foreach ($methods as $v) {
-			$this->methods[$v->getName()] = $v;
+		foreach ($methods as $m) {
+			$this->methods[strtolower($m->getName())] = $m;
 		}
 		return $this;
 	}
@@ -573,16 +573,21 @@ public function setMethods(array $methods): self
 	/** @return Method[] */
 	public function getMethods(): array
 	{
-		return $this->methods;
+		$res = [];
+		foreach ($this->methods as $m) {
+			$res[$m->getName()] = $m;
+		}
+		return $res;
 	}
 
 
 	public function getMethod(string $name): Method
 	{
-		if (!isset($this->methods[$name])) {
+		$m = $this->methods[strtolower($name)] ?? null;
+		if (!$m) {
 			throw new Nette\InvalidArgumentException("Method '$name' not found.");
 		}
-		return $this->methods[$name];
+		return $m;
 	}
 
 
@@ -594,21 +599,21 @@ public function addMethod(string $name): Method
 		} else {
 			$method->setPublic();
 		}
-		return $this->methods[$name] = $method;
+		return $this->methods[strtolower($name)] = $method;
 	}
 
 
 	/** @return static */
 	public function removeMethod(string $name): self
 	{
-		unset($this->methods[$name]);
+		unset($this->methods[strtolower($name)]);
 		return $this;
 	}
 
 
 	public function hasMethod(string $name): bool
 	{
-		return isset($this->methods[$name]);
+		return isset($this->methods[strtolower($name)]);
 	}
 
 

tests/PhpGenerator/ClassType.addMember.phpt

@@ -16,6 +16,7 @@ Assert::exception(function () {
 
 
 $class = (new ClassType('Example'))
+	->addMember($method = new Nette\PhpGenerator\Method('GETHANDLE'))
 	->addMember($method = new Nette\PhpGenerator\Method('getHandle'))
 	->addMember($property = new Nette\PhpGenerator\Property('handle'))
 	->addMember($const = new Nette\PhpGenerator\Constant('ROLE'))

tests/PhpGenerator/ClassType.phpt

@@ -186,7 +186,8 @@ $class->removeProperty('b')->removeProperty('c');
 Assert::same(['a'], array_keys($class->getProperties()));
 
 $class->addMethod('a');
+$class->addMethod('A');
 $class->addMethod('b');
-$class->removeMethod('b')->removeMethod('c');
+$class->removeMethod('B')->removeMethod('c');
 
-Assert::same(['a'], array_keys($class->getMethods()));
+Assert::same(['A'], array_keys($class->getMethods()));