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 +33 added lines, -22 removed lines

@@ -51,24 +51,30 @@ 
 	 */
 	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;
+		}
 
-		if(is_string($start_at))
-			$start_at = Carbon::parse($start_at);
+		if(is_string($start_at)) {
+					$start_at = Carbon::parse($start_at);
+		}
 
-		if(is_string($end_at))
-			$end_at = Carbon::parse($end_at);
+		if(is_string($end_at)) {
+					$end_at = Carbon::parse($end_at);
+		}
 
-		if(is_int($end_at))
-			$end_at = Carbon::parse($start_at->toDateTimeString())->addMinutes($end_at);
+		if(is_int($end_at)) {
+					$end_at = Carbon::parse($start_at->toDateTimeString())->addMinutes($end_at);
+		}
 
-		if(Config::get('scheduler.enable_schedule_conflict'))
-			if(Scheduler::hasScheduleBetween($start_at, $end_at ?? $start_at))
+		if(Config::get('scheduler.enable_schedule_conflict')) {
+					if(Scheduler::hasScheduleBetween($start_at, $end_at ?? $start_at))
 				throw new CantAddWithSameStartAt;
+		}
 
-		if($start_at->greaterThan($end_at) && !is_null($end_at))
-			throw new EndCantBeforeStart;
+		if($start_at->greaterThan($end_at) && !is_null($end_at)) {
+					throw new EndCantBeforeStart;
+		}
 
 		$model_id = $this->getKey();
 		$model_type = self::class;
@@ -90,20 +96,25 @@ 
 	 */
 	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;
+		}
 
-		if(is_int($schedule))
-			$schedule = Schedule::find($schedule);
+		if(is_int($schedule)) {
+					$schedule = Schedule::find($schedule);
+		}
 
-		if(is_string($schedule) || $schedule instanceof Carbon)
-			$schedule = Schedule::byStartAt($schedule)->first();
+		if(is_string($schedule) || $schedule instanceof Carbon) {
+					$schedule = Schedule::byStartAt($schedule)->first();
+		}
 
-		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)
-			throw new DoesNotBelong;
+		if($schedule->model_type != self::class) {
+					throw new DoesNotBelong;
+		}
 
 		return $schedule->delete();
 	}

src/Scheduler/Scheduler.php

Braces +15 added lines, -10 removed lines

@@ -84,29 +84,34 @@
             $add = true;
 
             foreach (Schedule::orderBy('start_at', 'DESC')->cursor() as $schedule) {
-            	if($schedule->model_type != $model_type)
-            		continue;
+            	if($schedule->model_type != $model_type) {
+            	            		continue;
+            	}
 
                 $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($start <= Carbon::parse($openingTime->toDateTimeString())
-                && $end >= Carbon::parse($openingTime->toDateTimeString())->addMinutes($durationMinutes))
-                    $add = false;
+                && $end >= Carbon::parse($openingTime->toDateTimeString())->addMinutes($durationMinutes)) {
+                                    $add = false;
+                }
             }
 
-            if($add)
-                $livres[] = [
+            if($add) {
+                            $livres[] = [
                     'start_at' => Carbon::parse($openingTime->toDateTimeString()),
                     'end_at' => Carbon::parse($openingTime->toDateTimeString())->addMinutes($durationMinutes)
                 ];
+            }
 
             $openingTime->addMinutes($durationMinutes);
         }

src/Scheduler/Models/Schedule.php

Braces +6 added lines, -4 removed lines

@@ -52,13 +52,15 @@
      */
     public function parseStatusKey($key)
     {
-    	if(is_int($key))
-    		return ['status' => $key];
+    	if(is_int($key)) {
+    	    		return ['status' => $key];
+    	}
 
     	$status = ScheduleStatus::where('name', $key)->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];
     }