Skip to content

Commit d6cbe3d

Browse files
committed
Merge pull request #1365 from jbenezech/1361_empty_serializer_group
Fixes 1361 - Define the serializer groups parameter as a collection
2 parents 783b5ea + fdcb489 commit d6cbe3d

File tree

2 files changed

+8
-1
lines changed

2 files changed

+8
-1
lines changed

Resources/config/view.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66

77
<parameters>
88
<parameter key="fos_rest.serializer.exclusion_strategy.version" />
9-
<parameter key="fos_rest.serializer.exclusion_strategy.groups"/>
9+
<parameter key="fos_rest.serializer.exclusion_strategy.groups" type="collection"/>
1010
<parameter key="fos_rest.view_handler.jsonp.callback_param"/>
1111
<parameter key="fos_rest.view.exception_wrapper_handler">FOS\RestBundle\View\ExceptionWrapperHandler</parameter>
1212
<parameter key="fos_rest.view_handler.default.class">FOS\RestBundle\View\ViewHandler</parameter>

Tests/DependencyInjection/FOSRestExtensionTest.php

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -334,6 +334,13 @@ public function testValidatorAliasWhenDisabled()
334334
$this->assertFalse($this->container->has('fos_rest.validator'));
335335
}
336336

337+
public function testDefaultSerializerGroupsIsArray()
338+
{
339+
$config = array();
340+
$this->extension->load($config, $this->container);
341+
$this->assertTrue(is_array($this->container->getParameter('fos_rest.serializer.exclusion_strategy.groups')));
342+
}
343+
337344
/**
338345
* Test that extension loads properly.
339346
*/

0 commit comments

Comments
 (0)