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/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];
     }

src/Scheduler/Scheduler.php

Braces +18 added lines, -12 removed lines

@@ -42,8 +42,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()
@@ -88,29 +89,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/Traits/SchedulerModelTrait.php

Braces +30 added lines, -20 removed lines

@@ -69,17 +69,20 @@ 
 	 */
 	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);
 		$end_at = $this->parseToCarbon($end_at, $start_at);
 
-		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;
+		}
 
-		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;
@@ -96,11 +99,13 @@ 
 	 */
 	public function parseToCarbon($date, $reference = null)
 	{
-		if(is_string($date))
-			return Carbon::parse($date);
+		if(is_string($date)) {
+					return Carbon::parse($date);
+		}
 
-		if(is_int($date))
-			return Carbon::parse($reference->toDateTimeString())->addMinutes($date);
+		if(is_int($date)) {
+					return Carbon::parse($reference->toDateTimeString())->addMinutes($date);
+		}
 
 		return $date;
 	}
@@ -113,11 +118,13 @@ 
 	 */
 	public function parseToSchedule($value)
 	{
-		if(is_int($value))
-			return Schedule::find($value);
+		if(is_int($value)) {
+					return Schedule::find($value);
+		}
 
-		if(is_string($value) || $value instanceof Carbon)
-			return Schedule::byStartAt($value)->first();
+		if(is_string($value) || $value instanceof Carbon) {
+					return Schedule::byStartAt($value)->first();
+		}
 
 		return null;
 	}
@@ -136,16 +143,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();
 	}