8000 bug #50246 [DependencyInjection] Do not check errored definitions’ ty… · symfony/symfony@af784ae · GitHub
[go: up one dir, main page]
Skip to content

Commit af784ae

Browse files
nicolas-grekasnicolas-grekas
committed
bug #50246 [DependencyInjection] Do not check errored definitions’ type (MatTheCat)
This PR was merged into the 5.4 branch. Discussion ---------- [DependencyInjection] Do not check errored definitions’ type | Q | A | ------------- | --- | Branch? | 5.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | Tickets | Needed for #50234 | License | MIT | Doc PR | N/A Commits ------- 668be94 Do not check errored definitions’ type
2 parents f1122a2 + 668be94 commit af784ae

File tree

3 files changed

+26
-1
lines changed

3 files changed

+26
-1
lines changed

‎src/Symfony/Component/DependencyInjection/Compiler/CheckTypeDeclarationsPass.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -210,7 +210,7 @@ private function checkType(Definition $checkedDefinition, $value, \ReflectionPar
210210
$class = null;
211211

212212
10000 if ($value instanceof Definition) {
213-
if ($value->getFactory()) {
213+
if ($value->hasErrors() || $value->getFactory()) {
214214
return;
215215
}
216216

‎src/Symfony/Component/DependencyInjection/Tests/Compiler/CheckTypeDeclarationsPassTest.php

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@
2323
use Symfony\Component\DependencyInjection\ParameterBag\EnvPlaceholderParameterBag;
2424
use Symfony\Component\DependencyInjection\Reference;
2525
use Symfony\Component\DependencyInjection\Tests\Fixtures\CheckTypeDeclarationsPass\Bar;
26+
use Symfony\Component\DependencyInjection\Tests\Fixtures\CheckTypeDeclarationsPass\BarErroredDependency;
2627
use Symfony\Component\DependencyInjection\Tests\Fixtures\CheckTypeDeclarationsPass\BarMethodCall;
2728
use Symfony\Component\DependencyInjection\Tests\Fixtures\CheckTypeDeclarationsPass\BarOptionalArgument;
2829
use Symfony\Component\DependencyInjection\Tests\Fixtures\CheckTypeDeclarationsPass\BarOptionalArgumentNotNull;
@@ -1010,6 +1011,20 @@ public function testIgnoreDefinitionFactoryArgument()
10101011

10111012
$this->addToAssertionCount(1);
10121013
}
1014+
1015+
public function testErroredDefinitionsAreNotChecked()
1016+
{
1017+
$container = new ContainerBuilder();
1018+
$container->register('errored_dependency', BarErroredDependency::class)
1019+
->setArguments([
1020+
(new Definition(Foo::class))
1021+
->addError('error'),
1022+
]);
1023+
1024+
(new CheckTypeDeclarationsPass(true))->process($container);
1025+
1026+
$this->addToAssertionCount(1);
1027+
}
10131028
}
10141029

10151030
class CallableClass

‎src/Symfony/Component/DependencyInjection/Tests/Fixtures/CheckTypeDeclarationsPass/BarErroredDependency.php

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
<?php
2+
3+
namespace Symfony\Component\DependencyInjection\Tests\Fixtures\CheckTypeDeclarationsPass;
4+
5+
class BarErroredDependency
6+
{
7+
public function __construct(\stdClass $foo)
8+
{
9+
}
10+
}

0 commit comments

Comments
 (0)
0