route4me / route4me-php-sdk

UnitTestFiles/Test/TerritoryTests.php

Braces +3 added lines, -1 removed lines

@@ -294,7 +294,9 @@
 
     public static function tearDownAfterClass()
     {
-        if (sizeof(self::$createdTerritories)<1) return;
+        if (sizeof(self::$createdTerritories)<1) {
+        	return;
+        }
 
         $territory = new Territory();
 

UnitTestFiles/Test/AvoidanceZoneUnitTests.php

Braces +1 added lines, -2 removed lines

@@ -199,8 +199,7 @@
 
             if ($result==true) {
                 echo "The test avoidance zone with territory_id=".$territoryId." removed. <br>";
-            }
-            else {
+            } else {
                 echo "Cannot remove the test avoidance zone with territory_id=".$territoryId."<br>";
             }
 

UnitTestFiles/Test/AddressBookLocationUnitTests.php

Braces +3 added lines, -1 removed lines

@@ -383,7 +383,9 @@
 
     public function testValidateScheduleNthWhat()
     {
-        for ($i=1;$i<11;$i++) $this->assertTrue(AddressBookLocation::validateScheduleNthWhat($i));
+        for ($i=1;$i<11;$i++) {
+        	$this->assertTrue(AddressBookLocation::validateScheduleNthWhat($i));
+        }
 
         $this->assertFalse(AddressBookLocation::validateScheduleNthWhat(0));
         $this->assertFalse(AddressBookLocation::validateScheduleNthWhat(14));

examples/optimizationWithFineTuningRoutes.php

Braces +6 added lines, -2 removed lines

@@ -59,7 +59,9 @@ 
 
 assert(is_array($duplicateResult1), "Cannot duplicate the route<br>");
 
-if (!$duplicateResult1['status'] || sizeof($duplicateResult1['route_ids'])<1) die("Cannot duplicate the route<br>");
+if (!$duplicateResult1['status'] || sizeof($duplicateResult1['route_ids'])<1) {
+	die("Cannot duplicate the route<br>");
+}
 
 $duplicatedRouteId1 = $duplicateResult1['route_ids'][0];
 
@@ -95,7 +97,9 @@ 
 
 assert(is_array($duplicateResult2), "Cannot duplicate the route<br>");
 
-if (!$duplicateResult2['status'] || sizeof($duplicateResult2['route_ids'])<1) die("Cannot duplicate the route<br>");
+if (!$duplicateResult2['status'] || sizeof($duplicateResult2['route_ids'])<1) {
+	die("Cannot duplicate the route<br>");
+}
 
 $duplicatedRouteId2 = $duplicateResult2['route_ids'][0];
 

src/Route4Me/AddressBookLocation.php

Switch Indentation +54 added lines, -54 removed lines

@@ -582,70 +582,70 @@
 
             if ('daily' != $mode) {
                 switch ($mode) {
-                    case 'weekly':
-                        if (isset($rows[$locationsFieldsMapping['schedule_weekdays']])) {
-                            if ($this->validateScheduleWeekDays($rows[$locationsFieldsMapping['schedule_weekdays']])) {
-                                $curSchedule .= '"weekdays":['.$rows[$locationsFieldsMapping['schedule_weekdays']].']}';
-                            } else {
-                                array_push($results['fail'], "$iRow --> Wrong weekdays");
-                            }
+                case 'weekly':
+                    if (isset($rows[$locationsFieldsMapping['schedule_weekdays']])) {
+                        if ($this->validateScheduleWeekDays($rows[$locationsFieldsMapping['schedule_weekdays']])) {
+                            $curSchedule .= '"weekdays":['.$rows[$locationsFieldsMapping['schedule_weekdays']].']}';
+                        } else {
+                            array_push($results['fail'], "$iRow --> Wrong weekdays");
+                        }
+                    } else {
+                        array_push($results['fail'], "$iRow --> The parameters sched_weekdays is not set");
+                    }
+                    break;
+                case 'monthly':
+                    $monthlyMode = '';
+                    if (isset($rows[$locationsFieldsMapping['monthly_mode']])) {
+                        if ($this->validateScheduleMonthlyMode($rows[$locationsFieldsMapping['monthly_mode']])) {
+                            $monthlyMode = $rows[$locationsFieldsMapping['monthly_mode']];
+                            $curSchedule .= '"mode": "'.$rows[$locationsFieldsMapping['monthly_mode']].'",';
                         } else {
-                            array_push($results['fail'], "$iRow --> The parameters sched_weekdays is not set");
+                            array_push($results['fail'], "$iRow --> Wrong monthly mode");
+                        }
+                    } else {
+                        array_push($results['fail'], "$iRow --> The parameter sched_monthly_mode is not set");
+                    }
+
+                    if ('' != $monthlyMode) {
+                        switch ($monthlyMode) {
+                        case 'dates':
+                        if (isset($rows[$locationsFieldsMapping['monthly_dates']])) {
+                        if ($this->validateScheduleMonthlyDates($rows[$locationsFieldsMapping['monthly_dates']])) {
+                            $curSchedule .= '"dates":['.$rows[$locationsFieldsMapping['monthly_dates']].']}';
+                        } else {
+                            array_push($results['fail'], "$iRow --> Wrong monthly dates");
+                        }
                         }
                         break;
-                    case 'monthly':
-                        $monthlyMode = '';
-                        if (isset($rows[$locationsFieldsMapping['monthly_mode']])) {
-                            if ($this->validateScheduleMonthlyMode($rows[$locationsFieldsMapping['monthly_mode']])) {
-                                $monthlyMode = $rows[$locationsFieldsMapping['monthly_mode']];
-                                $curSchedule .= '"mode": "'.$rows[$locationsFieldsMapping['monthly_mode']].'",';
-                            } else {
-                                array_push($results['fail'], "$iRow --> Wrong monthly mode");
-                            }
+                        case 'nth':
+                        if (isset($rows[$locationsFieldsMapping['monthly_nth_n']])) {
+                        if ($this->validateScheduleNthN($rows[$locationsFieldsMapping['monthly_nth_n']])) {
+                            $curSchedule .= '"nth":{"n":'.$rows[$locationsFieldsMapping['monthly_nth_n']].',';
                         } else {
-                            array_push($results['fail'], "$iRow --> The parameter sched_monthly_mode is not set");
+                            array_push($results['fail'], "$iRow --> Wrong parameter sched_nth_n");
+                        }
+                        } else {
+                        array_push($results['fail'], "$iRow --> The parameter sched_nth_n is not set");
                         }
 
-                        if ('' != $monthlyMode) {
-                            switch ($monthlyMode) {
-                                case 'dates':
-                                    if (isset($rows[$locationsFieldsMapping['monthly_dates']])) {
-                                        if ($this->validateScheduleMonthlyDates($rows[$locationsFieldsMapping['monthly_dates']])) {
-                                            $curSchedule .= '"dates":['.$rows[$locationsFieldsMapping['monthly_dates']].']}';
-                                        } else {
-                                            array_push($results['fail'], "$iRow --> Wrong monthly dates");
-                                        }
-                                    }
-                                    break;
-                                case 'nth':
-                                    if (isset($rows[$locationsFieldsMapping['monthly_nth_n']])) {
-                                        if ($this->validateScheduleNthN($rows[$locationsFieldsMapping['monthly_nth_n']])) {
-                                            $curSchedule .= '"nth":{"n":'.$rows[$locationsFieldsMapping['monthly_nth_n']].',';
-                                        } else {
-                                            array_push($results['fail'], "$iRow --> Wrong parameter sched_nth_n");
-                                        }
-                                    } else {
-                                        array_push($results['fail'], "$iRow --> The parameter sched_nth_n is not set");
-                                    }
-
-                                    if ('' != $curSchedule) {
-                                        if (isset($rows[$locationsFieldsMapping['monthly_nth_what']])) {
-                                            if ($this->validateScheduleNthWhat($rows[$locationsFieldsMapping['monthly_nth_what']])) {
-                                                $curSchedule .= '"what":'.$rows[$locationsFieldsMapping['monthly_nth_what']].'}}';
-                                            } else {
-                                                array_push($results['fail'], "$iRow --> Wrong parameter sched_nth_what");
-                                            }
-                                        } else {
-                                            array_push($results['fail'], "$iRow --> The parameter sched_nth_what is not set");
-                                        }
-                                    }
-                                    break;
+                        if ('' != $curSchedule) {
+                        if (isset($rows[$locationsFieldsMapping['monthly_nth_what']])) {
+                            if ($this->validateScheduleNthWhat($rows[$locationsFieldsMapping['monthly_nth_what']])) {
+                                $curSchedule .= '"what":'.$rows[$locationsFieldsMapping['monthly_nth_what']].'}}';
+                            } else {
+                                array_push($results['fail'], "$iRow --> Wrong parameter sched_nth_what");
                             }
+                        } else {
+                            array_push($results['fail'], "$iRow --> The parameter sched_nth_what is not set");
+                        }
                         }
                         break;
-                    default:
-                        $curSchedule = '';
+                        }
+                        }
                         break;
+                default:
+                    $curSchedule = '';
+                    break;
                 }
             }
 

Braces +12 added lines, -4 removed lines

@@ -390,8 +390,12 @@ 
     public static function validateScheduleEnable($scheduleEnabled)
     {
         if (is_string($scheduleEnabled)) {
-            if (strtolower($scheduleEnabled)=="true") $scheduleEnabled = true;
-            if (strtolower($scheduleEnabled)=="false") $scheduleEnabled = false;
+            if (strtolower($scheduleEnabled)=="true") {
+            	$scheduleEnabled = true;
+            }
+            if (strtolower($scheduleEnabled)=="false") {
+            	$scheduleEnabled = false;
+            }
         }
 
         $schedEnables = [true, false,];
@@ -418,7 +422,9 @@ 
 
     public static function validateScheduleWeekDays($scheduleWeekDays)
     {
-        if (is_bool($scheduleWeekDays)) return false;
+        if (is_bool($scheduleWeekDays)) {
+        	return false;
+        }
 
         $weekdays = explode(',', $scheduleWeekDays);
         $weekdaysSize = sizeof($weekdays);
@@ -458,7 +464,9 @@ 
 
     public static function validateScheduleMonthlyDates($scheduleMonthlyDates)
     {
-        if (is_bool($scheduleMonthlyDates)) return false;
+        if (is_bool($scheduleMonthlyDates)) {
+        	return false;
+        }
 
         $monthlyDates = explode(',', $scheduleMonthlyDates);
         $monthlyDatesSize = sizeof($monthlyDates);

src/Route4Me/AddressNote.php

Braces +3 added lines, -1 removed lines

@@ -178,7 +178,9 @@
             'method'    => 'GET',
         ]);
 
-        if (is_null($result) || !is_array($result)) return null;
+        if (is_null($result) || !is_array($result)) {
+        	return null;
+        }
 
         foreach ($result as $custNoteType) {
             if (isset($custNoteType["note_custom_type"]) && $custNoteType["note_custom_type"] == $params) {

src/Route4Me/MyQ.php

Switch Indentation +10 added lines, -10 removed lines

@@ -209,16 +209,16 @@
 
             foreach ($device->Attributes as $attr) {
                 switch ($attr->AttributeDisplayName) {
-                    case 'desc':
-                        $this->_doorName = $attr->Value;
-                        break;
-                    case 'doorstate':
-                        $this->_doorState = $attr->Value;
-                        // UpdatedTime is a timestamp in ms, so we truncate
-                        $this->_doorStateTime = (int) $attr->UpdatedTime / 1000;
-                        break;
-                    default:
-                        continue;
+                case 'desc':
+                    $this->_doorName = $attr->Value;
+                    break;
+                case 'doorstate':
+                    $this->_doorState = $attr->Value;
+                    // UpdatedTime is a timestamp in ms, so we truncate
+                    $this->_doorStateTime = (int) $attr->UpdatedTime / 1000;
+                    break;
+                default:
+                    continue;
                 }
             }
         }

src/Route4Me/Vehicles/VehiclesResponseV4.php

Braces +3 added lines, -1 removed lines

@@ -83,7 +83,9 @@
         $vehiclesResponse = new self();
 
         foreach ($params as $key => $value) {
-            if (is_null(Common::getValue($params, $key))) continue;
+            if (is_null(Common::getValue($params, $key))) {
+            	continue;
+            }
             if (property_exists($vehiclesResponse, $key)) {
                 $vehiclesResponse->$key = $value;
             }

src/Route4Me/Vehicles/VehicleCreateResponseV4.php

Braces +3 added lines, -1 removed lines

@@ -137,7 +137,9 @@
         $vehicleCreateResponse = new self();
 
         foreach ($params as $key => $value) {
-            if (is_null(Common::getValue($params, $key))) continue;
+            if (is_null(Common::getValue($params, $key))) {
+            	continue;
+            }
             if (property_exists($vehicleCreateResponse, $key)) {
                 $vehicleCreateResponse->$key = $value;
             }