BurningFlipside / VolunteerSystem

api/v1/class.Processor.php

Doc Comments +6 added lines

@@ -12,6 +12,9 @@ 
 
     protected abstract function isVolunteerAdmin($request);
 
+    /**
+     * @param VolunteerProfile|null $user
+     */
     public function canUserDoRole($user, $role)
     {
         static $certs = null;
@@ -155,6 +158,9 @@ 
         return !in_array($deptId, $privateDepts);
     }
 
+    /**
+     * @param VolunteerShift $shift
+     */
     protected function doShiftTimeChecks($shift, $entry)
     {
         $now = new DateTime();

app/Emails/class.PendingRejectedEmail.php

Doc Comments +3 added lines

@@ -3,6 +3,9 @@
 
 class PendingRejectedEmail extends VolunteerEmail
 {
+    /**
+     * @param \VolunteerProfile $userProfile
+     */
     public function __construct($userProfile)
     {
         parent::__construct($userProfile);

groupSignup.php

Indentation +25 added lines, -25 removed lines

@@ -9,9 +9,9 @@ 
 
 if(!isset($_GET['id']))
 {
-  $page->body .= 'Error! Missing Group ID. You must have followed a bad link!';
-  $page->printPage();
-  return;
+    $page->body .= 'Error! Missing Group ID. You must have followed a bad link!';
+    $page->printPage();
+    return;
 }
 
 $groupID = $_GET['id'];
@@ -20,9 +20,9 @@ 
 $shifts = $dataTable->read($filter);
 if(empty($shifts))
 {
-  $page->body .= 'Error! Could not locate shifts. You must have followed an old link!';
-  $page->printPage();
-  return;
+    $page->body .= 'Error! Could not locate shifts. You must have followed an old link!';
+    $page->printPage();
+    return;
 }
 
 $filled = array();
@@ -32,35 +32,35 @@ 
 $count = count($shifts);
 for($i = 0; $i < $count; $i++)
 {
-  $shifts[$i] = new \VolunteerShift(false, $shifts[$i]);
-  if($shifts[$i]->status === 'filled' || $shifts[$i]->status === 'pending')
-  {
-      array_push($filled, $shifts[$i]);
-      if($page->user && $shifts[$i]->participant === $page->user->uid)
-      {
-          $alreadySignedUp = true;
-      }
-  }
-  else
-  {
-      array_push($available, $shifts[$i]);
-  }
+    $shifts[$i] = new \VolunteerShift(false, $shifts[$i]);
+    if($shifts[$i]->status === 'filled' || $shifts[$i]->status === 'pending')
+    {
+        array_push($filled, $shifts[$i]);
+        if($page->user && $shifts[$i]->participant === $page->user->uid)
+        {
+            $alreadySignedUp = true;
+        }
+    }
+    else
+    {
+        array_push($available, $shifts[$i]);
+    }
 }
 $page->body .= '<div class="row"><h3>Filled Shifts</h3></div><div class="row"><table class="table"><tr><th>Role</th><th>Participant</th></tr>';
 foreach($filled as $shift)
 {
-  $page->body.= '<tr><td>'.$shift->role->display_name.'</td><td>'.$shift->webParticipantName.'</td></tr>';
+    $page->body.= '<tr><td>'.$shift->role->display_name.'</td><td>'.$shift->webParticipantName.'</td></tr>';
 }
 $page->body .= '</table></div>';
 $page->body .= '<div class="row"><h3>Unfilled Shifts</h3></div><div class="row"><table class="table"><tr><th>Role</th><th>Sign Up</th></tr>';
 foreach($available as $shift)
 {
-  $cell = '<i>You already have a shift!</i>';
-  if($alreadySignedUp === false)
-  {
+    $cell = '<i>You already have a shift!</i>';
+    if($alreadySignedUp === false)
+    {
     $cell = '<button type="button" class="btn btn-primary" onClick="signUp(\''.$shift->_id.'\');">Sign Up</button>';
-  } 
-  $page->body.= '<tr><td>'.$shift->role->display_name.'</td><td>'.$cell.'</td></tr>';
+    } 
+    $page->body.= '<tr><td>'.$shift->role->display_name.'</td><td>'.$cell.'</td></tr>';
 }
 $page->body .= '</table></div>';
 $page->printPage();

app/class.VolunteerShift.php

Indentation +1 added lines, -1 removed lines

@@ -113,7 +113,7 @@
 
     public function isFilled()
     {
-         return isset($this->dbData['status']) && ($this->dbData['status'] === 'pending' || $this->dbData['status'] === 'filled' || $this->dbData['status'] === 'groupPending');
+            return isset($this->dbData['status']) && ($this->dbData['status'] === 'pending' || $this->dbData['status'] === 'filled' || $this->dbData['status'] === 'groupPending');
     }
 
     public function findOverlaps($uid, $shortCircuit = false)

Doc Comments +3 added lines

@@ -177,6 +177,9 @@
         return $res;
     }
 
+    /**
+     * @param Data\DataTable $dataTable
+     */
     public function makeCopy($dataTable)
     {
         $tmp = $this->dbData;

_admin/class.VolunteerAdminPage.php

Indentation +1 added lines, -1 removed lines

@@ -36,7 +36,7 @@
             $this->isLead = $this->user->isInGroupNamed('Leads');
             if($this->isLead)
             {
-                 $this->is_admin = true;
+                    $this->is_admin = true;
             }
             else
             {

app/Emails/class.ShiftEmail.php

Indentation +9 added lines, -9 removed lines

@@ -44,15 +44,15 @@
         $start = $this->shift->startTime->format('r');
         $end = $this->shift->endTime->format('r');
         $vars = array(
-          '{$firstName}' => $firstName,
-          '{$lastName}' => $lastName,
-          '{$paperName}' => $paperName,
-          '{$webName}' => $webName,
-          '{$department}' => $department,
-          '{$role}' => $role,
-          '{$event}' => $event,
-          '{$start}' => $start,
-          '{$end}' => $end
+            '{$firstName}' => $firstName,
+            '{$lastName}' => $lastName,
+            '{$paperName}' => $paperName,
+            '{$webName}' => $webName,
+            '{$department}' => $department,
+            '{$role}' => $role,
+            '{$event}' => $event,
+            '{$start}' => $start,
+            '{$end}' => $end
         );
         if(strpos($this->text, '{$newStart}') !== false || strpos($this->text, '{$newEnd}') !== false)
         {

app/Emails/class.CertificationEmail.php

Indentation +6 added lines, -6 removed lines

@@ -48,12 +48,12 @@
         $certType = $this->additionalProps['certType'];
         $rejectReason = $this->additionalProps['reason'];
         $vars = array(
-          '{$firstName}' => $firstName,
-          '{$lastName}' => $lastName,
-          '{$paperName}' => $paperName,
-          '{$webName}' => $webName,
-          '{$certType}' => $certType,
-          '{$rejectReason}' => $rejectReason
+            '{$firstName}' => $firstName,
+            '{$lastName}' => $lastName,
+            '{$paperName}' => $paperName,
+            '{$webName}' => $webName,
+            '{$certType}' => $certType,
+            '{$rejectReason}' => $rejectReason
         );
         if($html === true)
         {

api/v1/class.ShiftAPI.php

Doc Comments +5 added lines

@@ -221,6 +221,11 @@
         return $response->withJSON($res, 500);
     }
 
+    /**
+     * @param string $status
+     * @param Data\Filter $filter
+     * @param Data\DataTable $dataTable
+     */
     protected function doSignup($uid, $status, $entity, $filter, $dataTable)
     {
         if(isset($entity['earlyLate']) && $entity['earlyLate'] !== '-1')

Indentation +8 added lines, -8 removed lines

@@ -52,13 +52,13 @@ 
         }
         if(!isset($obj['departmentID']))
         {
-             return false;
+                return false;
         }
         if(isset($obj['unbounded']) && $obj['unbounded'])
         {
             if(!isset($obj['minShifts']) || $obj['minShifts'] === 0 || $obj['minShifts'] === '')
             {
-                 $obj['minShifts'] = '1';
+                    $obj['minShifts'] = '1';
             }
         }
         return $this->isUserDepartmentLead($obj['departmentID'], $this->user);
@@ -159,8 +159,8 @@ 
             $ret = $dataTable->update($filter, $entArray[$i]);
             if($ret === false)
             {
-               $myRet = false;
-               array_push($errors, $array[$i]);
+                $myRet = false;
+                array_push($errors, $array[$i]);
             }
         }
         if($myRet)
@@ -265,7 +265,7 @@ 
         $entity = $this->processShift($entity, $request);
         if(isset($entity['minShifts']) && $entity['minShifts'] > 0)
         {
-          $shift->makeCopy($dataTable);
+            $shift->makeCopy($dataTable);
         }
         if(isset($entity['overlap']) && $entity['overlap'])
         {
@@ -331,7 +331,7 @@ 
         $entity['status'] = 'unfilled';
         if(isset($entity['needEEApproval']))
         {
-          unset($entity['needEEApproval']);
+            unset($entity['needEEApproval']);
         }
         return $response->withJSON($dataTable->update($filter, $entity));
     }
@@ -528,7 +528,7 @@ 
         $entity['status'] = 'unfilled';
         if(isset($entity['needEEApproval']))
         {
-          unset($entity['needEEApproval']);
+            unset($entity['needEEApproval']);
         }
         $ret = $dataTable->update($filter, $entity);
         if($ret)
@@ -563,7 +563,7 @@ 
         $entity['status'] = 'unfilled';
         if(isset($entity['needEEApproval']))
         {
-          unset($entity['needEEApproval']);
+            unset($entity['needEEApproval']);
         }
         return $response->withJSON($dataTable->update($filter, $entity));
     }

app/class.VolunteerEvent.php

Doc Comments +6 added lines

@@ -6,6 +6,9 @@ 
         parent::__construct($departmentID, $dbData, 'events', '_id');
     }
 
+    /**
+     * @param integer $eeListIndex
+     */
     public function hasVolOnEEList($uid, $eeListIndex)
     {
         if($eeListIndex === 1 || $eeListIndex === 0)
@@ -28,6 +31,9 @@ 
         return false;
     }
 
+    /**
+     * @param integer $eeListIndex
+     */
     public function addToEEList($uid, $eeListIndex)
     {
         if(!isset($this->dbData['eeLists']))