Commit 4628dec

committed

ClassManipulator::implementInterface() renamed to implement()

1 parent 5dab02e commit 4628decCopy full SHA for 4628dec

File tree

+18

-10

lines changed

+18

-10

lines changed

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -902,10 +902,10 @@ $property = $manipulator->inheritProperty('foo');`
`902`	`902`	`$property->setValue('new value');`
`903`	`903`	```
`904`	`904`
`905`		-The `implementInterface()` method automatically implements all methods from the given interface in your class:
	`905`	+The `implement()` method automatically implements all methods from the given interface in your class:
`906`	`906`
`907`	`907`	```php
`908`		`-$manipulator->implementInterface(SomeInterface::class);`
	`908`	`+$manipulator->implement(SomeInterface::class);`
`909`	`909`	`// Now your class implements SomeInterface and includes all its methods`
`910`	`910`	```
`911`	`911`

Lines changed: 14 additions & 6 deletions

Original file line number	Diff line number	Diff line change
`@@ -80,16 +80,24 @@ public function inheritMethod(string $name, bool $returnIfExists = false): Metho`
`80`	`80`	`/**`
`81`	`81`	`* Implements all methods from the given interface.`
`82`	`82`	`*/`
`83`		`- public function implementInterface(string $interfaceName): void`
	`83`	`+ public function implement(string $name): void`
`84`	`84`	`{`
`85`		`- $interface = new \ReflectionClass($interfaceName);`
`86`		`- if (!$interface->isInterface()) {`
`87`		`- throw new Nette\InvalidArgumentException("Class '$interfaceName' is not an interface.");`
	`85`	`+ $definition = new \ReflectionClass($name);`
	`86`	`+ if (!$definition->isInterface()) {`
	`87`	`+ throw new Nette\InvalidArgumentException("Class '$name' is not an interface.");`
`88`	`88`	`}`
`89`	`89`
`90`		`- $this->class->addImplement($interfaceName);`
`91`		`- foreach ($interface->getMethods() as $method) {`
	`90`	`+ $this->class->addImplement($name);`
	`91`	`+ foreach ($definition->getMethods() as $method) {`
`92`	`92`	`$this->inheritMethod($method->getName(), returnIfExists: true);`
`93`	`93`	`}`
`94`	`94`	`}`
	`95`	`+`
	`96`	`+`
	`97`	`+ /** @deprecated use implement() */`
	`98`	`+ public function implementInterface(string $interfaceName): void`
	`99`	`+ {`
	`100`	`+ trigger_error(__METHOD__ . '() is deprecated, use implement()', E_USER_DEPRECATED);`
	`101`	`+ $this->implement($interfaceName);`
	`102`	`+ }`
`95`	`103`	`}`

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -18,12 +18,12 @@ $class = new ClassType('TestClass');`
`18`	`18`	`$manipulator = new ClassManipulator($class);`
`19`	`19`
`20`	`20`	`// Test valid interface implementation`
`21`		`-$manipulator->implementInterface(TestInterface::class);`
	`21`	`+$manipulator->implement(TestInterface::class);`
`22`	`22`	`Assert::true(in_array(TestInterface::class, $class->getImplements(), true));`
`23`	`23`	`Assert::true($class->hasMethod('testMethod'));`
`24`	`24`
`25`	`25`	`// Test exception for non-interface`
`26`	`26`	`Assert::exception(`
`27`		`- fn() => $manipulator->implementInterface(stdClass::class),`
	`27`	`+ fn() => $manipulator->implement(stdClass::class),`
`28`	`28`	`InvalidArgumentException::class,`
`29`	`29`	`);`

Comments

(0)