xchopin / LAWA-UL

public/index.php

Spacing +2 added lines, -2 removed lines

@@ -5,7 +5,7 @@ 
 use Symfony\Component\Dotenv\Dotenv;
 use Symfony\Component\HttpFoundation\Request;
 
-require __DIR__.'/../vendor/autoload.php';
+require __DIR__ . '/../vendor/autoload.php';
 
 session_start();
 
@@ -14,7 +14,7 @@ 
     if (!class_exists(Dotenv::class)) {
         throw new \RuntimeException('APP_ENV environment variable is not defined. You need to define environment variables for configuration or add "symfony/dotenv" as a Composer dependency to load variables from a .env file.');
     }
-    (new Dotenv())->load(__DIR__.'/../.env');
+    (new Dotenv())->load(__DIR__ . '/../.env');
 }
 
 if ($_SERVER['APP_DEBUG'] ?? ('prod' !== ($_SERVER['APP_ENV'] ?? 'dev'))) {

src/Kernel.php

Spacing +15 added lines, -15 removed lines

@@ -16,17 +16,17 @@ 
 
     public function getCacheDir()
     {
-        return $this->getProjectDir().'/var/cache/'.$this->environment;
+        return $this->getProjectDir() . '/var/cache/' . $this->environment;
     }
 
     public function getLogDir()
     {
-        return $this->getProjectDir().'/var/log';
+        return $this->getProjectDir() . '/var/log';
     }
 
     public function registerBundles()
     {
-        $contents = require $this->getProjectDir().'/config/bundles.php';
+        $contents = require $this->getProjectDir() . '/config/bundles.php';
         foreach ($contents as $class => $envs) {
             if (isset($envs['all']) || isset($envs[$this->environment])) {
                 yield new $class();
@@ -38,24 +38,24 @@ 
     {
         $container->setParameter('container.autowiring.strict_mode', true);
         $container->setParameter('container.dumper.inline_class_loader', true);
-        $confDir = $this->getProjectDir().'/config';
-        $loader->load($confDir.'/packages/*'.self::CONFIG_EXTS, 'glob');
-        if (is_dir($confDir.'/packages/'.$this->environment)) {
-            $loader->load($confDir.'/packages/'.$this->environment.'/**/*'.self::CONFIG_EXTS, 'glob');
+        $confDir = $this->getProjectDir() . '/config';
+        $loader->load($confDir . '/packages/*' . self::CONFIG_EXTS, 'glob');
+        if (is_dir($confDir . '/packages/' . $this->environment)) {
+            $loader->load($confDir . '/packages/' . $this->environment . '/**/*' . self::CONFIG_EXTS, 'glob');
         }
-        $loader->load($confDir.'/services'.self::CONFIG_EXTS, 'glob');
-        $loader->load($confDir.'/services_'.$this->environment.self::CONFIG_EXTS, 'glob');
+        $loader->load($confDir . '/services' . self::CONFIG_EXTS, 'glob');
+        $loader->load($confDir . '/services_' . $this->environment . self::CONFIG_EXTS, 'glob');
     }
 
     protected function configureRoutes(RouteCollectionBuilder $routes)
     {
-        $confDir = $this->getProjectDir().'/config';
-        if (is_dir($confDir.'/routes/')) {
-            $routes->import($confDir.'/routes/*'.self::CONFIG_EXTS, '/', 'glob');
+        $confDir = $this->getProjectDir() . '/config';
+        if (is_dir($confDir . '/routes/')) {
+            $routes->import($confDir . '/routes/*' . self::CONFIG_EXTS, '/', 'glob');
         }
-        if (is_dir($confDir.'/routes/'.$this->environment)) {
-            $routes->import($confDir.'/routes/'.$this->environment.'/**/*'.self::CONFIG_EXTS, '/', 'glob');
+        if (is_dir($confDir . '/routes/' . $this->environment)) {
+            $routes->import($confDir . '/routes/' . $this->environment . '/**/*' . self::CONFIG_EXTS, '/', 'glob');
         }
-        $routes->import($confDir.'/routes'.self::CONFIG_EXTS, '/', 'glob');
+        $routes->import($confDir . '/routes' . self::CONFIG_EXTS, '/', 'glob');
     }
 }

src/TwigExtension/TranslatorExtension.php

Indentation +4 added lines, -4 removed lines

@@ -83,10 +83,10 @@
      */
     public function languagesAvailable()
     {
-       $languages = [];
-       foreach ($this->container->getParameter('dictionaries') as $languageId => $dictionary) {
-           $languages += [ $dictionary['self_name'] => $languageId ];
-       }
+        $languages = [];
+        foreach ($this->container->getParameter('dictionaries') as $languageId => $dictionary) {
+            $languages += [ $dictionary['self_name'] => $languageId ];
+        }
         return $languages;
     }
 }

Spacing +1 added lines, -1 removed lines

@@ -85,7 +85,7 @@
     {
        $languages = [];
        foreach ($this->container->getParameter('dictionaries') as $languageId => $dictionary) {
-           $languages += [ $dictionary['self_name'] => $languageId ];
+           $languages += [$dictionary['self_name'] => $languageId];
        }
         return $languages;
     }

src/Event/AdminSubscriber.php

Spacing +1 added lines, -1 removed lines

@@ -43,6 +43,6 @@
 
     public static function getSubscribedEvents()
     {
-        return [ KernelEvents::CONTROLLER => 'onKernelController' ];
+        return [KernelEvents::CONTROLLER => 'onKernelController'];
     }
 }
\ No newline at end of file

Braces +9 added lines, -5 removed lines

@@ -36,17 +36,21 @@
         if ($controller[0] instanceof AdminAuthenticatedController) {
             $isAdmin = false;
             if (isset($_SESSION['phpCAS']['user'])) {
-                if (in_array($_SESSION['phpCAS']['user'], $this->administrators))
-                    $isAdmin = true;
+                if (in_array($_SESSION['phpCAS']['user'], $this->administrators)) {
+                                    $isAdmin = true;
+                }
+            }
+            if (!$isAdmin) {
+                throw new AccessDeniedHttpException('Access forbidden.');
             }
-            if (!$isAdmin) throw new AccessDeniedHttpException('Access forbidden.');
         }
     }
 
     public function onKernelRequest(GetResponseEvent $event)
     {
-        if (HttpKernel::MASTER_REQUEST != $event->getRequestType())
-            return;
+        if (HttpKernel::MASTER_REQUEST != $event->getRequestType()) {
+                    return;
+        }
     }
 
 

src/Event/LocaleListener.php

Spacing +1 added lines, -1 removed lines

@@ -58,6 +58,6 @@
 
     public static function getSubscribedEvents()
     {
-        return [ KernelEvents::REQUEST => [['onKernelRequest', 200]] ];
+        return [KernelEvents::REQUEST => [['onKernelRequest', 200]]];
     }
 }
\ No newline at end of file

Braces +3 added lines, -2 removed lines

@@ -33,8 +33,9 @@
      */
     public function onKernelRequest(GetResponseEvent $event)
     {
-        if (HttpKernel::MASTER_REQUEST != $event->getRequestType())
-            return;
+        if (HttpKernel::MASTER_REQUEST != $event->getRequestType()) {
+                    return;
+        }
 
         $this->registerLDAP();
         $this->registerCapsule();

src/Controller/AdminController.php

Spacing +3 added lines, -3 removed lines

@@ -37,9 +37,9 @@ 
             $students = $this->ldap($filter, ['displayname', 'uid']);
             $res = [];
 
-            foreach($students as $student) {
+            foreach ($students as $student) {
                 if ($student['uid'][0] != null)
-                    $res += [ $student['uid'][0] => $student['displayname'][0] ];
+                    $res += [$student['uid'][0] => $student['displayname'][0]];
             }
 
             return $this->json($res);
@@ -77,7 +77,7 @@ 
                 array_push($classes, $enrollment->class);
         }
 
-        return $this->render('admin/user.twig',[
+        return $this->render('admin/user.twig', [
             'classes' => $classes,
             'student_name' => $this->ldapFirst("uid=$id")['displayname'][0]
         ]);

Braces +6 added lines, -4 removed lines

@@ -38,8 +38,9 @@ 
             $res = [];
 
             foreach($students as $student) {
-                if ($student['uid'][0] != null)
-                    $res += [ $student['uid'][0] => $student['displayname'][0] ];
+                if ($student['uid'][0] != null) {
+                                    $res += [ $student['uid'][0] => $student['displayname'][0] ];
+                }
             }
 
             return $this->json($res);
@@ -73,8 +74,9 @@ 
         ]);
 
         foreach (json_decode($enrollments->getBody()->getContents()) as $enrollment) {
-            if ($enrollment->class->title != null)
-                array_push($classes, $enrollment->class);
+            if ($enrollment->class->title != null) {
+                            array_push($classes, $enrollment->class);
+            }
         }
 
         return $this->render('admin/user.twig',[

src/Controller/AbstractController.php

Indentation +2 added lines, -2 removed lines

@@ -6,7 +6,7 @@ 
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;
 use Symfony\Component\DependencyInjection\ContainerInterface;
 
- abstract class AbstractController extends Controller
+    abstract class AbstractController extends Controller
 {
 
     /**
@@ -39,7 +39,7 @@ 
                 'password' => env('API_PASSWORD')
             ]
         ])->getBody()
-          ->getContents());
+            ->getContents());
     }
 
     /**

Spacing +2 added lines, -2 removed lines

@@ -32,8 +32,8 @@
      */
     protected function createJWT()
     {
-        return json_decode( $this->http->request('POST', 'auth/login', [
-            'headers' => [ 'X-Requested-With' => 'XMLHttpRequest' ],
+        return json_decode($this->http->request('POST', 'auth/login', [
+            'headers' => ['X-Requested-With' => 'XMLHttpRequest'],
             'json' => [
                 'username' => env('API_USERNAME'),
                 'password' => env('API_PASSWORD')