BurningFlipside / VolunteerSystem

app/Schedules/class.SimplePDF.php

Spacing +7 added lines, -7 removed lines

@@ -35,7 +35,7 @@ 
             $timeStr = $start->format('g:i A').' till '.$end->format('g:i A');
             if(strlen($shifts[$i]['name']) > 0)
             {
-                $timeStr .=' - <i>'.$shifts[$i]['name'].'</i>';
+                $timeStr .= ' - <i>'.$shifts[$i]['name'].'</i>';
             }
             if(!isset($days[$dateStr]))
             {
@@ -50,22 +50,22 @@ 
         ksort($days);
         foreach($days as $dateStr=>$day)
         {
-            $html .='<h2>'.$dateStr.'</h2>';
+            $html .= '<h2>'.$dateStr.'</h2>';
             uksort($day, array($this, 'groupSort'));
             foreach($day as $shiftStr=>$shifts)
             {
                 usort($shifts, array($this, 'shiftSort'));
-                $html .='<h3>'.$shiftStr.'</h3>';
-                $html .='<table width="100%"><tr><th style="width: 20%">Role</th><th>Volunteer Name</th><th>Volunteer Camp</th></tr>';
+                $html .= '<h3>'.$shiftStr.'</h3>';
+                $html .= '<table width="100%"><tr><th style="width: 20%">Role</th><th>Volunteer Name</th><th>Volunteer Camp</th></tr>';
                 foreach($shifts as $shift)
                 {
                     //TODO Volunteer info for shift...
-                    $html .='<tr><td>'.$this->getRoleNameFromID($shift['roleID']).'</td><td></td><td></td></tr>';
+                    $html .= '<tr><td>'.$this->getRoleNameFromID($shift['roleID']).'</td><td></td><td></td></tr>';
                 }
-                $html .='</table>';
+                $html .= '</table>';
             }
         }
-        $html .='</body>';
+        $html .= '</body>';
         $this->setPDFFromHTML($html);
     }
 } 

api/v1/class.VolunteerAPI.php

Doc Comments +3 added lines

@@ -5,6 +5,9 @@
     protected $isAdmin = null;
     protected $isLead = null;
 
+    /**
+     * @param string $dataTableName
+     */
     public function __construct($dataTableName, $idField = '_id')
     {
         parent::__construct('fvs', $dataTableName, $idField);

api/v1/class.DepartmentAPI.php

Doc Comments +6 added lines

@@ -109,6 +109,9 @@ 
         return $response->withJson($shifts);
     }
 
+    /**
+     * @param string $table
+     */
     protected function createEntityForDepartment($request, $response, $args, $table)
     {
         $deptId = $args['dept'];
@@ -212,6 +215,9 @@ 
         return $response;
     }
 
