Skip to content

Commit 82980e1

Browse files
committed
minor #10439 [OptionsResolver] Documenting Options argument for closure deprecation func (yceruto)
This PR was merged into the master branch. Discussion ---------- [OptionsResolver] Documenting Options argument for closure deprecation func See symfony/symfony#28738 Commits ------- abb5189 Documenting Options argument for closure deprecation func
2 parents 8531ef7 + abb5189 commit 82980e1

File tree

1 file changed

+7
-1
lines changed

1 file changed

+7
-1
lines changed

components/options_resolver.rst

+7-1
Original file line numberDiff line numberDiff line change
@@ -660,13 +660,19 @@ This closure is useful to only deprecate some of the allowed types or values of
660660
the option::
661661

662662
$resolver
663+
->setDefault('encryption', null)
663664
->setDefault('port', null)
664665
->setAllowedTypes('port', array('null', 'int'))
665-
->setDeprecated('port', function ($value) {
666+
->setDeprecated('port', function (Options $options, $value) {
666667
if (null === $value) {
667668
return 'Passing "null" to option "port" is deprecated, pass an integer instead.';
668669
}
669670

671+
// deprecation may also depend on another option
672+
if ('ssl' === $options['encryption'] && 456 !== $value) {
673+
return 'Passing a different port than "456" when the "encryption" option is set to "ssl" is deprecated.';
674+
}
675+
670676
return '';
671677
})
672678
;

0 commit comments

Comments
 (0)