Use multiple options by default for RolesMatrixType form (#1661)

sonata-project · Oct 1, 2023 · 4314383 · 4314383
1 parent f97f681
commit 4314383
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/src/Form/Type/RolesMatrixType.php b/src/Form/Type/RolesMatrixType.php
@@ -35,6 +35,7 @@ public function configureOptions(OptionsResolver $resolver): void
  {
  $resolver->setDefaults([
  'expanded' => true,
+ 'multiple' => true,
  'choices' => function (Options $options, ?array $parentChoices): array {
  if (null !== $parentChoices && [] !== $parentChoices) {
  return [];

diff --git a/tests/Form/Type/RolesMatrixTypeTest.php b/tests/Form/Type/RolesMatrixTypeTest.php
@@ -58,6 +58,8 @@ public function testGetDefaultOptions(): void
  $choices = $options['choices'];
  static::assertCount(2, $choices);
  static::assertNotContains(UserInterface::ROLE_DEFAULT, $choices);
+ static::assertTrue($options['expanded']);
+ static::assertTrue($options['multiple']);
  }
 
  public function testDifferentExcludedRoles(): void