annotations @Inject is deprecated, use #[Inject] (BC break)

Author: dg

src/DI/Extensions/InjectExtension.php

@@ -120,6 +120,9 @@ public static function getInjectProperties(string $class): array
 		foreach ((new \ReflectionClass($class))->getProperties() as $rp) {
 			$hasAttr = $rp->getAttributes(DI\Attributes\Inject::class);
 			if ($hasAttr || DI\Helpers::parseAnnotation($rp, 'inject') !== null) {
+				if (!$hasAttr) {
+					trigger_error('Annotation @inject is deprecated, use #[Nette\DI\Attributes\Inject] and native typehint (used in ' . Reflection::toString($rp) . ')', E_USER_DEPRECATED);
+				}
 				if (!$rp->isPublic() || $rp->isStatic() || $rp->isReadOnly()) {
 					throw new Nette\InvalidStateException(sprintf('Property %s for injection must not be static, readonly and must be public.', Reflection::toString($rp)));
 				}

tests/DI/Container.inject.properties.phpt

@@ -7,6 +7,7 @@
 declare(strict_types=1);
 
 use Nette\DI;
+use Nette\DI\Attributes\Inject;
 use Tester\Assert;
 
 
@@ -23,20 +24,17 @@ class Foo implements IFoo
 
 class Test1
 {
-	/** @inject @var stdClass */
-	public $varA;
-
-	/** @var stdClass @inject */
-	public $varB;
+	#[Inject]
+	public stdClass $varA;
 }
 
 class Test2 extends Test1
 {
-	/** @var stdClass @inject */
-	public $varC;
+	#[Inject]
+	public stdClass $varC;
 
-	/** @var IFoo @inject */
-	public $varD;
+	#[Inject]
+	public IFoo $varD;
 }
 
 
@@ -52,6 +50,5 @@ $container = createContainer($builder);
 $test = new Test2;
 $container->callInjects($test);
 Assert::type(stdClass::class, $test->varA);
-Assert::type(stdClass::class, $test->varB);
 Assert::type(stdClass::class, $test->varC);
 Assert::type(Foo::class, $test->varD);

tests/DI/InjectExtension.basic.phpt

@@ -7,6 +7,7 @@
 declare(strict_types=1);
 
 use Nette\DI;
+use Nette\DI\Attributes\Inject;
 use Nette\DI\Definitions\Reference;
 use Nette\DI\Definitions\Statement;
 use Tester\Assert;
@@ -31,8 +32,8 @@ class ConcreteDependencyB extends AbstractDependency
 
 class ParentClass
 {
-	/** @var stdClass @inject */
-	public $a;
+	#[Inject]
+	public stdClass $a;
 
 
 	public function injectA()
@@ -47,11 +48,11 @@ class ParentClass
 
 class Service extends ParentClass
 {
-	/** @var stdClass @inject */
-	public $c;
+	#[Inject]
+	public stdClass $c;
 
-	/** @var AbstractDependency @inject */
-	public $e;
+	#[Inject]
+	public AbstractDependency $e;
 
 
 	public function injectC()

tests/DI/InjectExtension.errors.phpt

@@ -7,6 +7,7 @@
 declare(strict_types=1);
 
 use Nette\DI;
+use Nette\DI\Attributes\Inject;
 use Nette\InvalidStateException;
 use Tester\Assert;
 
@@ -16,35 +17,35 @@ require __DIR__ . '/../bootstrap.php';
 
 class ServiceA
 {
-	/** @var DateTimeImmutable @inject */
-	public $a;
+	#[Inject]
+	public DateTimeImmutable $a;
 }
 
 
 class ServiceB
 {
-	/** @var Unknown @inject */
-	public $a;
+	#[Inject]
+	public Unknown $a;
 }
 
 
 class ServiceC
 {
-	/** @inject */
+	#[Inject]
 	public $a;
 }
 
 
 class ServiceD
 {
-	/** @inject */
+	#[Inject]
 	protected $a;
 }
 
 
 class ServiceE
 {
-	/** @inject */
+	#[Inject]
 	public static $a;
 }
 

tests/DI/InjectExtension.getInjectProperties().phpt

@@ -6,106 +6,51 @@
 
 declare(strict_types=1);
 
-namespace A
+use Nette\DI\Attributes\Inject;
+use Nette\DI\Extensions\InjectExtension;
+use Tester\Assert;
+
+
+class AClass
 {
-	class AClass
-	{
-		/** @var Different @inject */
-		public AInjected $varA;
-
-		/** @var B\BInjected @inject */
-		public $varB;
-
-		/** @inject */
-		public AInjected $varC;
-
-		/** @var AInjected */
-		public $varD;
-	}
-
-	class AInjected
-	{
-	}
-
-	class BadClass
-	{
-		/** @inject */
-		public AClass|\stdClass $var;
-	}
+	#[Inject]
+	public AInjected $varA;
+
+	#[Inject]
+	public BInjected $varB;
+
+	public $varD;
+
+	#[Inject]
+	public stdClass $varF;
 }
 
-namespace A\B
+class BadClass
 {
-	use A;
-	use Nette\DI\Attributes\Inject;
-
-	class BClass extends A\AClass
-	{
-		#[Inject]
-		public BInjected $varF;
-	}
-
-	class BInjected
-	{
-	}
+	#[Inject]
+	public AClass|stdClass $var;
 }
 
-namespace C
+class AInjected
 {
-	use A\AInjected;
-	use A\B;
-	use C\CInjected as CAlias;
+}
 
-	class CClass
-	{
-		/** @var AInjected @inject */
-		public $var1;
+class BInjected
+{
+}
 
-		/** @var B\BInjected @inject */
-		public $var2;
 
-		/** @var CAlias @inject */
-		public $var3;
+require __DIR__ . '/../bootstrap.php';
 
-		/** @var CInjected @inject */
-		public $var4;
-	}
 
-	class CInjected
-	{
-	}
-}
+Assert::same([
+	'varA' => AInjected::class,
+	'varB' => BInjected::class,
+	'varF' => stdClass::class,
+], InjectExtension::getInjectProperties(AClass::class));
 
-namespace {
-	use Nette\DI\Extensions\InjectExtension;
-	use Tester\Assert;
-
-	require __DIR__ . '/../bootstrap.php';
-
-
-	Assert::same([
-		'varA' => A\AInjected::class,
-		'varB' => A\B\BInjected::class,
-		'varC' => A\AInjected::class,
-	], InjectExtension::getInjectProperties(A\AClass::class));
-
-	Assert::same([
-		'varA' => A\AInjected::class,
-		'varB' => A\B\BInjected::class,
-		'varC' => A\AInjected::class,
-		'varF' => A\B\BInjected::class,
-	], InjectExtension::getInjectProperties(A\B\BClass::class));
-
-	Assert::same([
-		'var1' => A\AInjected::class,
-		'var2' => A\B\BInjected::class,
-		'var3' => C\CInjected::class,
-		'var4' => C\CInjected::class,
-	], InjectExtension::getInjectProperties(C\CClass::class));
-
-	Assert::exception(
-		fn() => InjectExtension::getInjectProperties(A\BadClass::class),
-		Nette\InvalidStateException::class,
-		"Type of property A\\BadClass::\$var is expected to not be nullable/built-in/complex, 'A\\AClass|stdClass' given.",
-	);
-}
+Assert::exception(
+	fn() => InjectExtension::getInjectProperties(BadClass::class),
+	Nette\InvalidStateException::class,
+	"Type of property BadClass::\$var is expected to not be nullable/built-in/complex, 'AClass|stdClass' given.",
+);

tests/DI/InjectExtension.getInjectProperties().traits.phpt

@@ -16,11 +16,12 @@ namespace A
 namespace B
 {
 	use A\AInjected;
+	use Nette\DI\Attributes\Inject;
 
 	trait BTrait
 	{
-		/** @var AInjected @inject */
-		public $varA;
+		#[Inject]
+		public AInjected $varA;
 	}
 }