BurningFlipside / VolunteerSystem

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

_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
             {

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
                 $uid = $this->user->uid;
                 $email = $this->user->mail;
                 $filter = new \Data\Filter("others eq $uid or others eq $email");
-                $dataTable = DataSetFactory::getDataTableByNames('fvs','departments');
+                $dataTable = DataSetFactory::getDataTableByNames('fvs', 'departments');
                 $depts = $dataTable->read($filter);
                 $this->isLead = !empty($depts);
                 $this->is_admin = true;

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

Spacing +1 added lines, -1 removed lines

@@ -76,7 +76,7 @@
                 $end = strpos($rawText, "</script>");
                 if($index === 0)
                 {
-                    $rawText = substr($rawText, $end+9);
+                    $rawText = substr($rawText, $end + 9);
                 }
             }
             return strtr(strip_tags($rawText), $vars);