+    /**
+     * @param string $type
+     */
     public function generateGridSchedule($dept, $shifts, $response, $type)
     {
         $ss = new \Schedules\GridSchedule($dept, $shifts);

api/v1/class.Processor.php

Doc Comments +9 added lines

@@ -1,6 +1,9 @@ 
 <?php
 trait Processor
 {
+    /**
+     * @param string $certType
+     */
     protected function certCheck($requirements, $certs, $certType)
     {
         if(isset($requirements[$certType]) && $requirements[$certType])
@@ -12,6 +15,9 @@ 
 
     protected abstract function isVolunteerAdmin($request);
 
+    /**
+     * @param VolunteerProfile|null $user
+     */
     public function canUserDoRole($user, $role)
     {
         if($role['publicly_visible'] === true)
@@ -152,6 +158,9 @@ 
         return !in_array($deptId, $privateDepts);
     }
 
+    /**
+     * @param VolunteerShift $shift
+     */
     protected function doShiftTimeChecks($shift, $entry)
     {
         $now = new DateTime();

app/Schedules/class.GridSchedule.php

Doc Comments +17 added lines

@@ -45,6 +45,11 @@ 
         return ($hour - 12).'p';
     }
 
+    /**
+     * @param integer $hourCount
+     * @param integer $rowCount
+     * @param \PhpOffice\PhpSpreadsheet\Worksheet\Worksheet $sheat
+     */
     protected function grayOutUnused($hourCount, $rowCount, $sheat)
     {
         for($i = 0; $i < $hourCount; $i++)
@@ -74,6 +79,9 @@ 
         }
     }
 
+    /**
+     * @param double $hourCount
+     */
     protected function getHoursArrays($hour, $hourCount)
     {
         $simpleHours = array();
@@ -91,12 +99,21 @@ 
         return array($simpleHours, $militaryHours);
     }
 
+    /**
+     * @param \PhpOffice\PhpSpreadsheet\Worksheet\Worksheet $sheat
+     * @param double $col
+     * @param integer $row
+     */
     protected function createShiftCell($sheat, $col, $row, $shift)
     {
         $sheat->mergeCellsByColumnAndRow($col, $row, $col+$shift['length']-1, $row);
         $this->setShiftNameInCell($sheat, $col, $row, $shift);
     }
 
+    /**
+     * @param double $col
+     * @param \PhpOffice\PhpSpreadsheet\Worksheet\Worksheet $sheat
+     */
     protected function getRowForShift($roleID, $rows, $col, $sheat)
     {
         $firstRow = array_search($roleID, $rows);

Spacing +12 added lines, -12 removed lines

@@ -51,7 +51,7 @@ 
         {
             for($j = 0; $j < $rowCount; $j++)
             {
-                 $cell = $sheat->getCellByColumnAndRow($i+2, $j+4);
+                 $cell = $sheat->getCellByColumnAndRow($i + 2, $j + 4);
                  if($cell->isInMergeRange())
                  {
                       continue;
@@ -93,28 +93,28 @@ 
 
     protected function createShiftCell($sheat, $col, $row, $shift)
     {
-        $sheat->mergeCellsByColumnAndRow($col, $row, $col+$shift['length']-1, $row);
+        $sheat->mergeCellsByColumnAndRow($col, $row, $col + $shift['length'] - 1, $row);
         $this->setShiftNameInCell($sheat, $col, $row, $shift);
     }
 
     protected function getRowForShift($roleID, $rows, $col, $sheat)
     {
         $firstRow = array_search($roleID, $rows);
-        $cell = $sheat->getCellByColumnAndRow($col, $firstRow+4);
+        $cell = $sheat->getCellByColumnAndRow($col, $firstRow + 4);
         if($cell->isInMergeRange())
         {
-            while($rows[$firstRow+$i] === $roleID)
+            while($rows[$firstRow + $i] === $roleID)
             {
-                $cell = $sheat->getCellByColumnAndRow($hoursFromStart+2, $firstRow+4+$i);
+                $cell = $sheat->getCellByColumnAndRow($hoursFromStart + 2, $firstRow + 4 + $i);
                 if(!$cell->isInMergeRange())
                 {
                     break;
                 }
                 $i++;
             }
-            return $firstRow+4+$i;
+            return $firstRow + 4 + $i;
         }
-        return $firstRow+4;
+        return $firstRow + 4;
     }
 
     protected function createSpreadSheet()
@@ -161,7 +161,7 @@ 
         $start = date_parse($str);
         $lastShift = $shifts[$count - 1];
         $interval = $lastShift['endTime']->diff($shifts[0]['startTime']);
-        $hourCount = ($interval->d*24) + $interval->h;
+        $hourCount = ($interval->d * 24) + $interval->h;
         $tmp = $this->getHoursArrays($start['hour'], $hourCount);
         $simpleHours = $tmp[0];
         $militaryHours = $tmp[1];
@@ -232,13 +232,13 @@ 
         {
             $i = 1;
             $timeDiff = $originalStartTime->diff($shift['startTime']);
-            $hoursFromStart = ($timeDiff->d*24)+$timeDiff->h;
-            $rowForShift = $this->getRowForShift($shift['roleID'], $rows, $hoursFromStart+2, $sheat);
-            $this->createShiftCell($sheat, $hoursFromStart+2, $rowForShift, $shift);
+            $hoursFromStart = ($timeDiff->d * 24) + $timeDiff->h;
+            $rowForShift = $this->getRowForShift($shift['roleID'], $rows, $hoursFromStart + 2, $sheat);
+            $this->createShiftCell($sheat, $hoursFromStart + 2, $rowForShift, $shift);
             $shift = array_shift($shifts);
         }
         $rowCount = count($rows);
-        $style = $sheat->getStyleByColumnAndRow(2, 4, 1+count($simpleHours), 3 + $rowCount);
+        $style = $sheat->getStyleByColumnAndRow(2, 4, 1 + count($simpleHours), 3 + $rowCount);
         $style->getBorders()->getAllBorders()->setBorderStyle(\PhpOffice\PhpSpreadsheet\Style\Border::BORDER_THIN);
         $hourCount = count($simpleHours);
         $this->grayOutUnused($hourCount, $rowCount, $sheat);