Skip to content

Commit d5e3f2f

Browse files
bug symfony#28639 [DI] add missing sprintf argument in AbstractRecursivePass::getExpressionLanguage (kaznovac)
This PR was merged into the 4.2-dev branch. Discussion ---------- [DI] add missing sprintf argument in AbstractRecursivePass::getExpressionLanguage | Q | A | ------------- | --- | Branch? | master | Bug fix? | yes | New feature? | no <!-- don't forget to update src/**/CHANGELOG.md files --> | BC breaks? | no <!-- see https://symfony.com/bc --> | Deprecations? | no <!-- don't forget to update UPGRADE-*.md and src/**/CHANGELOG.md files --> | Tests pass? | yes <!-- please add some, will be required by reviewers --> | License | MIT runtime exception in AbstractRecursivePass::getExpressionLanguage is referencing service as third parameter in sprintf, but none was given. Commits ------- dc5bf84 third parameter of sprintf missing in AbstractRecursivePass::getExpressionLanguage
2 parents 15aa25a + dc5bf84 commit d5e3f2f

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/Symfony/Component/DependencyInjection/Compiler/AbstractRecursivePass.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -209,7 +209,7 @@ private function getExpressionLanguage()
209209
$arg = $this->processValue(new Reference($id));
210210
$this->inExpression = false;
211211
if (!$arg instanceof Reference) {
212-
throw new RuntimeException(sprintf('"%s::processValue()" must return a Reference when processing an expression, %s returned for service("%s").', \get_class($this), \is_object($arg) ? \get_class($arg) : \gettype($arg)));
212+
throw new RuntimeException(sprintf('"%s::processValue()" must return a Reference when processing an expression, %s returned for service("%s").', \get_class($this), \is_object($arg) ? \get_class($arg) : \gettype($arg), $id));
213213
}
214214
$arg = sprintf('"%s"', $arg);
215215
}

0 commit comments

Comments
 (0)