@@ -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