Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commitbeaa7a4

Browse files
committed
minor#5753 configureOptions(...) : protected => public (lucascherifi)
This PR was merged into the 2.7 branch.Discussion----------configureOptions(...) : protected => publicAs in Symfony\Component\Form\AbstractType the method is public, overrides should be public too./** * Configures the options for this type. * *@param OptionsResolver $resolver The resolver for the options. */ public function configureOptions(OptionsResolver $resolver) { }Commits-------824fe40 configureOptions(...) : protected => public
2 parentse9b50e4 +824fe40 commitbeaa7a4

File tree

1 file changed

+19
-19
lines changed

1 file changed

+19
-19
lines changed

‎components/options_resolver.rst‎

Lines changed: 19 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -173,7 +173,7 @@ It's a good practice to split the option configuration into a separate method::
173173
$this->options = $resolver->resolve($options);
174174
}
175175

176-
protected function configureOptions(OptionsResolver $resolver)
176+
public function configureOptions(OptionsResolver $resolver)
177177
{
178178
$resolver->setDefaults(array(
179179
'host' => 'smtp.example.org',
@@ -192,7 +192,7 @@ than processing options. Second, sub-classes may now override the
192192
// ...
193193
class GoogleMailer extends Mailer
194194
{
195-
protected function configureOptions(OptionsResolver $resolver)
195+
public function configureOptions(OptionsResolver $resolver)
196196
{
197197
parent::configureOptions($resolver);
198198

@@ -215,7 +215,7 @@ For example, to make the ``host`` option required, you can do::
215215
{
216216
// ...
217217

218-
protected function configureOptions(OptionsResolver $resolver)
218+
public function configureOptions(OptionsResolver $resolver)
219219
{
220220
// ...
221221
$resolver->setRequired('host');
@@ -243,7 +243,7 @@ one required option::
243243
{
244244
// ...
245245

246-
protected function configureOptions(OptionsResolver $resolver)
246+
public function configureOptions(OptionsResolver $resolver)
247247
{
248248
// ...
249249
$resolver->setRequired(array('host', 'username', 'password'));
@@ -263,7 +263,7 @@ retrieve the names of all required options::
263263
// ...
264264
class GoogleMailer extends Mailer
265265
{
266-
protected function configureOptions(OptionsResolver $resolver)
266+
public function configureOptions(OptionsResolver $resolver)
267267
{
268268
parent::configureOptions($resolver);
269269

@@ -291,7 +291,7 @@ been set::
291291
{
292292
// ...
293293

294-
protected function configureOptions(OptionsResolver $resolver)
294+
public function configureOptions(OptionsResolver $resolver)
295295
{
296296
// ...
297297
$resolver->setRequired('host');
@@ -301,7 +301,7 @@ been set::
301301
// ...
302302
class GoogleMailer extends Mailer
303303
{
304-
protected function configureOptions(OptionsResolver $resolver)
304+
public function configureOptions(OptionsResolver $resolver)
305305
{
306306
parent::configureOptions($resolver);
307307

@@ -336,7 +336,7 @@ correctly. To validate the types of the options, call
336336
{
337337
// ...
338338

339-
protected function configureOptions(OptionsResolver $resolver)
339+
public function configureOptions(OptionsResolver $resolver)
340340
{
341341
// ...
342342
$resolver->setAllowedTypes('host', 'string');
@@ -381,7 +381,7 @@ to verify that the passed option contains one of these values::
381381
{
382382
// ...
383383

384-
protected function configureOptions(OptionsResolver $resolver)
384+
public function configureOptions(OptionsResolver $resolver)
385385
{
386386
// ...
387387
$resolver->setDefault('transport', 'sendmail');
@@ -432,7 +432,7 @@ option. You can configure a normalizer by calling
432432
{
433433
// ...
434434

435-
protected function configureOptions(OptionsResolver $resolver)
435+
public function configureOptions(OptionsResolver $resolver)
436436
{
437437
// ...
438438

@@ -459,7 +459,7 @@ if you need to use other options during normalization::
459459
class Mailer
460460
{
461461
// ...
462-
protected function configureOptions(OptionsResolver $resolver)
462+
public function configureOptions(OptionsResolver $resolver)
463463
{
464464
// ...
465465
$resolver->setNormalizer('host', function ($options, $value) {
@@ -493,7 +493,7 @@ these options, you can return the desired default value::
493493
class Mailer
494494
{
495495
// ...
496-
protected function configureOptions(OptionsResolver $resolver)
496+
public function configureOptions(OptionsResolver $resolver)
497497
{
498498
// ...
499499
$resolver->setDefault('encryption', null);
@@ -525,7 +525,7 @@ the closure::
525525
class Mailer
526526
{
527527
// ...
528-
protected function configureOptions(OptionsResolver $resolver)
528+
public function configureOptions(OptionsResolver $resolver)
529529
{
530530
// ...
531531
$resolver->setDefaults(array(
@@ -537,7 +537,7 @@ the closure::
537537

538538
class GoogleMailer extends Mailer
539539
{
540-
protected function configureOptions(OptionsResolver $resolver)
540+
public function configureOptions(OptionsResolver $resolver)
541541
{
542542
parent::configureOptions($resolver);
543543

@@ -568,7 +568,7 @@ comes from the default::
568568
class Mailer
569569
{
570570
// ...
571-
protected function configureOptions(OptionsResolver $resolver)
571+
public function configureOptions(OptionsResolver $resolver)
572572
{
573573
// ...
574574
$resolver->setDefault('port', 25);
@@ -600,7 +600,7 @@ be included in the resolved options if it was actually passed to
600600
{
601601
// ...
602602

603-
protected function configureOptions(OptionsResolver $resolver)
603+
public function configureOptions(OptionsResolver $resolver)
604604
{
605605
// ...
606606
$resolver->setDefined('port');
@@ -634,7 +634,7 @@ options in one go::
634634
class Mailer
635635
{
636636
// ...
637-
protected function configureOptions(OptionsResolver $resolver)
637+
public function configureOptions(OptionsResolver $resolver)
638638
{
639639
// ...
640640
$resolver->setDefined(array('port', 'encryption'));
@@ -655,7 +655,7 @@ let you find out which options are defined::
655655
{
656656
// ...
657657

658-
protected function configureOptions(OptionsResolver $resolver)
658+
public function configureOptions(OptionsResolver $resolver)
659659
{
660660
parent::configureOptions($resolver);
661661

@@ -701,7 +701,7 @@ can change your code to do the configuration only once per class::
701701
$this->options = self::$resolversByClass[$class]->resolve($options);
702702
}
703703

704-
protected function configureOptions(OptionsResolver $resolver)
704+
public function configureOptions(OptionsResolver $resolver)
705705
{
706706
// ...
707707
}

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp