csfixes

src/Bundle/ChillMainBundle/Entity/User.php

@@ -112,6 +112,11 @@ class User implements AdvancedUserInterface
      */
     private string $password = '';
 
+    /**
+     * @ORM\Column(type="json")
+     */
+    private array $roles = ['ROLE_USER'];
+
     /**
      * @internal must be set to null if we use bcrypt
      *
@@ -138,11 +143,6 @@ class User implements AdvancedUserInterface
      */
     private ?string $usernameCanonical = null;
 
-    /**
-     * @ORM\Column(type="json")
-     */
-    private array $roles = ['ROLE_USER'];
-
     /**
      * User constructor.
      */
@@ -156,6 +156,13 @@ class User implements AdvancedUserInterface
         return $this->getLabel();
     }
 
+    public function addGroupCenter(GroupCenter $groupCenter): self
+    {
+        $this->groupCenters->add($groupCenter);
+
+        return $this;
+    }
+
     // empty function... remove?
     public function eraseCredentials()
     {
@@ -300,13 +307,6 @@ class User implements AdvancedUserInterface
         }
     }
 
-    public function addGroupCenter(GroupCenter $groupCenter): self
-    {
-        $this->groupCenters->add($groupCenter);
-
-        return $this;
-    }
-
     /**
      * Set attributes.
      *
@@ -328,12 +328,6 @@ class User implements AdvancedUserInterface
         return $this;
     }
 
-    public function setRoles($roles): self
-    {
-        $this->roles = $roles;
-        return $this;
-    }
-
     public function setEmail($email): self
     {
         $this->email = $email;
@@ -390,6 +384,13 @@ class User implements AdvancedUserInterface
         return $this;
     }
 
+    public function setRoles($roles): self
+    {
+        $this->roles = $roles;
+
+        return $this;
+    }
+
     public function setSalt($salt): self
     {
         $this->salt = $salt;

src/Bundle/ChillMainBundle/Form/UserType.php

@@ -65,7 +65,7 @@ class UserType extends AbstractType
                 'multiple' => true,
                 'expanded' => true,
                 'label' => 'Roles',
-                'choices'  => [
+                'choices' => [
                     'Usager' => 'ROLE_USER',
                     'Administrateur' => 'ROLE_ADMIN',
                 ],

src/Bundle/ChillMainBundle/Routing/MenuBuilder/SectionMenuBuilder.php

@@ -64,7 +64,6 @@ class SectionMenuBuilder implements LocalMenuBuilderInterface
                 ]
             );
 
-
         if ($this->authorizationChecker->isGranted(ChillExportVoter::EXPORT)) {
             $menu->addChild($this->translator->trans('Export Menu'), [
                 'route' => 'chill_main_export_index',

src/Bundle/ChillMainBundle/migrations/Version20220308104030.php

@@ -1,5 +1,12 @@
 <?php
 
+/**
+ * Chill is a software for social workers
+ *
+ * For the full copyright and license information, please view
+ * the LICENSE file that was distributed with this source code.
+ */
+
 declare(strict_types=1);
 
 namespace Chill\Migrations\Main;
@@ -12,6 +19,11 @@ use Doctrine\Migrations\AbstractMigration;
  */
 final class Version20220308104030 extends AbstractMigration
 {
+    public function down(Schema $schema): void
+    {
+        $this->addSql('ALTER TABLE users DROP roles');
+    }
+
     public function getDescription(): string
     {
         return 'Add roles property to user';
@@ -21,9 +33,4 @@ final class Version20220308104030 extends AbstractMigration
     {
         $this->addSql('ALTER TABLE users ADD roles JSONB DEFAULT \'["ROLE_USER"]\' NOT NULL');
     }
-
-    public function down(Schema $schema): void
-    {
-        $this->addSql('ALTER TABLE users DROP roles');
-    }
 }