configureOptions(...) : protected => public

Lucas CHERIFI · Lucas CHERIFI · commit 824fe40715f0 · 2015-10-02T23:57:45.000+02:00
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) { }
diff --git 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)
         {
             // ...
         }

Original file line number	Diff line number	Diff line change
`@@ -173,7 +173,7 @@ It's a good practice to split the option configuration into a separate method::`
`173`	`173`	`$this->options = $resolver->resolve($options);`
`174`	`174`	`}`
`175`	`175`
`176`		`- protected function configureOptions(OptionsResolver $resolver)`
	`176`	`+ public function configureOptions(OptionsResolver $resolver)`
`177`	`177`	`{`
`178`	`178`	`$resolver->setDefaults(array(`
`179`	`179`	`'host' => 'smtp.example.org',`
`@@ -192,7 +192,7 @@ than processing options. Second, sub-classes may now override the`
`192`	`192`	`// ...`
`193`	`193`	`class GoogleMailer extends Mailer`
`194`	`194`	`{`
`195`		`- protected function configureOptions(OptionsResolver $resolver)`
	`195`	`+ public function configureOptions(OptionsResolver $resolver)`
`196`	`196`	`{`
`197`	`197`	`parent::configureOptions($resolver);`
`198`	`198`
@@ -215,7 +215,7 @@ For example, to make the ``host`` option required, you can do::
`215`	`215`	`{`
`216`	`216`	`// ...`
`217`	`217`
`218`		`- protected function configureOptions(OptionsResolver $resolver)`
	`218`	`+ public function configureOptions(OptionsResolver $resolver)`
`219`	`219`	`{`
`220`	`220`	`// ...`
`221`	`221`	`$resolver->setRequired('host');`
`@@ -243,7 +243,7 @@ one required option::`
`243`	`243`	`{`
`244`	`244`	`// ...`
`245`	`245`
`246`		`- protected function configureOptions(OptionsResolver $resolver)`
	`246`	`+ public function configureOptions(OptionsResolver $resolver)`
`247`	`247`	`{`
`248`	`248`	`// ...`
`249`	`249`	`$resolver->setRequired(array('host', 'username', 'password'));`
`@@ -263,7 +263,7 @@ retrieve the names of all required options::`
`263`	`263`	`// ...`
`264`	`264`	`class GoogleMailer extends Mailer`
`265`	`265`	`{`
`266`		`- protected function configureOptions(OptionsResolver $resolver)`
	`266`	`+ public function configureOptions(OptionsResolver $resolver)`
`267`	`267`	`{`
`268`	`268`	`parent::configureOptions($resolver);`
`269`	`269`
`@@ -291,7 +291,7 @@ been set::`
`291`	`291`	`{`
`292`	`292`	`// ...`
`293`	`293`
`294`		`- protected function configureOptions(OptionsResolver $resolver)`
	`294`	`+ public function configureOptions(OptionsResolver $resolver)`
`295`	`295`	`{`
`296`	`296`	`// ...`
`297`	`297`	`$resolver->setRequired('host');`
`@@ -301,7 +301,7 @@ been set::`
`301`	`301`	`// ...`
`302`	`302`	`class GoogleMailer extends Mailer`
`303`	`303`	`{`
`304`		`- protected function configureOptions(OptionsResolver $resolver)`
	`304`	`+ public function configureOptions(OptionsResolver $resolver)`
`305`	`305`	`{`
`306`	`306`	`parent::configureOptions($resolver);`
`307`	`307`
`@@ -336,7 +336,7 @@ correctly. To validate the types of the options, call`
`336`	`336`	`{`
`337`	`337`	`// ...`
`338`	`338`
`339`		`- protected function configureOptions(OptionsResolver $resolver)`
	`339`	`+ public function configureOptions(OptionsResolver $resolver)`
`340`	`340`	`{`
`341`	`341`	`// ...`
`342`	`342`	`$resolver->setAllowedTypes('host', 'string');`
`@@ -381,7 +381,7 @@ to verify that the passed option contains one of these values::`
`381`	`381`	`{`
`382`	`382`	`// ...`
`383`	`383`
`384`		`- protected function configureOptions(OptionsResolver $resolver)`
	`384`	`+ public function configureOptions(OptionsResolver $resolver)`
