H4ad / laravel-scheduler

src/Scheduler/Traits/SchedulerModelTrait.php

Braces +27 added lines, -18 removed lines

@@ -69,20 +69,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;
@@ -99,11 +102,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) && !is_null($reference))
-			return Carbon::parse($reference->toDateTimeString())->addMinutes($date);
+		if(is_int($date) && !is_null($reference)) {
+					return Carbon::parse($reference->toDateTimeString())->addMinutes($date);
+		}
 
 		return $date;
 	}
@@ -116,8 +121,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();
 	}
@@ -136,16 +142,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();
 	}