ClassType: cloning includes attributes and parameters

Author: dg

src/PhpGenerator/ClassLike.php

@@ -140,4 +140,10 @@ protected function validateNames(array $names): void
 	public function validate(): void
 	{
 	}
+
+
+	public function __clone(): void
+	{
+		$this->attributes = array_map(fn($attr) => clone $attr, $this->attributes);
+	}
 }

src/PhpGenerator/ClassType.php

@@ -260,8 +260,9 @@ public function validate(): void
 	}
 
 
-	public function __clone()
+	public function __clone(): void
 	{
+		parent::__clone();
 		$clone = fn($item) => clone $item;
 		$this->consts = array_map($clone, $this->consts);
 		$this->methods = array_map($clone, $this->methods);

src/PhpGenerator/Closure.php

@@ -57,4 +57,10 @@ public function addUse(string $name): Parameter
 	{
 		return $this->uses[] = new Parameter($name);
 	}
+
+
+	public function __clone(): void
+	{
+		$this->parameters = array_map(fn($param) => clone $param, $this->parameters);
+	}
 }

src/PhpGenerator/EnumType.php

@@ -136,8 +136,9 @@ public function addMember(Method|Constant|EnumCase|TraitUse $member, bool $overw
 	}
 
 
-	public function __clone()
+	public function __clone(): void
 	{
+		parent::__clone();
 		$clone = fn($item) => clone $item;
 		$this->consts = array_map($clone, $this->consts);
 		$this->methods = array_map($clone, $this->methods);

src/PhpGenerator/GlobalFunction.php

@@ -32,4 +32,10 @@ public function __toString(): string
 	{
 		return (new Printer)->printFunction($this);
 	}
+
+
+	public function __clone(): void
+	{
+		$this->parameters = array_map(fn($param) => clone $param, $this->parameters);
+	}
 }

src/PhpGenerator/InterfaceType.php

@@ -69,8 +69,9 @@ public function addMember(Method|Constant $member, bool $overwrite = false): sta
 	}
 
 
-	public function __clone()
+	public function __clone(): void
 	{
+		parent::__clone();
 		$clone = fn($item) => clone $item;
 		$this->consts = array_map($clone, $this->consts);
 		$this->methods = array_map($clone, $this->methods);

src/PhpGenerator/Method.php

@@ -100,4 +100,10 @@ public function validate(): void
 			throw new Nette\InvalidStateException("Method $this->name() cannot be abstract and final or private at the same time.");
 		}
 	}
+
+
+	public function __clone(): void
+	{
+		$this->parameters = array_map(fn($param) => clone $param, $this->parameters);
+	}
 }

src/PhpGenerator/TraitType.php

@@ -42,8 +42,9 @@ public function addMember(Method|Property|Constant|TraitUse $member, bool $overw
 	}
 
 
-	public function __clone()
+	public function __clone(): void
 	{
+		parent::__clone();
 		$clone = fn($item) => clone $item;
 		$this->consts = array_map($clone, $this->consts);
 		$this->methods = array_map($clone, $this->methods);

tests/PhpGenerator/ClassType.clone.phpt

@@ -10,12 +10,16 @@ require __DIR__ . '/../bootstrap.php';
 
 $class = new ClassType('Example');
 
+$class->addAttribute('Attr');
 $class->addConstant('A', 10);
 $class->addProperty('a');
-$class->addMethod('a');
+$class->addMethod('a')
+	->addParameter('foo');
 
 $dolly = clone $class;
 
+Assert::notSame($dolly->getAttributes(), $class->getAttributes());
 Assert::notSame($dolly->getConstants(), $class->getConstants());
 Assert::notSame($dolly->getProperty('a'), $class->getProperty('a'));
 Assert::notSame($dolly->getMethod('a'), $class->getMethod('a'));
+Assert::notSame($dolly->getMethod('a')->getParameter('foo'), $class->getMethod('a')->getParameter('foo'));