diogo/symfony
Archived
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

minor #34663 [Config] Remove extra sprintf arg (ogizanagi)

Browse Source 
This PR was merged into the 4.3 branch.

Discussion
----------

[Config] Remove extra sprintf arg

| Q             | A
| ------------- | ---
| Branch?       | 5.0 <!-- see below -->
| Bug fix?      | no
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tickets       | N/A <!-- prefix each issue number with "Fix #", if any -->
| License       | MIT
| Doc PR        | N/A

there is only one placeholder here

(code in previous branches is commented out for 5.0)

Commits
-------

9f19ae185d [Config] Remove extra sprintf arg
This commit is contained in:
Nicolas Grekas 2019-11-28 09:36:08 +01:00
commit 3c3c8bfb9b
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/Symfony/Component/Config/Definition/VariableNode.php
View File

@ -85,7 +85,7 @@ class VariableNode extends BaseNode implements PrototypeNodeInterface
// this avoids ever passing an empty value to final validation closures
if (!$this->allowEmptyValue && $this->isHandlingPlaceholder() && $this->finalValidationClosures) {
@trigger_error(sprintf('Setting path "%s" to an environment variable is deprecated since Symfony 4.3. Remove "cannotBeEmpty()", "validate()" or include a prefix/suffix value instead.', $this->getPath()), E_USER_DEPRECATED);
// $e = new InvalidConfigurationException(sprintf('The path "%s" cannot contain an environment variable when empty values are not allowed by definition and are validated.', $this->getPath(), json_encode($value)));
// $e = new InvalidConfigurationException(sprintf('The path "%s" cannot contain an environment variable when empty values are not allowed by definition and are validated.', $this->getPath()));
// if ($hint = $this->getInfo()) {
// $e->addHint($hint);
// }