Skip to content

Commit 5de9475

Browse files
committed
[2.7] setDefaultOptions is renamed to configureOptions
1 parent d1fdc64 commit 5de9475

File tree

2 files changed

+19
-6
lines changed

2 files changed

+19
-6
lines changed

Form/Type/RouteTypeType.php

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,14 +12,18 @@
1212
namespace Symfony\Cmf\Bundle\RoutingBundle\Form\Type;
1313

1414
use Symfony\Component\Form\AbstractType;
15+
use Symfony\Component\OptionsResolver\OptionsResolver;
1516
use Symfony\Component\OptionsResolver\OptionsResolverInterface;
1617

1718
class RouteTypeType extends AbstractType
1819
{
1920
protected $routeTypes = array();
2021
protected $translator;
2122

22-
public function setDefaultOptions(OptionsResolverInterface $resolver)
23+
/**
24+
* {@inheritdoc}
25+
*/
26+
public function configureOptions(OptionsResolver $resolver)
2327
{
2428
$choices = array();
2529
foreach ($this->routeTypes as $routeType) {
@@ -32,6 +36,16 @@ public function setDefaultOptions(OptionsResolverInterface $resolver)
3236
));
3337
}
3438

39+
/**
40+
* {@inheritdoc}
41+
*
42+
* @todo Remove when Symfony <2.7 support is dropped.
43+
*/
44+
public function setDefaultOptions(OptionsResolverInterface $resolver)
45+
{
46+
$this->configureOptions($resolver);
47+
}
48+
3549
/**
3650
* Register a route type
3751
*

Tests/Unit/Form/Type/RouteTypeTypeTest.php

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -19,16 +19,14 @@ class RouteTypeTypeTest extends \PHPUnit_Framework_Testcase
1919
public function setUp()
2020
{
2121
$this->type = new RouteTypeType;
22-
$this->ori = $this->getMock(
23-
'Symfony\Component\OptionsResolver\OptionsResolverInterface');
2422
}
2523

2624
public function testSetDefaultOptions()
2725
{
2826
$type = new RouteTypeType;
2927
$optionsResolver = new OptionsResolver();
3028

31-
$type->setDefaultOptions($optionsResolver);
29+
$type->configureOptions($optionsResolver);
3230

3331
$options = $optionsResolver->resolve();
3432

@@ -40,7 +38,8 @@ public function testDefaultsSet()
4038
$this->type->addRouteType('foobar');
4139
$this->type->addRouteType('barfoo');
4240

43-
$this->ori->expects($this->once())
41+
$optionsResolver = $this->getMock('Symfony\Component\OptionsResolver\OptionsResolver');
42+
$optionsResolver->expects($this->once())
4443
->method('setDefaults')
4544
->with(array(
4645
'choices' => array(
@@ -50,6 +49,6 @@ public function testDefaultsSet()
5049
'translation_domain' => 'CmfRoutingBundle',
5150
));
5251

53-
$this->type->setDefaultOptions($this->ori);
52+
$this->type->configureOptions($optionsResolver);
5453
}
5554
}

0 commit comments

Comments
 (0)