Skip to content

Commit ac67b6a

Browse files
minor symfony#60732 [DependencyInjection] Improve docblock on compile() (Crell)
This PR was submitted for the 7.4 branch but it was merged into the 6.4 branch instead. Discussion ---------- [DependencyInjection] Improve docblock on compile() | Q | A | ------------- | --- | Branch? | 7.4 | Bug fix? | yes | New feature? | no | Deprecations? | no | License | MIT This is just a documentation fix to make the `$resolveEnvPlaceholders` parameter more self-explanatory. Its current form confused me greatly and I had it set wrong because I was confused. Commits ------- d890828 Improve docblock on compile()
2 parents 09d02eb + d890828 commit ac67b6a

File tree

1 file changed

+5
-4
lines changed

1 file changed

+5
-4
lines changed

src/Symfony/Component/DependencyInjection/ContainerBuilder.php

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -742,10 +742,11 @@ public function deprecateParameter(string $name, string $package, string $versio
742742
* * The parameter bag is frozen;
743743
* * Extension loading is disabled.
744744
*
745-
* @param bool $resolveEnvPlaceholders Whether %env()% parameters should be resolved using the current
746-
* env vars or be replaced by uniquely identifiable placeholders.
747-
* Set to "true" when you want to use the current ContainerBuilder
748-
* directly, keep to "false" when the container is dumped instead.
745+
* @param bool $resolveEnvPlaceholders Whether %env()% parameters should be resolved at build time using
746+
* the current env var values (true), or be resolved at runtime based
747+
* on the environment (false). In general, this should be set to "true"
748+
* when you want to use the current ContainerBuilder directly, and to
749+
* "false" when the container is dumped instead.
749750
*
750751
* @return void
751752
*/

0 commit comments

Comments
 (0)