H4ad / laravel-scheduler

src/Scheduler/Scheduler.php

Braces +36 added lines, -24 removed lines

@@ -47,8 +47,9 @@ 
      */
     public function hasScheduleBetween($model_type, $start_at, $end_at)
     {
-        if(!Config::get('scheduler.enable_schedule_conflict'))
-            return false;
+        if(!Config::get('scheduler.enable_schedule_conflict')) {
+                    return false;
+        }
 
         return !is_null(
             Schedule::latest()
@@ -103,23 +104,27 @@ 
                 $start = Carbon::parse($schedule->start_at);
                 $begin = Carbon::parse($start->toDateString());
 
-                if($begin->greaterThan($today))
-                    break;
+                if($begin->greaterThan($today)) {
+                                    break;
+                }
 
-                if($begin->notEqualTo($today))
-                    continue;
+                if($begin->notEqualTo($today)) {
+                                    continue;
+                }
 
                 $end = Carbon::parse($schedule->end_at);
 
-                if($this->isShouldntAdd($opening, $closing, $start, $end))
-                    $add = false;
+                if($this->isShouldntAdd($opening, $closing, $start, $end)) {
+                                    $add = false;
+                }
             }
 
-            if($add && $closing->lessThanOrEqualTo($closingTime))
-                $livres[] = [
+            if($add && $closing->lessThanOrEqualTo($closingTime)) {
+                            $livres[] = [
                     'start_at' => $opening,
                     'end_at' => $closing
                 ];
+            }
 
             $openingTime->addMinutes($durationMinutes);
         }
@@ -156,20 +161,23 @@ 
      */
     public function validateSchedule($model_type, $start_at, $end_at = null, $status = null)
     {
-        if(!Config::get('scheduler.enable_schedule_without_end') && is_null($end_at))
-            throw new CantAddWithoutEnd;
+        if(!Config::get('scheduler.enable_schedule_without_end') && is_null($end_at)) {
+                    throw new CantAddWithoutEnd;
+        }
 
         $start_at  = $this->parseToCarbon($start_at);
 
         if(!is_null($end_at)) {
             $end_at = $this->parseToCarbon($end_at, $start_at);
 
-            if($start_at->greaterThan($end_at))
-                throw new EndCantBeforeStart;
+            if($start_at->greaterThan($end_at)) {
+                            throw new EndCantBeforeStart;
+            }
         }
 
-        if($this->hasScheduleBetween($model_type, $start_at, $end_at ?? $start_at))
-            throw new CantAddWithSameStartAt;
+        if($this->hasScheduleBetween($model_type, $start_at, $end_at ?? $start_at)) {
+                    throw new CantAddWithSameStartAt;
+        }
 
         return compact('model_type', 'start_at', 'end_at', 'status');
     }
@@ -185,14 +193,17 @@ 
      */
     public function parseToCarbon($date, $reference = null)
     {
-        if($date instanceof Carbon)
-            return $date;
+        if($date instanceof Carbon) {
+                    return $date;
+        }
 
-        if(is_string($date))
-            return Carbon::parse($date);
+        if(is_string($date)) {
+                    return Carbon::parse($date);
+        }
 
-        if(is_int($date) && !is_null($reference))
-            return Carbon::parse($reference->toDateTimeString())->addMinutes($date);
+        if(is_int($date) && !is_null($reference)) {
+                    return Carbon::parse($reference->toDateTimeString())->addMinutes($date);
+        }
 
         throw new IntInvalidArgument;
     }
@@ -205,8 +216,9 @@ 
      */
     public function parseToSchedule($value)
     {
-        if(is_int($value))
-            return Schedule::find($value);
+        if(is_int($value)) {
+                    return Schedule::find($value);
+        }
 
         return Schedule::byStartAt($value)->first();
     }

src/Scheduler/Traits/SchedulerModelTrait.php

Braces +9 added lines, -6 removed lines

@@ -110,16 +110,19 @@
 	 */
 	public function removeSchedule($schedule)
 	{
-		if(!Config::get('scheduler.enable_schedule_conflict') && !is_int($schedule))
-			throw new CantRemoveByDate;
+		if(!Config::get('scheduler.enable_schedule_conflict') && !is_int($schedule)) {
+					throw new CantRemoveByDate;
+		}
 
 		$schedule = Scheduler::parseToSchedule($schedule);
 
-		if(!($schedule instanceof Model))
-			throw (new ModelNotFound)->setValues(Schedule::class);
+		if(!($schedule instanceof Model)) {
+					throw (new ModelNotFound)->setValues(Schedule::class);
+		}
 
-		if($schedule->model_type != self::class || $schedule->model_id != $this->getKey())
-			throw new DoesNotBelong;
+		if($schedule->model_type != self::class || $schedule->model_id != $this->getKey()) {
+					throw new DoesNotBelong;
+		}
 
 		return $schedule->delete();
 	}