From 824fe40715f0ddd45f7b76c5753fe6ab6f117fe1 Mon Sep 17 00:00:00 2001 From: Lucas CHERIFI Date: Fri, 2 Oct 2015 23:57:45 +0200 Subject: [PATCH] configureOptions(...) : protected => public As in Symfony\Component\Form\AbstractType the method /** * Configures the options for this type. * * @param OptionsResolver $resolver The resolver for the options. */ public function configureOptions(OptionsResolver $resolver) { } --- components/options_resolver.rst | 38 ++++++++++++++++----------------- 1 file changed, 19 insertions(+), 19 deletions(-) diff --git a/components/options_resolver.rst b/components/options_resolver.rst index d10eb7df180..ec5248f03a5 100644 --- a/components/options_resolver.rst +++ b/components/options_resolver.rst @@ -173,7 +173,7 @@ It's a good practice to split the option configuration into a separate method:: $this->options = $resolver->resolve($options); } - protected function configureOptions(OptionsResolver $resolver) + public function configureOptions(OptionsResolver $resolver) { $resolver->setDefaults(array( 'host' => 'smtp.example.org', @@ -192,7 +192,7 @@ than processing options. Second, sub-classes may now override the // ... class GoogleMailer extends Mailer { - protected function configureOptions(OptionsResolver $resolver) + public function configureOptions(OptionsResolver $resolver) { parent::configureOptions($resolver); @@ -215,7 +215,7 @@ For example, to make the ``host`` option required, you can do:: { // ... - protected function configureOptions(OptionsResolver $resolver) + public function configureOptions(OptionsResolver $resolver) { // ... $resolver->setRequired('host'); @@ -243,7 +243,7 @@ one required option:: { // ... - protected function configureOptions(OptionsResolver $resolver) + public function configureOptions(OptionsResolver $resolver) { // ... $resolver->setRequired(array('host', 'username', 'password')); @@ -263,7 +263,7 @@ retrieve the names of all required options:: // ... class GoogleMailer extends Mailer { - protected function configureOptions(OptionsResolver $resolver) + public function configureOptions(OptionsResolver $resolver) { parent::configureOptions($resolver); @@ -291,7 +291,7 @@ been set:: { // ... - protected function configureOptions(OptionsResolver $resolver) + public function configureOptions(OptionsResolver $resolver) { // ... $resolver->setRequired('host'); @@ -301,7 +301,7 @@ been set:: // ... class GoogleMailer extends Mailer { - protected function configureOptions(OptionsResolver $resolver) + public function configureOptions(OptionsResolver $resolver) { parent::configureOptions($resolver); @@ -336,7 +336,7 @@ correctly. To validate the types of the options, call { // ... - protected function configureOptions(OptionsResolver $resolver) + public function configureOptions(OptionsResolver $resolver) { // ... $resolver->setAllowedTypes('host', 'string'); @@ -381,7 +381,7 @@ to verify that the passed option contains one of these values:: { // ... - protected function configureOptions(OptionsResolver $resolver) + public function configureOptions(OptionsResolver $resolver) { // ... $resolver->setDefault('transport', 'sendmail'); @@ -432,7 +432,7 @@ option. You can configure a normalizer by calling { // ... - protected function configureOptions(OptionsResolver $resolver) + public function configureOptions(OptionsResolver $resolver) { // ... @@ -459,7 +459,7 @@ if you need to use other options during normalization:: class Mailer { // ... - protected function configureOptions(OptionsResolver $resolver) + public function configureOptions(OptionsResolver $resolver) { // ... $resolver->setNormalizer('host', function ($options, $value) { @@ -493,7 +493,7 @@ these options, you can return the desired default value:: class Mailer { // ... - protected function configureOptions(OptionsResolver $resolver) + public function configureOptions(OptionsResolver $resolver) { // ... $resolver->setDefault('encryption', null); @@ -525,7 +525,7 @@ the closure:: class Mailer { // ... - protected function configureOptions(OptionsResolver $resolver) + public function configureOptions(OptionsResolver $resolver) { // ... $resolver->setDefaults(array( @@ -537,7 +537,7 @@ the closure:: class GoogleMailer extends Mailer { - protected function configureOptions(OptionsResolver $resolver) + public function configureOptions(OptionsResolver $resolver) { parent::configureOptions($resolver); @@ -568,7 +568,7 @@ comes from the default:: class Mailer { // ... - protected function configureOptions(OptionsResolver $resolver) + public function configureOptions(OptionsResolver $resolver) { // ... $resolver->setDefault('port', 25); @@ -600,7 +600,7 @@ be included in the resolved options if it was actually passed to { // ... - protected function configureOptions(OptionsResolver $resolver) + public function configureOptions(OptionsResolver $resolver) { // ... $resolver->setDefined('port'); @@ -634,7 +634,7 @@ options in one go:: class Mailer { // ... - protected function configureOptions(OptionsResolver $resolver) + public function configureOptions(OptionsResolver $resolver) { // ... $resolver->setDefined(array('port', 'encryption')); @@ -655,7 +655,7 @@ let you find out which options are defined:: { // ... - protected function configureOptions(OptionsResolver $resolver) + public function configureOptions(OptionsResolver $resolver) { parent::configureOptions($resolver); @@ -701,7 +701,7 @@ can change your code to do the configuration only once per class:: $this->options = self::$resolversByClass[$class]->resolve($options); } - protected function configureOptions(OptionsResolver $resolver) + public function configureOptions(OptionsResolver $resolver) { // ... }