remove default role from user filter and role select (#993)

src/Form/Type/UserRoleType.php

@@ -9,9 +9,11 @@
 
 namespace App\Form\Type;
 
+use App\Entity\User;
 use App\Security\RoleService;
 use Symfony\Component\Form\AbstractType;
 use Symfony\Component\Form\Extension\Core\Type\ChoiceType;
+use Symfony\Component\OptionsResolver\Options;
 use Symfony\Component\OptionsResolver\OptionsResolver;
 
 /**
@@ -22,7 +24,7 @@ class UserRoleType extends AbstractType
     /**
      * @var RoleService
      */
-    protected $roles;
+    private $roles;
 
     public function __construct(RoleService $roles)
     {
@@ -34,15 +36,23 @@ class UserRoleType extends AbstractType
      */
     public function configureOptions(OptionsResolver $resolver)
     {
+        $resolver->setDefaults([
+            'label' => 'label.roles',
+            'include_default' => false,
+        ]);
+
+        $resolver->setDefault('choices', function (Options $options) {
             $roles = [];
             foreach ($this->roles->getAvailableNames() as $name) {
                 $roles[$name] = $name;
             }
 
-        $resolver->setDefaults([
+            if ($options['include_default'] !== true && isset($roles[User::DEFAULT_ROLE])) {
-            'label' => 'label.roles',
+                unset($roles[User::DEFAULT_ROLE]);
-            'choices' => $roles,
+            }
-        ]);
+
+            return $roles;
+        });
     }
 
     /**

src/Form/UserRolesType.php

@@ -26,10 +26,10 @@ class UserRolesType extends AbstractType
     public function buildForm(FormBuilderInterface $builder, array $options)
     {
         $builder
-            // string[]
             ->add('roles', UserRoleType::class, [
                 'label' => 'label.roles',
                 'multiple' => true,
+                'expanded' => true,
             ])
         ;
     }

tests/Controller/ProfileControllerTest.php

@@ -298,8 +298,8 @@ class ProfileControllerTest extends ControllerBaseTest
         $form = $client->getCrawler()->filter('form[name=user_roles]')->form();
         $client->submit($form, [
             'user_roles[roles]' => [
-                'ROLE_TEAMLEAD',
+                0 => 'ROLE_TEAMLEAD',
-                'ROLE_SUPER_ADMIN',
+                2 => 'ROLE_SUPER_ADMIN',
             ]
         ]);