TechWilk / rota

src/classes/Site.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
         $hostname = $_SERVER['HTTP_HOST'];
 
         return [
-            'base'     => $http.'://'.$hostname,
+            'base'     => $http . '://' . $hostname,
             'protocol' => $http,
             'host'     => $hostname,
         ];

src/classes/AuthProvider/UsernamePassword/OneBodyAuth.php

Spacing +1 added lines, -1 removed lines

@@ -71,7 +71,7 @@
         $base_uri = $this->guzzle->getConfig()['base_uri'];
 
         if (strlen($base_uri) > 0) {
-            $url = $base_uri.'/account/new?forgot=true';
+            $url = $base_uri . '/account/new?forgot=true';
         }
 
         return $url;

src/classes/Crypt.php

Spacing +1 added lines, -1 removed lines

@@ -32,7 +32,7 @@
     public static function generateInt($min, $max)
     {
         if ($min >= $max) {
-            throw new InvalidArgumentException('Invalid constraints - minimum ('.$min.') is above the maximum ('.$max.')');
+            throw new InvalidArgumentException('Invalid constraints - minimum (' . $min . ') is above the maximum (' . $max . ')');
         }
 
         if (function_exists('random_int')) { // use secure random numbers when available

src/classes/Controller/AvailabilityController.php

Spacing +4 added lines, -4 removed lines

@@ -14,7 +14,7 @@ 
 {
     public function getAvailabilityForm(ServerRequestInterface $request, ResponseInterface $response, $args)
     {
-        $this->logger->info("Fetch user availability GET '/user/".$args['id']."/availability'");
+        $this->logger->info("Fetch user availability GET '/user/" . $args['id'] . "/availability'");
 
         $u = UserQuery::create()->findPk($args['id']);
 
@@ -33,7 +33,7 @@ 
 
     public function postAvailability(ServerRequestInterface $request, ResponseInterface $response, $args)
     {
-        $this->logger->info("Update user availability POST '/user/".$args['id']."/availability'");
+        $this->logger->info("Update user availability POST '/user/" . $args['id'] . "/availability'");
 
         $u = UserQuery::create()
             ->findPk($args['id']);
@@ -62,8 +62,8 @@ 
                 $a->setAvailable(false);
             }
 
-            if (strlen($data['event'.$eventId.'comment']) > 0) {
-                $a->setComment($data['event'.$eventId.'comment']);
+            if (strlen($data['event' . $eventId . 'comment']) > 0) {
+                $a->setComment($data['event' . $eventId . 'comment']);
             } else {
                 $a->setComment(null);
             }

src/classes/Controller/RoleController.php

Spacing +2 added lines, -2 removed lines

@@ -13,7 +13,7 @@ 
 {
     public function getAssignRolesForm(ServerRequestInterface $request, ResponseInterface $response, $args)
     {
-        $this->logger->info("Fetch user GET '/user/".$args['id']."/roles'");
+        $this->logger->info("Fetch user GET '/user/" . $args['id'] . "/roles'");
         $r = RoleQuery::create()->find();
         $u = UserQuery::create()->findPK($args['id']);
 
@@ -26,7 +26,7 @@ 
 
     public function postUserAssignRoles(ServerRequestInterface $request, ResponseInterface $response, $args)
     {
-        $this->logger->info("Create user people POST '/user/".$args['id']."/roles'");
+        $this->logger->info("Create user people POST '/user/" . $args['id'] . "/roles'");
 
         $userId = filter_var($args['id'], FILTER_SANITIZE_NUMBER_INT);
         $existingRoles = RoleQuery::create()->useUserRoleQuery()->filterByUserId($userId)->endUse()->find();

src/classes/Controller/CalendarController.php

Spacing +3 added lines, -3 removed lines

@@ -76,7 +76,7 @@ 
 
     public function getRevokeCalendar(ServerRequestInterface $request, ResponseInterface $response, $args)
     {
-        $this->logger->info("Fetch settings GET '/user/me/calendar/".$args['id']."/revoke'");
+        $this->logger->info("Fetch settings GET '/user/me/calendar/" . $args['id'] . "/revoke'");
 
         $auth = $this->auth;
         $u = $auth->currentUser();
@@ -101,7 +101,7 @@ 
 
     public function getRenderedCalendar(ServerRequestInterface $request, ResponseInterface $response, $args)
     {
-        $this->logger->info("Fetch calendar GET '/calendar/".$args['token'].'.'.$args['format']."'");
+        $this->logger->info("Fetch calendar GET '/calendar/" . $args['token'] . '.' . $args['format'] . "'");
 
         $c = CalendarTokenQuery::create()
             ->filterByToken($args['token'])
@@ -118,7 +118,7 @@ 
     {
         $getParameters = $request->getQueryParams();
 
-        $this->logger->info("Fetch -LEGACY- calendar GET '/calendar.php?user=".$getParameters['user'].'&token='.$getParameters['token'].'&format='.$getParameters['format']."'");
+        $this->logger->info("Fetch -LEGACY- calendar GET '/calendar.php?user=" . $getParameters['user'] . '&token=' . $getParameters['token'] . '&format=' . $getParameters['format'] . "'");
 
         $userId = filter_var($getParameters['user'], FILTER_VALIDATE_INT);
         $token = $getParameters['token'];

src/classes/EmailAddress.php

Spacing +1 added lines, -1 removed lines

@@ -13,7 +13,7 @@
         if (filter_var(trim($email), FILTER_VALIDATE_EMAIL)) {
             $this->email = filter_var(trim($email), FILTER_SANITIZE_EMAIL);
         } else {
-            throw new InvalidArgumentException('Invalid email address: '.$email);
+            throw new InvalidArgumentException('Invalid email address: ' . $email);
         }
     }
 

src/middleware.php

Spacing +1 added lines, -1 removed lines

@@ -10,7 +10,7 @@
 // e.g: $app->add(new \Slim\Csrf\Guard);
 
 // 3) add route information to twig
-$app->add(function (Request $request, Response $response, callable $next) {
+$app->add(function(Request $request, Response $response, callable $next) {
     $route = $request->getAttribute('route');
 
     // return NotFound for non existent route

api-classes/Database.php

Doc Comments +11 added lines

@@ -148,6 +148,12 @@ 
         return $statement->fetchAll(PDO::FETCH_CLASS, $returnClass);
     }
 
+    /**
+     * @param string $returnClass
+     * @param string $table
+     * @param string[] $columns
+     * @param string[] $where
+     */
     public function selectSingle($returnClass, $table, $columns, $where = null)
     {
         $statement = $this->selectStatement($table, $columns, $where);
@@ -215,6 +221,8 @@ 
 
     /**
      * Use with caution: it is often better to archive items to prevent creating null references.
+     * @param string $table
+     * @param string[] $where
      */
     public function delete($table, $where)
     {
@@ -253,6 +261,9 @@ 
         return $table;
     }
 
+    /**
+     * @param string $table
+     */
     private function tableExists($table)
     {
         // todo: implement table check

Indentation +2 added lines, -2 removed lines

@@ -272,8 +272,8 @@
     public function count($table, $column, $where = null)
     {
         $columns = [
-      'COUNT('.$column.') AS count',
-     ];
+        'COUNT('.$column.') AS count',
+        ];
         $statement = $this->selectStatement($table, $columns, $where);
 
         return $statement->fetchObject()->count;

Spacing +7 added lines, -7 removed lines

@@ -180,10 +180,10 @@ 
             if ($first) {
                 $first = false;
                 $fields .= $item['field'];
-                $valuesPlaceholder .= ':'.$item['field'];
+                $valuesPlaceholder .= ':' . $item['field'];
             } else {
-                $fields .= ', '.$item['field'];
-                $valuesPlaceholder .= ', :'.$item['field'];
+                $fields .= ', ' . $item['field'];
+                $valuesPlaceholder .= ', :' . $item['field'];
             }
 
             if ($item['type'] == 's' || $item['type'] == 'string') {
@@ -201,9 +201,9 @@ 
             }
         }
 
-        $statement = $this->db_connection->prepare('INSERT INTO '.$table.' ('.$fields.') VALUES ('.$valuesPlaceholder.')');
+        $statement = $this->db_connection->prepare('INSERT INTO ' . $table . ' (' . $fields . ') VALUES (' . $valuesPlaceholder . ')');
         foreach ($data as $item) {
-            $statement->bindParam(':'.$item['field'], $item['value']);
+            $statement->bindParam(':' . $item['field'], $item['value']);
         }
 
         if ($statement->execute()) {
@@ -248,7 +248,7 @@ 
 
     private function addPrefix($table)
     {
-        $table = $this->db_prefix.$table;
+        $table = $this->db_prefix . $table;
 
         return $table;
     }
@@ -272,7 +272,7 @@ 
     public function count($table, $column, $where = null)
     {
         $columns = [
-      'COUNT('.$column.') AS count',
+      'COUNT(' . $column . ') AS count',
      ];
         $statement = $this->selectStatement($table, $columns, $where);