ContainerBuilder::resolveEntityClass() checking whether factory is callable counts with interfaces and traits [Closes #32]

Author: dg

src/DI/ContainerBuilder.php

@@ -376,15 +376,16 @@ private function resolveEntityClass($entity, $recursive = array())
 
 			try {
 				$reflection = Nette\Utils\Callback::toReflection($entity);
+				$refClass = $reflection instanceof \ReflectionMethod ? $reflection->getDeclaringClass() : NULL;
 			} catch (\ReflectionException $e) {
 			}
-			if (isset($e) || !is_callable($entity)) {
+			if (isset($e) || ($refClass && !$reflection->isPublic() || !($reflection->isStatic() || $refClass->isInstantiable() || $refClass->isInterface()))) {
 				$name = array_slice(array_keys($recursive), -1);
 				throw new ServiceCreationException(sprintf("Factory '%s' used in service '%s' is not callable.", Nette\Utils\Callback::toString($entity), $name[0]));
 			}
 			$class = preg_replace('#[|\s].*#', '', $reflection->getAnnotation('return'));
-			if ($class && $reflection instanceof \ReflectionMethod) {
-				$class = Reflection\AnnotationsParser::expandClassName($class, $reflection->getDeclaringClass());
+			if ($class && $refClass) {
+				$class = Reflection\AnnotationsParser::expandClassName($class, $refClass);
 			}
 			return $class;
 

tests/DI/ContainerBuilder.byClass.phpt

@@ -11,11 +11,16 @@ use Nette\DI,
 require __DIR__ . '/../bootstrap.php';
 
 
-class Factory
+interface IFactory
+{
+	static function create();
+}
+
+class Factory implements IFactory
 {
 	public static $methods;
 
-	static function create($arg)
+	static function create()
 	{
 		self::$methods[] = array(__FUNCTION__, func_get_args());
 		return new stdClass;
@@ -59,6 +64,11 @@ $builder->addDefinition('four')
 	->setAutowired(FALSE)
 	->setFactory('@\AnnotatedFactory::create');
 
+$builder->addDefinition('five')
+	->setAutowired(FALSE)
+	->setFactory('@\IFactory::create');
+
+
 
 $container = createContainer($builder);
 
@@ -85,3 +95,5 @@ Assert::type( 'stdClass', $container->getService('four') );
 Assert::same(array(
 	array('create', array()),
 ), $annotatedFactory->methods);
+
+Assert::type( 'stdClass', $container->getService('five') );

tests/DI/ContainerBuilder.factory.error2.phpt

@@ -0,0 +1,24 @@
+<?php
+
+/**
+ * Test: Nette\DI\ContainerBuilder and generated factories errors.
+ * @phpversion 5.4
+ */
+
+use Nette\DI,
+	Tester\Assert;
+
+
+require __DIR__ . '/../bootstrap.php';
+
+
+trait Bad1
+{
+	function method() {}
+}
+
+Assert::exception(function() {
+	$builder = new DI\ContainerBuilder;
+	$builder->addDefinition('one')->setFactory('Bad1::method');
+	$builder->generateClasses();
+}, 'Nette\InvalidStateException', "Factory 'Bad1::method' used in service 'one' is not callable.");