TechWilk / rota

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

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);
 

generated-classes/TechWilk/Rota/User.php

Doc Comments +1 added lines, -1 removed lines

@@ -196,7 +196,7 @@
     /**
      * Determine if the user is marked as available for an event.
      *
-     * @return bool if user is available
+     * @return null|boolean if user is available
      */
     public function isAvailableForEvent(Event $event)
     {

Spacing +13 added lines, -13 removed lines

@@ -77,7 +77,7 @@ 
      */
     public function getName()
     {
-        return $this->firstname.' '.$this->lastname;
+        return $this->firstname . ' ' . $this->lastname;
     }
 
     /**
@@ -96,16 +96,16 @@ 
                 if ($socialAuth->getPlatform() == 'facebook') {
                     switch ($size) {
                         case 'small': // 50px x 50px
-                            return '//graph.facebook.com/'.$socialAuth->getSocialId().'/picture?type=square';
+                            return '//graph.facebook.com/' . $socialAuth->getSocialId() . '/picture?type=square';
                             break;
                         case 'medium': // 200px x 200px
-                            return '//graph.facebook.com/'.$socialAuth->getSocialId().'/picture?type=large';
+                            return '//graph.facebook.com/' . $socialAuth->getSocialId() . '/picture?type=large';
                             break;
                         case 'large': // 200px x 200px
-                            return '//graph.facebook.com/'.$socialAuth->getSocialId().'/picture?type=large';
+                            return '//graph.facebook.com/' . $socialAuth->getSocialId() . '/picture?type=large';
                             break;
                         default:
-                            return '//graph.facebook.com/'.$socialAuth->getSocialId().'/picture';
+                            return '//graph.facebook.com/' . $socialAuth->getSocialId() . '/picture';
                             break;
                     }
                 } elseif ($socialAuth->getPlatform() == 'onebody') {
@@ -114,16 +114,16 @@ 
                     $extension = pathinfo($socialAuth->getMeta()['photo-file-name'], PATHINFO_EXTENSION);
                     switch ($size) {
                         case 'small': // 50px x 50px
-                            return $baseUrl.'/system/production/people/photos/'.$socialAuth->getSocialId().'/tn/'.$photoFingerprint.'.'.$extension;
+                            return $baseUrl . '/system/production/people/photos/' . $socialAuth->getSocialId() . '/tn/' . $photoFingerprint . '.' . $extension;
                             break;
                         case 'medium': // 150px x 150px
-                            return $baseUrl.'/system/production/people/photos/'.$socialAuth->getSocialId().'/small/'.$photoFingerprint.'.'.$extension;
+                            return $baseUrl . '/system/production/people/photos/' . $socialAuth->getSocialId() . '/small/' . $photoFingerprint . '.' . $extension;
                             break;
                         case 'large': // 500px x 500px
-                            return $baseUrl.'/system/production/people/photos/'.$socialAuth->getSocialId().'/medium/'.$photoFingerprint.'.'.$extension;
+                            return $baseUrl . '/system/production/people/photos/' . $socialAuth->getSocialId() . '/medium/' . $photoFingerprint . '.' . $extension;
                             break;
                         default:
-                            return $baseUrl.'/system/production/people/photos/'.$socialAuth->getSocialId().'/tn/'.$photoFingerprint.'.'.$extension;
+                            return $baseUrl . '/system/production/people/photos/' . $socialAuth->getSocialId() . '/tn/' . $photoFingerprint . '.' . $extension;
                         break;
                     }
                 }
@@ -132,16 +132,16 @@ 
 
         switch ($size) {
             case 'small': // 50px x 50px
-                return '//www.gravatar.com/avatar/'.md5(strtolower(trim($this->email))).'?s=50&d=mm';
+                return '//www.gravatar.com/avatar/' . md5(strtolower(trim($this->email))) . '?s=50&d=mm';
                 break;
             case 'medium': // 200px x 200px
-                return '//www.gravatar.com/avatar/'.md5(strtolower(trim($this->email))).'?s=200&d=mm';
+                return '//www.gravatar.com/avatar/' . md5(strtolower(trim($this->email))) . '?s=200&d=mm';
                 break;
             case 'large': // 500px x 500px
-                return '//www.gravatar.com/avatar/'.md5(strtolower(trim($this->email))).'?s=500&d=mm';
+                return '//www.gravatar.com/avatar/' . md5(strtolower(trim($this->email))) . '?s=500&d=mm';
                 break;
             default:
-                return '//www.gravatar.com/avatar/'.md5(strtolower(trim($this->email))).'?s=50&d=mm';
+                return '//www.gravatar.com/avatar/' . md5(strtolower(trim($this->email))) . '?s=50&d=mm';
             break;
         }
     }

src/classes/Authentication.php

Doc Comments +3 added lines

@@ -119,6 +119,9 @@
         return true;
     }
 
+    /**
+     * @param EmailAddress $username
+     */
     private function numberOfLoginAttemptsIsOk($username)
     {
         $numberOfAllowedAttempts = 8;

public/api/v1/index.php

Spacing +31 added lines, -31 removed lines

@@ -23,15 +23,15 @@ 
 // Start of API
 // ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
 
-require_once dirname(__FILE__).'/../../../vendor/autoload.php';
+require_once dirname(__FILE__) . '/../../../vendor/autoload.php';
 
-include_once dirname(__FILE__).'/../../../config/database.php';
+include_once dirname(__FILE__) . '/../../../config/database.php';
 
 // Create and configure Slim app
 $app = new \Slim\App(['settings' => $config]);
 
-spl_autoload_register(function ($classname) {
-    require dirname(__FILE__).'/../../../api-classes/'.$classname.'.php';
+spl_autoload_register(function($classname) {
+    require dirname(__FILE__) . '/../../../api-classes/' . $classname . '.php';
 });
 
 // ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
@@ -40,7 +40,7 @@ 
 
 $container = $app->getContainer();
 
-$container['db'] = function ($c) {
+$container['db'] = function($c) {
     $db_config = $c['settings']['db'];
     $db = new Database($db_config);
 
@@ -53,7 +53,7 @@ 
 
 // ~~~~~~~~~~~~~~~ Event ~~~~~~~~~~~~~~~
 
-$app->post('/events', function ($request, $response, $args) {
+$app->post('/events', function($request, $response, $args) {
     $postData = $request->getParsedBody();
 
     $name = filter_var($postData['name'], FILTER_SANITIZE_STRING);
@@ -65,7 +65,7 @@ 
     return $response->withJson($data);
 });
 
-$app->get('/events/{id}', function ($request, $response, $args) {
+$app->get('/events/{id}', function($request, $response, $args) {
     $id = filter_var($args['id'], FILTER_SANITIZE_NUMBER_INT);
 
     $event = new Event();
@@ -91,8 +91,8 @@ 
                     'relationships' => [
                                         'series' => [
                                                     'links' => [
-                                                                'self'    => '/events/'.$event->getId().'/relationships/series',
-                                                                'related' => '/events/'.$event->getId().'/series/'.$series->getId(),
+                                                                'self'    => '/events/' . $event->getId() . '/relationships/series',
+                                                                'related' => '/events/' . $event->getId() . '/series/' . $series->getId(),
                                                     ],
                                                     'data' => [
                                                       'type' => 'series',
@@ -101,8 +101,8 @@ 
                                         ],
                                         'types' => [
                                                     'links' => [
-                                                                'self'    => '/events/'.$event->getId().'/relationships/types',
-                                                                'related' => '/events/'.$event->getId().'/types/'.$type->getId(),
+                                                                'self'    => '/events/' . $event->getId() . '/relationships/types',
+                                                                'related' => '/events/' . $event->getId() . '/types/' . $type->getId(),
                                                     ],
                                                     'data' => [
                                                       'type' => 'types',
@@ -111,8 +111,8 @@ 
                                         ],
                                         'sub-types' => [
                                                     'links' => [
-                                                                'self'    => '/events/'.$event->getId().'/relationships/sub-types',
-                                                                'related' => '/events/'.$event->getId().'/sub-types/'.$sub_type->getId(),
+                                                                'self'    => '/events/' . $event->getId() . '/relationships/sub-types',
+                                                                'related' => '/events/' . $event->getId() . '/sub-types/' . $sub_type->getId(),
                                                     ],
                                                     'data' => [
                                                                 'type' => 'sub-types',
@@ -121,8 +121,8 @@ 
                                         ],
                                         'location' => [
                                                     'links' => [
-                                                                'self'    => '/events/'.$event->getId().'/relationships/locations',
-                                                                'related' => '/events/'.$event->getId().'/locations/'.$location->getId(),
+                                                                'self'    => '/events/' . $event->getId() . '/relationships/locations',
+                                                                'related' => '/events/' . $event->getId() . '/locations/' . $location->getId(),
                                                     ],
                                                     'data' => [
                                                                 'type' => 'locations',
@@ -140,7 +140,7 @@ 
                                           'description' => $series->description,
                             ],
                             'links' => [
-                                        'self' => '/series/'.$series->getId(),
+                                        'self' => '/series/' . $series->getId(),
                             ],
                           ],
                           [
@@ -151,7 +151,7 @@ 
                                       'description' => $type->description,
                             ],
                             'links' => [
-                                        'self' => '/types/'.$type->getId(),
+                                        'self' => '/types/' . $type->getId(),
                             ],
                           ],
                           [
@@ -162,7 +162,7 @@ 
                                       'description' => $sub_type->description,
                             ],
                             'links' => [
-                                        'self' => '/sub-types/'.$sub_type->getId(),
+                                        'self' => '/sub-types/' . $sub_type->getId(),
                             ],
                           ],
                           [
@@ -172,7 +172,7 @@ 
                                       'name' => $location->name,
                             ],
                             'links' => [
-                                        'self' => '/locations/'.$location->getId(),
+                                        'self' => '/locations/' . $location->getId(),
                             ],
                           ],
           ],
@@ -186,7 +186,7 @@ 
 
 // ~~~~~~~~~~~~~~~ Series ~~~~~~~~~~~~~~~
 
-$app->post('/series', function ($request, $response, $args) {
+$app->post('/series', function($request, $response, $args) {
     $postData = $request->getParsedBody();
 
     $name = filter_var($postData['name'], FILTER_SANITIZE_STRING);
@@ -209,7 +209,7 @@ 
     return $response->withJson($data);
 });
 
-$app->get('/series/{id}', function ($request, $response, $args) {
+$app->get('/series/{id}', function($request, $response, $args) {
     $id = (int) filter_var($args['id'], FILTER_SANITIZE_NUMBER_INT);
 
     $series = new Series();
@@ -226,7 +226,7 @@ 
     return $response->withJson($data);
 });
 
-$app->delete('/series/{id}', function ($request, $response, $args) {
+$app->delete('/series/{id}', function($request, $response, $args) {
     $id = (int) filter_var($args['id'], FILTER_SANITIZE_NUMBER_INT);
 
     $series = new Series();
@@ -244,7 +244,7 @@ 
 
 // ~~~~~~~~~~~~~~~ Type ~~~~~~~~~~~~~~~
 
-$app->post('/types', function ($request, $response, $args) {
+$app->post('/types', function($request, $response, $args) {
     $postData = $request->getParsedBody();
 
     $name = filter_var($postData['name'], FILTER_SANITIZE_STRING);
@@ -267,7 +267,7 @@ 
     return $response->withJson($data);
 });
 
-$app->get('/types/{id}', function ($request, $response, $args) {
+$app->get('/types/{id}', function($request, $response, $args) {
     $id = (int) filter_var($args['id'], FILTER_SANITIZE_NUMBER_INT);
 
     $type = new Type();
@@ -284,7 +284,7 @@ 
     return $response->withJson($data);
 });
 
-$app->delete('/types/{id}', function ($request, $response, $args) {
+$app->delete('/types/{id}', function($request, $response, $args) {
     $id = (int) filter_var($args['id'], FILTER_SANITIZE_NUMBER_INT);
 
     $type = new Type();
@@ -302,7 +302,7 @@ 
 
 // ~~~~~~~~~~~~~~~ Sub Type ~~~~~~~~~~~~~~~
 
-$app->post('/sub-types', function ($request, $response, $args) {
+$app->post('/sub-types', function($request, $response, $args) {
     $postData = $request->getParsedBody();
 
     $name = filter_var($postData['name'], FILTER_SANITIZE_STRING);
@@ -325,7 +325,7 @@ 
     return $response->withJson($data);
 });
 
-$app->get('/sub-types/{id}', function ($request, $response, $args) {
+$app->get('/sub-types/{id}', function($request, $response, $args) {
     $id = (int) filter_var($args['id'], FILTER_SANITIZE_NUMBER_INT);
 
     $type = new SubType();
@@ -342,7 +342,7 @@ 
     return $response->withJson($data);
 });
 
-$app->delete('/sub-types/{id}', function ($request, $response, $args) {
+$app->delete('/sub-types/{id}', function($request, $response, $args) {
     $id = (int) filter_var($args['id'], FILTER_SANITIZE_NUMBER_INT);
 
     $type = new SubType();
@@ -360,7 +360,7 @@ 
 
 // ~~~~~~~~~~~~~~~ Location ~~~~~~~~~~~~~~~
 
-$app->post('/locations', function ($request, $response, $args) {
+$app->post('/locations', function($request, $response, $args) {
     $postData = $request->getParsedBody();
 
     $name = filter_var($postData['name'], FILTER_SANITIZE_STRING);
@@ -380,7 +380,7 @@ 
     return $response->withJson($data);
 });
 
-$app->get('/locations/{id}', function ($request, $response, $args) {
+$app->get('/locations/{id}', function($request, $response, $args) {
     $id = (int) filter_var($args['id'], FILTER_SANITIZE_NUMBER_INT);
 
     $location = new Location();
@@ -396,7 +396,7 @@ 
     return $response->withJson($data);
 });
 
-$app->delete('/locations/{id}', function ($request, $response, $args) {
+$app->delete('/locations/{id}', function($request, $response, $args) {
     $id = (int) filter_var($args['id'], FILTER_SANITIZE_NUMBER_INT);
 
     $location = new Location();

public/plugins/datatables/extensions/Scroller/examples/data/ssp.php

Spacing +5 added lines, -5 removed lines

@@ -29,11 +29,11 @@
 // parameter represents the DataTables column identifier. In this case simple
 // indexes
 $columns = [
-    ['db' => 'id',         'dt' => 0],
-    ['db' => 'firstname',  'dt' => 1],
-    ['db' => 'surname',    'dt' => 2],
-    ['db' => 'zip',        'dt' => 3],
-    ['db' => 'country',    'dt' => 4],
+    ['db' => 'id', 'dt' => 0],
+    ['db' => 'firstname', 'dt' => 1],
+    ['db' => 'surname', 'dt' => 2],
+    ['db' => 'zip', 'dt' => 3],
+    ['db' => 'country', 'dt' => 4],
 ];
 
 // SQL server connection information

public/old/swaps.php

Spacing +3 added lines, -3 removed lines

@@ -39,7 +39,7 @@ 
 // ensure user is allowed to see page
 
 if (!isAdmin() && $queryStringUser != $sessionUserId) {
-    header('Location: swaps.php?user='.$sessionUserId);
+    header('Location: swaps.php?user=' . $sessionUserId);
 }
 
 // fetch swaps
@@ -90,11 +90,11 @@ 
               <p>
                 <strong>
                   <s class="text-red">
-                    <?php echo $swap->getOldUserRole()->getUser()->getFirstName().' '.$swap->getOldUserRole()->getUser()->getLastName() ?> (<?php echo $swap->getOldUserRole()->getRole()->getName() ?>)
+                    <?php echo $swap->getOldUserRole()->getUser()->getFirstName() . ' ' . $swap->getOldUserRole()->getUser()->getLastName() ?> (<?php echo $swap->getOldUserRole()->getRole()->getName() ?>)
                   </s>
                   &#8594;
                   <span class="text-green">
-                    <?php echo $swap->getNewUserRole()->getUser()->getFirstName().' '.$swap->getNewUserRole()->getUser()->getLastName() ?> (<?php echo $swap->getNewUserRole()->getRole()->getName() ?>)
+                    <?php echo $swap->getNewUserRole()->getUser()->getFirstName() . ' ' . $swap->getNewUserRole()->getUser()->getLastName() ?> (<?php echo $swap->getNewUserRole()->getRole()->getName() ?>)
                   </span>
                 </strong>
               </p>

public/old/overview.php

Spacing +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
     exit;
 }
 if (!isAdmin()) {
-    header('Location: error.php?no=100&page='.basename($_SERVER['SCRIPT_FILENAME']));
+    header('Location: error.php?no=100&page=' . basename($_SERVER['SCRIPT_FILENAME']));
     exit;
 }
 
@@ -115,7 +115,7 @@ 
 
 		  <?php
             else:
-                echo '<div class="box-body">'.$sentSuccess.'</div>';
+                echo '<div class="box-body">' . $sentSuccess . '</div>';
             endif;
             ?>
 				</div>

public/old/editBand.php

Spacing +11 added lines, -11 removed lines

@@ -33,7 +33,7 @@ 
     exit;
 }
 if (!isAdmin()) {
-    header('Location: error.php?no=100&page='.basename($_SERVER['SCRIPT_FILENAME']));
+    header('Location: error.php?no=100&page=' . basename($_SERVER['SCRIPT_FILENAME']));
     exit;
 }
 
@@ -119,15 +119,15 @@ 
                 }
 
                 if ($viewPeople['skill'] != '') {
-                    $skill = ' - <em>'.$viewPeople['skill'].'</em>';
+                    $skill = ' - <em>' . $viewPeople['skill'] . '</em>';
                 } else {
                     $skill = '';
                 }
                 if ($position == 2) {
-                    echo "<div class='checkboxitem right'><label class='styled' for='rehearsaldate[".$viewPeople['skillID']."]'>".
-                        $viewPeople['name'].$skill."</em></label><input class='styled' ".$checked."type='checkbox' id='rehearsaldate[".
-                        $viewPeople['skillID']."]'	name='rehearsaldate[]' value='".
-                        $viewPeople['skillID']."' /></div></div>";
+                    echo "<div class='checkboxitem right'><label class='styled' for='rehearsaldate[" . $viewPeople['skillID'] . "]'>" .
+                        $viewPeople['name'] . $skill . "</em></label><input class='styled' " . $checked . "type='checkbox' id='rehearsaldate[" .
+                        $viewPeople['skillID'] . "]'	name='rehearsaldate[]' value='" .
+                        $viewPeople['skillID'] . "' /></div></div>";
 
                     $position = 1;
                 } else {
@@ -139,11 +139,11 @@ 
                         $i = '1';
                     }
 
-                    echo "<div class='row".$class."'>
-						<div class='checkboxitem'><label class='styled' for='rehearsaldate[".$viewPeople['skillID']."]'>".
-                        $viewPeople['name'].$skill."</em></label><input class='styled' ".$checked."type='checkbox' id='rehearsaldate[".
-                        $viewPeople['skillID']."]'	name='rehearsaldate[]' value='".
-                        $viewPeople['skillID']."' /></div>";
+                    echo "<div class='row" . $class . "'>
+						<div class='checkboxitem'><label class='styled' for='rehearsaldate[".$viewPeople['skillID'] . "]'>" .
+                        $viewPeople['name'] . $skill . "</em></label><input class='styled' " . $checked . "type='checkbox' id='rehearsaldate[" .
+                        $viewPeople['skillID'] . "]'	name='rehearsaldate[]' value='" .
+                        $viewPeople['skillID'] . "' /></div>";
 
                     $position = 2;
                 }

public/old/plugins/datatables/extensions/Scroller/examples/data/ssp.php

Spacing +5 added lines, -5 removed lines

@@ -29,11 +29,11 @@
 // parameter represents the DataTables column identifier. In this case simple
 // indexes
 $columns = [
-    ['db' => 'id',         'dt' => 0],
-    ['db' => 'firstname',  'dt' => 1],
-    ['db' => 'surname',    'dt' => 2],
-    ['db' => 'zip',        'dt' => 3],
-    ['db' => 'country',    'dt' => 4],
+    ['db' => 'id', 'dt' => 0],
+    ['db' => 'firstname', 'dt' => 1],
+    ['db' => 'surname', 'dt' => 2],
+    ['db' => 'zip', 'dt' => 3],
+    ['db' => 'country', 'dt' => 4],
 ];
 
 // SQL server connection information