8000 Removed deprecated setDefaultOptions methods · symfony/symfony@6026781 · GitHub
[go: up one dir, main page]
Skip to content

Commit 6026781

Browse files
peterrehmpeterrehm
committed
Removed deprecated setDefaultOptions methods
1 parent db80498 commit 6026781

6 files changed

+13
-56
lines changed

‎src/Symfony/Component/Form/AbstractType.php

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@
1212
namespace Symfony\Component\Form;
1313

1414
use Symfony\Component\OptionsResolver\OptionsResolver;
15-
use Symfony\Component\OptionsResolver\OptionsResolverInterface;
1615

1716
/**
1817
* @author Bernhard Schussek <bschussek@gmail.com>
@@ -43,16 +42,6 @@ public function finishView(FormView $view, FormInterface $form, array $options)
4342
/**
4443
* {@inheritdoc}
4544
*/
46-
public function setDefaultOptions(OptionsResolverInterface $resolver)
47-
{
48-
$this->configureOptions($resolver);
49-
}
50-
51-
/**
52-
* Configures the options for this type.
53-
*
54-
* @param OptionsResolver $resolver The resolver for the options.
55-
*/
5645
public function configureOptions(OptionsResolver $resolver)
5746
{
5847
}

‎src/Symfony/Component/Form/AbstractTypeExtension.php

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,6 @@
1212
namespace Symfony\Component\Form;
1313

1414
use Symfony\Component\OptionsResolver\OptionsResolver;
15-
use Symfony\Component\OptionsResolver\OptionsResolverInterface;
1615

1716
/**
1817
* @author Bernhard Schussek <bschussek@gmail.com>
@@ -43,16 +42,6 @@ public function finishView(FormView $view, FormInterface $form, array $options)
4342
/**
4443
* {@inheritdoc}
4544
*/
46-
public function setDefaultOptions(OptionsResolverInterface $resolver)
47-
{
48-
$this->configureOptions($resolver);
49-
}
50-
51-
/**
52-
* Configures the options for this type.
53-
*
54-
* @param OptionsResolver $resolver The resolver for the options.
55-
*/
5645
public function configureOptions(OptionsResolver $resolver)
5746
{
5847
}

‎src/Symfony/Component/Form/FormTypeExtensionInterface.php

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111

1212
namespace Symfony\Component\Form;
1313

14-
use Symfony\Component\OptionsResolver\OptionsResolverInterface;
14+
use Symfony\Component\OptionsResolver\OptionsResolver;
1515

1616
/**
1717
* @author Bernhard Schussek <bschussek@gmail.com>
@@ -60,15 +60,12 @@ public function buildView(FormView $view, FormInterface $form, array $options);
6060
public function finishView(FormView $view, FormInterface $form, array $options);
6161

6262
/**
63-
* Overrides the default options from the extended type.
63+
* Configures the options for this type.
6464
*
65-
* @param OptionsResolverInterface $resolver The resolver for the options.
65+
* @param OptionsResolver $resolver The resolver for the options.
6666
*
67-
* @deprecated Deprecated since Symfony 2.7, to be removed in Symfony 3.0.
68-
* Use the method configureOptions instead. This method will be
69-
* added to the FormTypeExtensionInterface with Symfony 3.0
7067
*/
71-
public function setDefaultOptions(OptionsResolverInterface $resolver);
68+
public function configureOptions(OptionsResolver $resolver);
7269

7370
/**
7471
* Returns the name of the type being extended.

‎src/Symfony/Component/Form/FormTypeInterface.php

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111

1212
namespace Symfony\Component\Form;
1313

14-
use Symfony\Component\OptionsResolver\OptionsResolverInterface;
14+
use Symfony\Component\OptionsResolver\OptionsResolver;
1515

1616
/**
1717
* @author Bernhard Schussek <bschussek@gmail.com>
@@ -69,15 +69,11 @@ public function buildView(FormView $view, FormInterface $form, array $options);
6969
public function finishView(FormView $view, FormInterface $form, array $options);
7070

7171
/**
72-
* Sets the default options for this type.
72+
* Configures the options for this type.
7373
*
74-
* @param OptionsResolverInterface $resolver The resolver for the options.
75-
*
76-
* @deprecated Deprecated since Symfony 2.7, to be renamed in Symfony 3.0.
77-
* Use the method configureOptions instead. This method will be
78-
* added to the FormTypeInterface with Symfony 3.0.
74+
* @param OptionsResolver $resolver The resolver for the options.
7975
*/
80-
public function setDefaultOptions(OptionsResolverInterface $resolver);
76+
public function configureOptions(OptionsResolver $resolver);
8177