`385`	`385`	`{`
`386`	`386`	`// ...`
`387`	`387`	`$resolver->setDefault('transport', 'sendmail');`
`@@ -432,7 +432,7 @@ option. You can configure a normalizer by calling`
`432`	`432`	`{`
`433`	`433`	`// ...`
`434`	`434`
`435`		`- protected function configureOptions(OptionsResolver $resolver)`
	`435`	`+ public function configureOptions(OptionsResolver $resolver)`
`436`	`436`	`{`
`437`	`437`	`// ...`
`438`	`438`
`@@ -459,7 +459,7 @@ if you need to use other options during normalization::`
`459`	`459`	`class Mailer`
`460`	`460`	`{`
`461`	`461`	`// ...`
`462`		`- protected function configureOptions(OptionsResolver $resolver)`
	`462`	`+ public function configureOptions(OptionsResolver $resolver)`
`463`	`463`	`{`
`464`	`464`	`// ...`
`465`	`465`	`$resolver->setNormalizer('host', function ($options, $value) {`
`@@ -493,7 +493,7 @@ these options, you can return the desired default value::`
`493`	`493`	`class Mailer`
`494`	`494`	`{`
`495`	`495`	`// ...`
`496`		`- protected function configureOptions(OptionsResolver $resolver)`
	`496`	`+ public function configureOptions(OptionsResolver $resolver)`
`497`	`497`	`{`
`498`	`498`	`// ...`
`499`	`499`	`$resolver->setDefault('encryption', null);`
`@@ -525,7 +525,7 @@ the closure::`
`525`	`525`	`class Mailer`
`526`	`526`	`{`
`527`	`527`	`// ...`
`528`		`- protected function configureOptions(OptionsResolver $resolver)`
	`528`	`+ public function configureOptions(OptionsResolver $resolver)`
`529`	`529`	`{`
`530`	`530`	`// ...`
`531`	`531`	`$resolver->setDefaults(array(`
`@@ -537,7 +537,7 @@ the closure::`
`537`	`537`
`538`	`538`	`class GoogleMailer extends Mailer`
`539`	`539`	`{`
`540`		`- protected function configureOptions(OptionsResolver $resolver)`
	`540`	`+ public function configureOptions(OptionsResolver $resolver)`
`541`	`541`	`{`
`542`	`542`	`parent::configureOptions($resolver);`
`543`	`543`
`@@ -568,7 +568,7 @@ comes from the default::`
`568`	`568`	`class Mailer`
`569`	`569`	`{`
`570`	`570`	`// ...`
`571`		`- protected function configureOptions(OptionsResolver $resolver)`
	`571`	`+ public function configureOptions(OptionsResolver $resolver)`
`572`	`572`	`{`
`573`	`573`	`// ...`
`574`	`574`	`$resolver->setDefault('port', 25);`
`@@ -600,7 +600,7 @@ be included in the resolved options if it was actually passed to`
`600`	`600`	`{`
`601`	`601`	`// ...`
`602`	`602`
`603`		`- protected function configureOptions(OptionsResolver $resolver)`
	`603`	`+ public function configureOptions(OptionsResolver $resolver)`
`604`	`604`	`{`
`605`	`605`	`// ...`
`606`	`606`	`$resolver->setDefined('port');`
`@@ -634,7 +634,7 @@ options in one go::`
`634`	`634`	`class Mailer`
`635`	`635`	`{`
`636`	`636`	`// ...`
`637`		`- protected function configureOptions(OptionsResolver $resolver)`
	`637`	`+ public function configureOptions(OptionsResolver $resolver)`
`638`	`638`	`{`
`639`	`639`	`// ...`
`640`	`640`	`$resolver->setDefined(array('port', 'encryption'));`
`@@ -655,7 +655,7 @@ let you find out which options are defined::`
`655`	`655`	`{`
`656`	`656`	`// ...`
`657`	`657`
`658`		`- protected function configureOptions(OptionsResolver $resolver)`
	`658`	`+ public function configureOptions(OptionsResolver $resolver)`
`659`	`659`	`{`
`660`	`660`	`parent::configureOptions($resolver);`
`661`	`661`
`@@ -701,7 +701,7 @@ can change your code to do the configuration only once per class::`
`701`	`701`	`$this->options = self::$resolversByClass[$class]->resolve($options);`
`702`	`702`	`}`
`703`	`703`
`704`		`- protected function configureOptions(OptionsResolver $resolver)`
	`704`	`+ public function configureOptions(OptionsResolver $resolver)`
`705`	`705`	`{`
`706`	`706`	`// ...`
`707`	`707`	`}`