Fix code styling

Author: jbrooksuk

database/seeders/DatabaseSeeder.php

@@ -19,7 +19,6 @@
 use Cachet\Settings\AppSettings;
 use Cachet\Settings\CustomizationSettings;
 use Cachet\Settings\ThemeSettings;
-use Illuminate\Database\Eloquent\Model;
 use Illuminate\Database\Seeder;
 use Illuminate\Support\Arr;
 use Illuminate\Support\Facades\DB;
@@ -67,7 +66,7 @@ public function run(): void
             'message' => 'We will be conducting maintenance on our documentation servers. You may experience degraded performance during this time.',
             'scheduled_at' => now()->addHours(24),
             'completed_at' => null,
-            /** @phpstan-ignore-next-line argument.type */
+        /** @phpstan-ignore-next-line argument.type */
         ]), function (Schedule $schedule) use ($user) {
             $update = new Update([
                 'message' => <<<'EOF'

src/Actions/Schedule/CreateSchedule.php

@@ -5,7 +5,6 @@
 use Cachet\Data\Requests\Schedule\CreateScheduleRequestData;
 use Cachet\Data\Requests\Schedule\ScheduleComponentRequestData;
 use Cachet\Models\Schedule;
-use Illuminate\Database\Eloquent\Model;
 
 class CreateSchedule
 {

src/Filters/ScheduleStatusFilter.php

@@ -11,27 +11,26 @@ class ScheduleStatusFilter implements Filter
 {
     /**
      * @param  ScheduleBuilder  $query
-     * @param  $value
-     * @param  string  $property
      * @return void
      */
     public function __invoke(Builder $query, $value, string $property)
     {
         if (is_array($value)) {
             $this->multiValueFilter($value, $query);
-            return ;
+
+            return;
         }
         $enum = $this->toEnum($value);
         if (! $enum instanceof ScheduleStatusEnum) {
-            return ;
+            return;
         }
         $this->toQuery($enum, $query);
     }
 
     public function multiValueFilter(array $value, ScheduleBuilder $query)
     {
         $query->where(function (ScheduleBuilder $query) use ($value) {
-            foreach($value as $status) {
+            foreach ($value as $status) {
                 $status = $this->toEnum($status);
                 if (! $status instanceof ScheduleStatusEnum) {
                     continue;
@@ -44,10 +43,9 @@ public function multiValueFilter(array $value, ScheduleBuilder $query)
         });
     }
 
-
     protected function toQuery(ScheduleStatusEnum $value, ScheduleBuilder $query)
     {
-        match($value) {
+        match ($value) {
             ScheduleStatusEnum::complete => $query->inThePast(),
             ScheduleStatusEnum::in_progress => $query->inProgress(),
             ScheduleStatusEnum::upcoming => $query->inTheFuture(),
@@ -56,10 +54,10 @@ protected function toQuery(ScheduleStatusEnum $value, ScheduleBuilder $query)
 
     protected function toEnum($value)
     {
-        if( ! $value instanceof ScheduleStatusEnum && is_numeric($value)) {
+        if (! $value instanceof ScheduleStatusEnum && is_numeric($value)) {
             return ScheduleStatusEnum::tryFrom($value);
         }
+
         return $value;
     }
-
 }

src/Models/Schedule.php

@@ -111,15 +111,14 @@ public function formattedMessage(): string
     /**
      * Create a new Eloquent query builder for the model.
      *
-     * @param \Illuminate\Database\Query\Builder $query
+     * @param  \Illuminate\Database\Query\Builder  $query
      * @return \Cachet\QueryBuilders\ScheduleBuilder
      */
     public function newEloquentBuilder($query)
     {
         return new ScheduleBuilder($query);
     }
 
-
     /**
      * Create a new factory instance for the model.
      */

src/QueryBuilders/ScheduleBuilder.php

@@ -7,33 +7,32 @@
 use Illuminate\Database\Eloquent\Builder;
 
 /**
- * @inheritDoc
+ * {@inheritDoc}
+ *
  * @method static \Cachet\QueryBuilders\ScheduleBuilder incomplete()
  * @method static \Cachet\QueryBuilders\ScheduleBuilder inProgress()
  * @method static \Cachet\QueryBuilders\ScheduleBuilder inTheFuture()
  * @method static \Cachet\QueryBuilders\ScheduleBuilder inThePast()
+ *
  * @mixin Schedule
  */
 class ScheduleBuilder extends Builder
 {
     /**
      * Scope schedules that are incomplete.
-     *
-     * @return self
      */
     public function incomplete(): self
     {
         $this->where(function (ScheduleBuilder $query) {
             $query->where('completed_at', '>=', Carbon::now())
                 ->orWhereNull('completed_at');
         });
+
         return $this;
     }
 
     /**
      * Scope schedules that are in progress.
-     *
-     * @return self
      */
     public function inProgress(): self
     {
@@ -42,29 +41,28 @@ public function inProgress(): self
                 $query->where('completed_at', '>=', Carbon::now())
                     ->orWhereNull('completed_at');
             });
+
         return $this;
     }
 
     /**
      * Scopes schedules to those in the future.
-     *
-     * @return self
      */
     public function inTheFuture(): self
     {
         $this->where('scheduled_at', '>=', Carbon::now());
+
         return $this;
     }
 
     /**
      * Scopes schedules to those scheduled in the past.
-     *
-     * @return self
      */
     public function inThePast(): self
     {
         $this->where('completed_at', '<=', Carbon::now())
             ->whereNotNull('completed_at');
+
         return $this;
     }
 }

tests/Feature/Api/ScheduleTest.php

@@ -168,10 +168,10 @@
 
     $query = http_build_query([
         'filter' => [
-            'status' => implode(",", [
+            'status' => implode(',', [
                 \Cachet\Enums\ScheduleStatusEnum::in_progress->value,
                 \Cachet\Enums\ScheduleStatusEnum::complete->value,
-                ]
+            ]
             ),
         ],
     ]);