BurningFlipside / VolunteerSystem

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

Spacing +2 added lines, -2 removed lines

@@ -49,7 +49,7 @@ 
 $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>';
@@ -60,7 +60,7 @@ 
   {
     $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;

app/Schedules/class.SimplePDF.php

Spacing +8 added lines, -8 removed lines

@@ -45,7 +45,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]))
             {
@@ -60,30 +60,30 @@ 
         uksort($days, array($this, 'daySort'));
         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)
                 {
                     $shift = new \VolunteerShift(false, $shift);
                     $participant = $shift->participantObj;
                     if($participant !== false)
                     { 
-                        $html .='<tr><td>'.$this->getRoleNameFromID($shift->roleID).'</td><td>'.$participant->getDisplayName('paperName').'</td><td>'.$participant->campName.'</td></tr>';
+                        $html .= '<tr><td>'.$this->getRoleNameFromID($shift->roleID).'</td><td>'.$participant->getDisplayName('paperName').'</td><td>'.$participant->campName.'</td></tr>';
                     }
                     else
                     {
-                        $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);
     }
 } 

Braces +1 added lines, -1 removed lines

@@ -72,7 +72,7 @@
                     $shift = new \VolunteerShift(false, $shift);
                     $participant = $shift->participantObj;
                     if($participant !== false)
-                    { 
+                    {
                         $html .='<tr><td>'.$this->getRoleNameFromID($shift->roleID).'</td><td>'.$participant->getDisplayName('paperName').'</td><td>'.$participant->campName.'</td></tr>';
                     }
                     else