Apply fixes from StyleCI

Author: garak

Command/PromoteUserCommand.php

@@ -2,7 +2,6 @@
 
 namespace Beelab\UserBundle\Command;
 
-use Beelab\UserBundle\Manager\LightUserManagerInterface;
 use Beelab\UserBundle\Manager\UserManagerInterface;
 use Symfony\Component\Console\Input\InputArgument;
 use Symfony\Component\Console\Input\InputInterface;

Controller/AuthController.php

@@ -65,7 +65,7 @@ public function loginCheckAction(): void
      * Get possible authentication error.
      *
      * @param LoggerInterface $logger
-     * @param Request $request
+     * @param Request         $request
      *
      * @return \Exception|null
      */

Controller/UserController.php

@@ -79,7 +79,7 @@ public function newAction(UserManagerInterface $manager, Request $request): Resp
             return $this->redirectToRoute('user_show', ['id' => $user->getId()]);
         }
 
-        return $this->render('BeelabUserBundle:User:new.html.twig',[
+        return $this->render('BeelabUserBundle:User:new.html.twig', [
             'user' => $user,
             'form' => $form->createView(),
         ]);
@@ -102,7 +102,7 @@ public function editAction($id, UserManagerInterface $manager, Request $request)
         }
         $deleteForm = $this->createDeleteForm($user->getId());
 
-        return $this->render('BeelabUserBundle:User:edit.html.twig',[
+        return $this->render('BeelabUserBundle:User:edit.html.twig', [
             'user' => $user,
             'edit_form' => $editForm->createView(),
             'delete_form' => $deleteForm->createView(),

Entity/User.php

@@ -422,7 +422,7 @@ public static function getRoleLabels(): array
      */
     public function getRoleLabel($role): string
     {
-        return $role == 'ROLE_SUPER_ADMIN' ? 'super admin' : static::$roleLabels[$role];
+        return 'ROLE_SUPER_ADMIN' == $role ? 'super admin' : static::$roleLabels[$role];
     }
 
     /**

Manager/UserManagerInterface.php

@@ -8,15 +8,15 @@
 
 interface UserManagerInterface extends LightUserManagerInterface
 {
-   /**
-    * List of users (can be paginated).
-    *
-    * @param int    $page
-    * @param int    $limit
-    * @param string $sortBy
-    *
-    * @return mixed \Knp\Component\Pager\Pagination\PaginationInterface or array
-    */
+    /**
+     * List of users (can be paginated).
+     *
+     * @param int    $page
+     * @param int    $limit
+     * @param string $sortBy
+     *
+     * @return mixed \Knp\Component\Pager\Pagination\PaginationInterface or array
+     */
     public function getList(int $page = 1, int $limit = 20, string $sortBy = 'email');
 
     /**

Repository/UserRepository.php

@@ -56,7 +56,7 @@ public function supportsClass($class): bool
      */
     public function filterByRole(string $role): QueryBuilder
     {
-        $role = $role === 'ROLE_USER' ? '{}' : '"'.$role.'"';
+        $role = 'ROLE_USER' === $role ? '{}' : '"'.$role.'"';
 
         return $this
             ->createQueryBuilder('u')
@@ -69,7 +69,7 @@ public function filterByRoles(array $roles): QueryBuilder
     {
         $qb = $this->createQueryBuilder('u');
         foreach ($roles as $key => $role) {
-            $role = $role === 'ROLE_USER' ? '{}' : '"'.$role.'"';
+            $role = 'ROLE_USER' === $role ? '{}' : '"'.$role.'"';
             $qb->orWhere('u.roles LIKE :role'.$key)->setParameter('role'.$key, '%'.$role.'%');
         }
 

Tests/Controller/UserControllerTest.php

@@ -23,9 +23,13 @@
 class UserControllerTest extends TestCase
 {
     protected $controller;
+
     protected $container;
+
     protected $formBuilder;
+
     protected $userManager;
+
     protected $router;
 
     protected function setUp(): void

Tests/Listener/LastLoginListenerTest.php

@@ -14,6 +14,7 @@
 class LastLoginListenerTest extends TestCase
 {
     protected $listener;
+
     protected $userManager;
 
     protected function setUp(): void

Tests/Manager/LightUserManagerTest.php

@@ -14,8 +14,11 @@
 class LightUserManagerTest extends TestCase
 {
     protected $manager;
+
     protected $em;
+
     protected $repository;
+
     protected $encoder;
 
     protected function setUp(): void

Tests/Manager/UserManagerTest.php

@@ -15,12 +15,19 @@
 class UserManagerTest extends TestCase
 {
     protected $manager;
+
     protected $em;
+
     protected $repository;
+
     protected $encoder;
+
     protected $authChecker;
+
     protected $tokenStorage;
+
     protected $paginator;
+
     protected $dispatcher;
 
     protected function setUp(): void

Tests/Repository/UserRepositoryTest.php

@@ -16,7 +16,9 @@
 class UserRepositoryTest extends TestCase
 {
     protected $repository;
+
     protected $em;
+
     protected $class;
 
     protected function setUp(): void

Twig/BeelabUserTwigExtension.php

@@ -14,7 +14,9 @@
 class BeelabUserTwigExtension extends Twig_Extension implements Twig_Extension_GlobalsInterface
 {
     protected $layout;
+
     protected $route;
+
     protected $hasPaginator = false;
 
     /**