ServiceDefinition: removed unnecessary $service variable [Closes #206]

Author: dg

src/DI/Definitions/ServiceDefinition.php

@@ -237,12 +237,16 @@ public function complete(Nette\DI\Resolver $resolver): void
 	public function generateMethod(Nette\PhpGenerator\Method $method, Nette\DI\PhpGenerator $generator): void
 	{
 		$entity = $this->factory->getEntity();
-		$code = '$service = ' . $generator->formatStatement($this->factory) . ";\n";
-		$type = $this->getType();
+		$code = $generator->formatStatement($this->factory) . ";\n";
+		if (!$this->setup) {
+			$method->setBody('return ' . $code);
+			return;
+		}
 
+		$code = '$service = ' . $code;
+		$type = $this->getType();
 		if (
-			$this->setup
-			&& $type !== $entity
+			$type !== $entity
 			&& !(is_array($entity) && $entity[0] instanceof Reference && $entity[0]->getValue() === Nette\DI\ContainerBuilder::THIS_CONTAINER)
 			&& !(is_string($entity) && preg_match('#^[\w\\\\]+$#D', $entity) && is_subclass_of($entity, $type))
 		) {

tests/DI/Definitions.FactoryDefinition.render.phpt

@@ -49,8 +49,7 @@ test(function () {
 
 		public function create(): stdClass
 		{
-			$service = new stdClass;
-			return $service;
+			return new stdClass;
 		}
 	};
 }', $method->__toString());