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';
     }

Spacing +10 added lines, -10 removed lines

@@ -64,7 +64,7 @@ 
      *
      * @param mixed $model
      */
-    public function __construct($model = 'foo')
+    public function __construct ($model = 'foo')
     {
         $this->model = $model;
     }
@@ -74,12 +74,12 @@ 
      *
      * @return \Illuminate\Http\JsonResponse
      */
-    public function render()
+    public function render ()
     {
         return response()->
             /** @scrutinizer ignore-call */
             json([
-            'messages' => trans('scheduler::exceptions.'. $this->trans, $this->parseValues())
+            'messages' => trans('scheduler::exceptions.' . $this->trans, $this->parseValues())
         ], $this->statusCode);
     }
 
@@ -88,9 +88,9 @@ 
      *
      * @return array
      */
-    protected function parseValues()
+    protected function parseValues ()
     {
-        if(is_array($this->attributes) && is_array($this->values))
+        if (is_array($this->attributes) && is_array($this->values))
             return collect($this->attributes)->combine($this->values)->all();
 
         return [ $this->attributes => $this->values ?? $this->isLower() ];
@@ -101,7 +101,7 @@ 
      *
      * @return string
      */
-    protected function isLower()
+    protected function isLower ()
     {
         return $this->lowercase ? strtolower($this->getAlias()) : $this->getAlias();
     }
@@ -111,12 +111,12 @@ 
      *
      * @return string
      */
-    protected function getAlias()
+    protected function getAlias ()
     {
-        if(is_object($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';
+        return collect(trans('scheduler::exceptions.aliases.' . $this->aliastype))->search($this->model) ?: 'Recurso';
     }
 
     /**
@@ -124,7 +124,7 @@ 
      *
      * @param mixed $values
      */
-    public function setValues($values)
+    public function setValues ($values)
     {
         $this->values = $values;
 

src/Scheduler/SchedulerServiceProvider.php

Spacing +9 added lines, -9 removed lines

@@ -17,17 +17,17 @@ 
      *
      * @return void
      */
-    public function boot()
+    public function boot ()
     {
         $this->publishes([
-            __DIR__.'/../config/config.php' => config_path('scheduler.php'),
+            __DIR__ . '/../config/config.php' => config_path('scheduler.php'),
         ]);
 
-        $this->loadMigrationsFrom(__DIR__.'/Migrations');
-        $this->loadTranslationsFrom(__DIR__.'/Translations', 'scheduler');
+        $this->loadMigrationsFrom(__DIR__ . '/Migrations');
+        $this->loadTranslationsFrom(__DIR__ . '/Translations', 'scheduler');
 
         $this->publishes([
-            __DIR__.'/Translations' => resource_path('lang/vendor/scheduler'),
+            __DIR__ . '/Translations' => resource_path('lang/vendor/scheduler'),
         ]);
     }
 
@@ -36,9 +36,9 @@ 
      *
      * @return void
      */
-    public function register()
+    public function register ()
     {
-        $this->app->singleton(Scheduler::class, function ($app) {
+        $this->app->singleton(Scheduler::class, function($app) {
             return new Scheduler($app);
         });
 
@@ -52,10 +52,10 @@ 
      *
      * @return void
      */
-    private function mergeConfig()
+    private function mergeConfig ()
     {
         $this->mergeConfigFrom(
-            __DIR__.'/../config/config.php', 'scheduler'
+            __DIR__ . '/../config/config.php', 'scheduler'
         );
     }
 }
\ No newline at end of file

src/Scheduler/Models/ScheduleStatus.php

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
     /**
      * Construtor para inicilizar a váriavel table.
      */
-    public function __construct(array $attributes = [])
+    public function __construct (array $attributes = [ ])
     {
         parent::__construct($attributes);
 

src/Scheduler/Facades/Scheduler.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
      *
      * @return string
      */
-    protected static function getFacadeAccessor()
+    protected static function getFacadeAccessor ()
     {
         return 'scheduler';
     }

src/Scheduler/Models/Schedule.php

Spacing +8 added lines, -8 removed lines

@@ -44,7 +44,7 @@ 
      *
      * @param int|string $status Pode ser passado o ID do status ou seu nome para seta-lo no horário.
      */
-    public function setStatus($name)
+    public function setStatus ($name)
     {
     	$this->fill($this->parseStatusKey($name))->save();
     }
@@ -57,18 +57,18 @@ 
      *
      * @throws \H4ad\Scheduler\Exceptions\ModelNotFound
      */
-    public function parseStatusKey($status)
+    public function parseStatusKey ($status)
     {
-    	if(is_int($status))
-    		$status =  ScheduleStatus::find($status);
+    	if (is_int($status))
+    		$status = ScheduleStatus::find($status);
 
-        if(is_string($status))
+        if (is_string($status))
         	$status = ScheduleStatus::where('name', $status)->first();
 
-    	if(is_null($status))
+    	if (is_null($status))
     		throw (new ModelNotFound)->setValues(ScheduleStatus::class);
 
-    	return ['status' => $status->id];
+    	return [ 'status' => $status->id ];
     }
 
     /**
@@ -78,7 +78,7 @@ 
      * @param \Carbon\Carbon|string $start_at
      * @return \Illuminate\Database\Eloquent\Builder
      */
-    public function scopeByStartAt($query, $start_at)
+    public function scopeByStartAt ($query, $start_at)
     {
         return $query->where('start_at', $start_at);
     }

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/Migrations/2018_08_18_225944_create_schedules_table.php

Spacing +3 added lines, -3 removed lines

@@ -20,9 +20,9 @@ 
      *
      * @return void
      */
-    public function up()
+    public function up ()
     {
-        Schema::create(Config::get('scheduler.schedules_table'), function (Blueprint $table) {
+        Schema::create(Config::get('scheduler.schedules_table'), function(Blueprint $table) {
             $table->increments('id');
             $table->string('model_type');
             $table->integer('model_id');
@@ -39,7 +39,7 @@ 
      *
      * @return void
      */
-    public function down()
+    public function down ()
     {
         Schema::drop(Config::get('scheduler.schedules_table'));
     }

src/Scheduler/Migrations/2018_08_18_225643_create_schedule_status_table.php

Spacing +3 added lines, -3 removed lines

@@ -20,9 +20,9 @@ 
      *
      * @return void
      */
-    public function up()
+    public function up ()
     {
-        Schema::create(Config::get('scheduler.schedule_status_table'), function (Blueprint $table) {
+        Schema::create(Config::get('scheduler.schedule_status_table'), function(Blueprint $table) {
             $table->increments('id');
             $table->string('name')->unique();
             $table->text('description')->nullable();
@@ -36,7 +36,7 @@ 
      *
      * @return void
      */
-    public function down()
+    public function down ()
     {
         Schema::drop(Config::get('scheduler.schedule_status_table'));
     }

src/Scheduler/Scheduler.php

Spacing +25 added lines, -25 removed lines

@@ -32,7 +32,7 @@ 
      *
      * @return void
      */
-    public function __construct($app)
+    public function __construct ($app)
     {
         $this->app = $app;
     }
@@ -45,9 +45,9 @@ 
      * @param string|\Carbon\Carbon $end_at
      * @return bool
      */
-    public function hasScheduleBetween($model_type, $start_at, $end_at)
+    public function hasScheduleBetween ($model_type, $start_at, $end_at)
     {
-        if(!Config::get('scheduler.enable_schedule_conflict'))
+        if (!Config::get('scheduler.enable_schedule_conflict'))
             return false;
 
         return !is_null(
@@ -69,7 +69,7 @@ 
      *                                         Se for nulo, ele busca a referencia da config.
      * @return array
      */
-    public function availableToday($model_type, $duration, $openingTime = null)
+    public function availableToday ($model_type, $duration, $openingTime = null)
     {
         return $this->availableOn($model_type, Carbon::now(), $duration, $openingTime);
     }
@@ -85,14 +85,14 @@ 
      *                                         Se for nulo, ele busca a referencia da config.
      * @return array
      */
-    public function availableOn($model_type, $today, $durationMinutes, $openingTime = null)
+    public function availableOn ($model_type, $today, $durationMinutes, $openingTime = null)
     {
         $openingTime = $openingTime ?? Carbon::parse(Config::get('scheduler.opening_time'))->setDateFrom($today);
         $closingTime = Carbon::parse(Config::get('scheduler.closing_time'))->setDateFrom($today);
 
-        $livres = [];
+        $livres = [ ];
         $today = Carbon::parse($today->toDateString());
-        while($openingTime <= $closingTime)
+        while ($openingTime <= $closingTime)
         {
             $add = true;
 
@@ -103,20 +103,20 @@ 
                 $start = Carbon::parse($schedule->start_at);
                 $begin = Carbon::parse($start->toDateString());
 
-                if($begin->greaterThan($today))
+                if ($begin->greaterThan($today))
                     break;
 
-                if($begin->notEqualTo($today))
+                if ($begin->notEqualTo($today))
                     continue;
 
                 $end = Carbon::parse($schedule->end_at);
 
-                if($this->isShouldntAdd($opening, $closing, $start, $end))
+                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
                 ];
@@ -136,7 +136,7 @@ 
      * @param  \Carbon\Carbon  $end
      * @return boolean
      */
-    private function isShouldntAdd($opening, $closing, $start, $end)
+    private function isShouldntAdd ($opening, $closing, $start, $end)
     {
         return $start <= $opening && $end >= $closing;
     }
@@ -154,21 +154,21 @@ 
      * @throws \H4ad\Scheduler\Exceptions\EndCantBeforeStart
      * @throws \H4ad\Scheduler\Exceptions\CantAddWithSameStartAt
      */
-    public function validateSchedule($model_type, $start_at, $end_at = null, $status = null)
+    public function validateSchedule ($model_type, $start_at, $end_at = null, $status = null)
     {
-        if(!Config::get('scheduler.enable_schedule_without_end') && is_null($end_at))
+        if (!Config::get('scheduler.enable_schedule_without_end') && is_null($end_at))
             throw new CantAddWithoutEnd;
 
-        $start_at  = $this->parseToCarbon($start_at);
+        $start_at = $this->parseToCarbon($start_at);
 
-        if(!is_null($end_at)) {
+        if (!is_null($end_at)) {
             $end_at = $this->parseToCarbon($end_at, $start_at);
 
-            if($start_at->greaterThan($end_at))
+            if ($start_at->greaterThan($end_at))
                 throw new EndCantBeforeStart;
         }
 
-        if($this->hasScheduleBetween($model_type, $start_at, $end_at ?? $start_at))
+        if ($this->hasScheduleBetween($model_type, $start_at, $end_at ?? $start_at))
             throw new CantAddWithSameStartAt;
 
         return compact('model_type', 'start_at', 'end_at', 'status');
@@ -183,15 +183,15 @@ 
      *
      * @throws \H4ad\Scheduler\Exceptions\IntInvalidArgument
      */
-    public function parseToCarbon($date, $reference = null)
+    public function parseToCarbon ($date, $reference = null)
     {
-        if($date instanceof Carbon)
+        if ($date instanceof Carbon)
             return $date;
 
-        if(is_string($date))
+        if (is_string($date))
             return Carbon::parse($date);
 
-        if(is_int($date) && !is_null($reference))
+        if (is_int($date) && !is_null($reference))
             return Carbon::parse($reference->toDateTimeString())->addMinutes($date);
 
         throw new IntInvalidArgument;
@@ -203,9 +203,9 @@ 
      * @param  \Carbon\Carbon|string|int $value Valor que representará a data ou o id a ser buscado.
      * @return \H4ad\Scheduler\Models\Schedule|null
      */
-    public function parseToSchedule($value)
+    public function parseToSchedule ($value)
     {
-        if(is_int($value))
+        if (is_int($value))
             return Schedule::find($value);
 
         return Schedule::byStartAt($value)->first();

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

Spacing +11 added lines, -11 removed lines

@@ -31,21 +31,21 @@ 
      * @param  string  $localKey
      * @return \Illuminate\Database\Eloquent\Relations\HasOne
      */
-    abstract public function hasOne($related, $foreignKey = null, $localKey = null);
+    abstract public function hasOne ($related, $foreignKey = null, $localKey = null);
 
 	/**
      * Get the value of the model's primary key.
      *
      * @return mixed
      */
-	abstract public function getKey();
+	abstract public function getKey ();
 
 	/**
      * Retorna apenas o horário que possui o mesmo [model_id] do [parent] dessa [trait].
      *
      * @return \Illuminate\Database\Eloquent\Relations\HasOne
      */
-	public function schedules()
+	public function schedules ()
 	{
 		return $this->hasOne(Config::get('scheduler.schedules_table'), 'model_id');
 	}
@@ -59,10 +59,10 @@ 
 	 * @param  int|null $status	Status desse horário ao ser agendado.
 	 * @return \H4ad\Scheduler\Models\Schedule
 	 */
-	public function addSchedule($start_at, $end_at = null, $status = null)
+	public function addSchedule ($start_at, $end_at = null, $status = null)
 	{
 		$data = Scheduler::validateSchedule(self::class, $start_at, $end_at, $status);
-		$data['model_id'] = $this->getKey();
+		$data[ 'model_id' ] = $this->getKey();
 
 		return Schedule::create($data);
 	}
@@ -76,7 +76,7 @@ 
      *                                         Se for nulo, ele busca a referencia da config.
 	 * @return array
 	 */
-	public function availableToday($duration = 0, $openingTime = null)
+	public function availableToday ($duration = 0, $openingTime = null)
 	{
 		return Scheduler::availableToday(self::class, $duration, $openingTime);
 	}
@@ -91,7 +91,7 @@ 
      *                                         Se for nulo, ele busca a referencia da config.
 	 * @return array
 	 */
-	public function availableOn($date, $duration = 0, $openingTime = null)
+	public function availableOn ($date, $duration = 0, $openingTime = null)
 	{
 		return Scheduler::availableOn(self::class, $date, $duration, $openingTime);
 	}
@@ -108,17 +108,17 @@ 
 	 * @throws \H4ad\Scheduler\Exceptions\CantRemoveByDate
 	 * @throws \H4ad\Scheduler\Exceptions\ModelNotFound
 	 */
-	public function removeSchedule($schedule)
+	public function removeSchedule ($schedule)
 	{
-		if(!Config::get('scheduler.enable_schedule_conflict') && !is_int($schedule))
+		if (!Config::get('scheduler.enable_schedule_conflict') && !is_int($schedule))
 			throw new CantRemoveByDate;
 
 		$schedule = Scheduler::parseToSchedule($schedule);
 
-		if(!($schedule instanceof Model))
+		if (!($schedule instanceof Model))
 			throw (new ModelNotFound)->setValues(Schedule::class);
 
-		if($schedule->model_type != self::class || $schedule->model_id != $this->getKey())
+		if ($schedule->model_type != self::class || $schedule->model_id != $this->getKey())
 			throw new DoesNotBelong;
 
 		return $schedule->delete();

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();
 	}