H4ad / laravel-scheduler

src/Scheduler/Exceptions/CustomException.php

Braces +6 added lines, -4 removed lines

@@ -88,8 +88,9 @@ 
      */
     protected function parseValues()
     {
-        if(is_array($this->attributes) && is_array($this->values))
-            return collect($this->attributes)->combine($this->values)->all();
+        if(is_array($this->attributes) && is_array($this->values)) {
+                    return collect($this->attributes)->combine($this->values)->all();
+        }
 
         return [ $this->attributes => $this->values ?? $this->isLower() ];
     }
@@ -111,8 +112,9 @@ 
      */
     protected function getAlias()
     {
-        if(is_object($this->model))
-            $this->model = get_class($this->model);
+        if(is_object($this->model)) {
+                    $this->model = get_class($this->model);
+        }
 
         return collect(trans('scheduler::exceptions.aliases.'. $this->aliastype))->search($this->model) ?: 'Recurso';
     }

src/Scheduler/Traits/SchedulerModelTrait.php

Braces +30 added lines, -20 removed lines

@@ -70,20 +70,23 @@ 
 	 */
 	public function addSchedule($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(Scheduler::hasScheduleBetween(self::class, $start_at, $end_at ?? $start_at))
-			throw new CantAddWithSameStartAt;
+		if(Scheduler::hasScheduleBetween(self::class, $start_at, $end_at ?? $start_at)) {
+					throw new CantAddWithSameStartAt;
+		}
 
 		$model_id = $this->getKey();
 		$model_type = self::class;
@@ -131,14 +134,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;
 	}
@@ -151,8 +157,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();
 	}
@@ -171,16 +178,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 = $this->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();
 	}

src/Scheduler/Models/Schedule.php

Braces +9 added lines, -6 removed lines

@@ -59,14 +59,17 @@
      */
     public function parseStatusKey($status)
     {
-    	if(is_int($status))
-    		$status =  ScheduleStatus::find($status);
+    	if(is_int($status)) {
+    	    		$status =  ScheduleStatus::find($status);
+    	}
 
-        if(is_string($status))
-        	$status = ScheduleStatus::where('name', $status)->first();
+        if(is_string($status)) {
+                	$status = ScheduleStatus::where('name', $status)->first();
+        }
 
-    	if(is_null($status))
-    		throw (new ModelNotFound)->setValues(ScheduleStatus::class);
+    	if(is_null($status)) {
+    	    		throw (new ModelNotFound)->setValues(ScheduleStatus::class);
+    	}
 
     	return ['status' => $status->id];
     }

src/Scheduler/Scheduler.php

Braces +15 added lines, -10 removed lines

@@ -43,8 +43,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()
@@ -99,23 +100,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);
         }