8278
/**
8379
* Returns the name of the parent type.

‎src/Symfony/Component/Form/ResolvedFormType.php

Lines changed: 3 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -192,7 +192,7 @@ public function finishView(FormView $view, FormInterface $form, array $options)
192192
/**
193193
* Returns the configured options resolver used for this type.
194194
*
195-
* @return \Symfony\Component\OptionsResolver\OptionsResolverInterface The options resolver.
195+
* @return \Symfony\Component\OptionsResolver\OptionsResolver The options resolver.
196196
*/
197197
public function getOptionsResolver()
198198
{
@@ -203,24 +203,10 @@ public function getOptionsResolver()
203203
$this->optionsResolver = new OptionsResolver();
204204
}
205205

206-
$this->innerType->setDefaultOptions($this->optionsResolver);
207-
208-
$reflector = new \ReflectionMethod($this->innerType, 'setDefaultOptions');
209-
$isOverwritten = ($reflector->getDeclaringClass()->getName() !== 'Symfony\Component\Form\AbstractType');
210-
211-
if (true === $isOverwritten) {
212-
trigger_error('The FormTypeInterface::setDefaultOptions() method is deprecated since version 2.7 and will be removed in 3.0. Use configureOptions() instead. This method will be added to the FormTypeInterface with Symfony 3.0.', E_USER_DEPRECATED);
213-
}
206+
$this->innerType->configureOptions($this->optionsResolver);
214207

215208
foreach ($this->typeExtensions as $extension) {
216-
$extension->setDefaultOptions($this->optionsResolver);
217-
218-
$reflector = new \ReflectionMethod($extension, 'setDefaultOptions');
219-
$isOverwritten = ($reflector->getDeclaringClass()->getName() !== 'Symfony\Component\Form\AbstractTypeExtension');
220-
221-
if (true === $isOverwritten) {
222-
trigger_error('The FormTypeExtensionInterface::setDefaultOptions() method is deprecated since version 2.7 and will be removed in 3.0. Use configureOptions() instead. This method will be added to the FormTypeExtensionInterface with Symfony 3.0.', E_USER_DEPRECATED);
223-
}
209+
$extension->configureOptions($this->optionsResolver);
224210
}
225211
}
226212

‎src/Symfony/Component/Form/Tests/ResolvedFormTypeTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ public function testCreateBuilder()
9999
{
100100
$givenOptions = array('a' => 'a_custom', 'c' => 'c_custom');
101101
$resolvedOptions = array('a' => 'a_custom', 'b' => 'b_default', 'c' => 'c_custom', 'd' => 'd_default');
102-
$optionsResolver = $this->getMock('Symfony\Component\OptionsResolver\OptionsResolverInterface');
102+
$optionsResolver = $this->getMock('Symfony\Component\OptionsResolver\OptionsResolver');
103103

104104
$this->resolvedType = $this->getMockBuilder('Symfony\Component\Form\ResolvedFormType')
105105
->setConstructorArgs(array($this->type, array($this->extension1, $this->extension2), $this->parentResolvedType))
@@ -127,7 +127,7 @@ public function testCreateBuilderWithDataClassOption()
127127
{
128128
$givenOptions = array('data_class' => 'Foo');
129129
$resolvedOptions = array('data_class' => '\stdClass');
130-
$optionsResolver = $this->getMock('Symfony\Component\OptionsResolver\OptionsResolverInterface');
130+
$optionsResolver = $this->getMock('Symfony\Component\OptionsResolver\OptionsResolver');
131131

132132
$this->resolvedType = $this->getMockBuilder('Symfony\Component\Form\ResolvedFormType')
133133
->setConstructorArgs(array($this->type, array($this->extension1, $this->extension2), $this->parentResolvedType))

0 commit comments

Comments
 (0)
0