fix: package.json & package-lock.json to reduce vulnerabilities

The following vulnerabilities are fixed with an upgrade:
- https://snyk.io/vuln/SNYK-JS-ANSIHTML-1296849

.github/workflows/build.yml

@@ -7,7 +7,7 @@ jobs:
     strategy:
       fail-fast: true
       matrix:
-        php-versions: ['8.0', '8.1']
+        php-versions: ['7.3', '7.4', '8.0', '8.1']
     name: PHP ${{ matrix.php-versions }}
     env:
       extensions: intl, pcov, mbstring
@@ -69,7 +69,6 @@ jobs:
       run: |
         php --version
         mysql --version
-        sleep 15
         # Downgrade composer version to 1.x
         composer install --dev --no-interaction --verbose
         cp .github/scripts/env.php env.php
@@ -80,7 +79,7 @@ jobs:
     - name: Run Tests
       run: |
         export PHP_CS_FIXER_IGNORE_ENV=1
-        #vendor/bin/php-cs-fixer fix --config=.php-cs-fixer.php -v --dry-run --diff --using-cache=no
+        vendor/bin/php-cs-fixer fix --config=.php-cs-fixer.php -v --dry-run --diff --using-cache=no
         vendor/bin/phpunit --debug --verbose
 
   # This runs after all of the tests, run have run. Creates a cleaned up version of the
@@ -88,7 +87,7 @@ jobs:
   artifacts:
     name: 'Create dev build'
     needs: build
-    runs-on: ubuntu-20.04
+    runs-on: 'ubuntu-18.04'
     if: github.ref == 'refs/heads/master' || github.ref == 'refs/heads/dev'
     steps:
       - name: Checkout
@@ -97,7 +96,7 @@ jobs:
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:
-          php-version: '8.0'
+          php-version: '7.4'
 
       - uses: olegtarasov/get-tag@v2.1
         id: tagName
@@ -150,7 +149,7 @@ jobs:
   release:
     name: 'Create Release'
     needs: build
-    runs-on: ubuntu-20.04
+    runs-on: 'ubuntu-18.04'
     if: startsWith(github.ref, 'refs/tags/')
     steps:
       - name: Checkout
@@ -159,7 +158,7 @@ jobs:
       - name: Setup PHP
         uses: shivammathur/setup-php@v2
         with:
-          php-version: 8.0
+          php-version: '7.4'
 
       - uses: olegtarasov/get-tag@v2.1
         id: tagName

.gitignore

@@ -22,7 +22,6 @@ storage/*.sqlite
 .env.*.php
 .env.php
 .env
-.env.*
 .env.bak
 env.php
 .env.generated
@@ -78,13 +77,3 @@ error_log
 /config.bak.php
 /VERSION
 sync.sh
-
-###> friendsofphp/php-cs-fixer ###
-/.php-cs-fixer.php
-/.php-cs-fixer.cache
-###< friendsofphp/php-cs-fixer ###
-
-###> phpunit/phpunit ###
-/phpunit.xml
-.phpunit.result.cache
-###< phpunit/phpunit ###

Dockerfile

@@ -1,15 +1,17 @@
-FROM php:8.1-fpm-alpine3.15
+FROM php:8.0.9-fpm-alpine3.14
 
 WORKDIR /var/www/
 
 # Setup composer
-COPY --from=composer:2.2.7 /usr/bin/composer /usr/local/bin/composer
+COPY --from=composer:2.1.5 /usr/bin/composer /usr/local/bin/composer
 
 RUN apk add gmp-dev icu-dev zlib-dev libpng-dev libzip-dev zip
+RUN curl --silent --show-error https://getcomposer.org/installer | php
 
 # Copy any config files in
 COPY resources/docker/php/ext-opcache.ini $PHP_INI_DIR/conf.d/
 COPY resources/docker/php/www.conf /usr/local/etc/php-fpm.d/www.conf
+RUN ln -sf /dev/stderr /var/log/fpm-error.log
 
 RUN docker-php-ext-install \
   calendar \
@@ -20,7 +22,7 @@ RUN docker-php-ext-install \
   bcmath \
   opcache \
   zip && \
-  docker-php-ext-enable pdo_mysql opcache bcmath zip intl
+  docker-php-ext-enable pdo_mysql opcache bcmath zip
 
 COPY . /var/www/
 RUN composer install \

Makefile

@@ -80,7 +80,7 @@ test:
 
 .PHONY: phpcs
 phpcs:
-	@PHP_CS_FIXER_IGNORE_ENV=1 vendor/bin/php-cs-fixer fix --config=.php-cs-fixer.php -v --dry-run --diff --using-cache=no
+	@vendor/bin/php-cs-fixer fix --config=.php_cs -v --diff --diff-format=udiff --dry-run
 
 .PHONY: replay-acars
 replay-acars:
@@ -102,7 +102,7 @@ reset-installer:
 
 .PHONY: docker-test
 docker-test:
-	@docker compose -f docker-compose.dev.yml up
+	@docker compose -f docker-compose.yml -f docker-compose.local.yml up
 
 .PHONY: docker-clean
 docker-clean:

README.md

@@ -11,7 +11,7 @@ A full distribution, with all of the composer dependencies, is available at this
 
 ### Requirements
 
-- PHP 8.0+, extensions:
+- PHP 7.3+, extensions:
   - cURL
   - JSON
   - mbstring
@@ -39,7 +39,7 @@ make docker-test
 
 # **OR** with docker-compose directly
 
-docker-compose -f docker-compose.yml -f docker-compose.dev.yml up
+docker-compose -f docker-compose.yml -f docker-compose.local.yml up
 ```
 
 Then go to `http://localhost`. If you're using dnsmasq, the `app` container is listening on `phpvms.test`, or you can add to your `/etc/hosts` file:

app/Console/Commands/AcarsReplay.php

@@ -17,25 +17,25 @@ class AcarsReplay extends Command
      *
      * @var string
      */
-    protected string $apiKey = 'testadminapikey';
+    protected $apiKey = 'testadminapikey';
 
     /**
      * For automatic updates, how many seconds to sleep between updates
      *
      * @var int
      */
-    protected int $sleepTime = 10;
+    protected $sleepTime = 10;
 
     /**
      * @var array key == update[callsign]
      *            value == PIREP ID
      */
-    protected array $pirepList = [];
+    protected $pirepList = [];
 
     /**
      * @var Client
      */
-    protected Client $httpClient;
+    protected $httpClient;
 
     /**
      * Return an instance of an HTTP client all ready to post

app/Console/Commands/CreateConfigs.php

@@ -2,11 +2,11 @@
 
 namespace App\Console\Commands;
 
+use App;
 use App\Contracts\Command;
 use App\Services\Installer\ConfigService;
 use App\Services\Installer\SeederService;
 use DatabaseSeeder;
-use Illuminate\Support\Facades\App;
 
 /**
  * Create the config files
@@ -16,8 +16,8 @@ class CreateConfigs extends Command
     protected $signature = 'phpvms:config {db_host} {db_name} {db_user} {db_pass}';
     protected $description = 'Create the config files';
 
-    private DatabaseSeeder $databaseSeeder;
-    private SeederService $seederSvc;
+    private $databaseSeeder;
+    private $seederSvc;
 
     public function __construct(DatabaseSeeder $databaseSeeder, SeederService $seederSvc)
     {

app/Console/Commands/CreateDatabase.php

@@ -11,11 +11,7 @@ class CreateDatabase extends Command
 {
     protected $signature = 'database:create {--reset} {--migrate} {--conn=?}';
     protected $description = 'Create a database';
-
-    /**
-     * @var Detector
-     */
-    protected Detector $os;
+    protected $os;
 
     /**
      * CreateDatabase constructor.

app/Console/Commands/DevCommands.php

@@ -24,8 +24,7 @@ class DevCommands extends Command
 {
     protected $signature = 'phpvms {cmd} {param?}';
     protected $description = 'Developer commands';
-
-    protected DatabaseService $dbSvc;
+    protected $dbSvc;
 
     /**
      * DevCommands constructor.

app/Console/Commands/DevInstall.php

@@ -13,7 +13,7 @@ class DevInstall extends Command
     protected $signature = 'phpvms:dev-install {--reset-db} {--reset-configs}';
     protected $description = 'Run a developer install and run the sample migration';
 
-    private \DatabaseSeeder $databaseSeeder;
+    private $databaseSeeder;
 
     public function __construct(\DatabaseSeeder $databaseSeeder)
     {

app/Console/Commands/ImportCsv.php

@@ -10,7 +10,7 @@ class ImportCsv extends Command
     protected $signature = 'phpvms:csv-import {type} {file}';
     protected $description = 'Import from a CSV file';
 
-    private ImportService $importer;
+    private $importer;
 
     /**
      * Import constructor.

app/Console/Commands/ProcessQueue.php

@@ -18,7 +18,7 @@ class ProcessQueue extends Command
     public function handle()
     {
         Artisan::call('queue:work', [
-            //'--sansdaemon'      => null,
+            '--sansdaemon'      => null,
             '--stop-when-empty' => null,
         ]);
 

app/Console/Commands/TestApi.php

@@ -8,11 +8,7 @@ use GuzzleHttp\Client;
 class TestApi extends Command
 {
     protected $signature = 'phpvms:test-api {apikey} {url}';
-
-    /**
-     * @var Client
-     */
-    protected Client $httpClient;
+    protected $httpClient;
 
     /**
      * Run dev related commands

app/Console/Commands/Version.php

@@ -10,14 +10,8 @@ class Version extends Command
 {
     protected $signature = 'phpvms:version {--write} {--base-only} {--write-full-version} {version?}';
 
-    /**
-     * @var VersionService
-     */
-    private VersionService $versionSvc;
+    private $versionSvc;
 
-    /**
-     * @param VersionService $versionSvc
-     */
     public function __construct(VersionService $versionSvc)
     {
         parent::__construct();

app/Console/Commands/YamlImport.php

@@ -12,11 +12,7 @@ class YamlImport extends Command
 {
     protected $signature = 'phpvms:yaml-import {files*}';
     protected $description = 'Developer commands';
-
-    /**
-     * @var DatabaseService
-     */
-    protected DatabaseService $dbSvc;
+    protected $dbSvc;
 
     /**
      * YamlImport constructor.

app/Contracts/Award.php

@@ -37,10 +37,10 @@ abstract class Award
      */
 
     /** @var \App\Models\Award|null */
-    protected ?AwardModel $award;
+    protected $award;
 
     /** @var \App\Models\User|null */
-    protected ?User $user;
+    protected $user;
 
     public function __construct(AwardModel $award = null, User $user = null)
     {
@@ -69,9 +69,9 @@ abstract class Award
     /**
      * Add the award to this user, if they don't already have it
      *
-     * @return bool|UserAward|null
+     * @return bool|UserAward
      */
-    protected function addAward(): bool|UserAward|null
+    protected function addAward()
     {
         $w = [
             'user_id'  => $this->user->id,

app/Contracts/Controller.php

@@ -84,7 +84,6 @@ abstract class Controller extends \Illuminate\Routing\Controller
      *
      * @param $message
      * @param null|mixed $count
-     * @param mixed      $attrs
      *
      * @return \Illuminate\Http\JsonResponse
      */

app/Contracts/Enum.php

@@ -7,14 +7,14 @@ namespace App\Contracts;
  */
 abstract class Enum
 {
-    protected static array $cache = [];
-    protected static array $codes = [];
-    protected static array $labels = [];
+    protected static $cache = [];
+    protected static $codes = [];
+    protected static $labels = [];
 
     /**
      * @var int
      */
-    protected int $value;
+    protected $value;
 
     /**
      * Create an instance of this Enum
@@ -29,9 +29,9 @@ abstract class Enum
     /**
      * Return the value that's been set if this is an instance
      *
-     * @return int|null
+     * @return mixed
      */
-    final public function getValue(): ?int
+    final public function getValue()
     {
         return $this->value;
     }
@@ -41,9 +41,9 @@ abstract class Enum
      *
      * @param $value
      *
-     * @return string
+     * @return mixed
      */
-    final public static function label($value): string
+    final public static function label($value)
     {
         if (isset(static::$labels[$value])) {
             $val = static::$labels[$value];
@@ -53,8 +53,6 @@ abstract class Enum
 
             return $val;
         }
-
-        return $value;
     }
 
     /**
@@ -89,11 +87,11 @@ abstract class Enum
      *
      * @return false|int|string
      */
-    public static function convertToCode($value): bool|int|string|null
+    public static function convertToCode($value)
     {
         $value = (int) $value;
         if (!array_key_exists($value, static::$codes)) {
-            return null;
+            return;
         }
 
         return static::$codes[$value];

app/Contracts/Factory.php

@@ -1,9 +0,0 @@
-<?php
-
-namespace App\Contracts;
-
-use Illuminate\Database\Eloquent\Factories\Factory as EloquentFactory;
-
-abstract class Factory extends EloquentFactory
-{
-}

app/Contracts/Repository.php

@@ -133,10 +133,7 @@ abstract class Repository extends BaseRepository
         $page = (int) request()->query('page', 1);
 
         $results = $this->model->{$method}($limit, $columns, 'page', $page);
-
-        $qs = request()->except(['page', 'user']);
-        $results->appends($qs);
-
+        $results->appends(app('request')->query());
         $this->resetModel();
 
         return $this->parserResult($results);

app/Contracts/Unit.php

@@ -3,6 +3,7 @@
 namespace App\Contracts;
 
 use ArrayAccess;
+use PhpUnitsOfMeasure\Exception\UnknownUnitOfMeasure;
 
 /**
  * Abstract unit wrapper
@@ -10,89 +11,45 @@ use ArrayAccess;
 class Unit implements ArrayAccess
 {
     /**
-     * The localized unit the user wants it displayed in
+     * The unit this is kept as
      */
-    public string $localUnit;
+    public $unit;
 
     /**
-     * The unit that this value is stored in locally
+     * All of the units of this class
      */
-    public string $internalUnit;
-
-    /**
-     * All of the units of this class which are reported in an API response
-     */
-    public array $units;
+    public $units;
 
     /**
      * Holds an instance of the PhpUnit type
+     *
+     * @var \PhpUnitsOfMeasure\AbstractPhysicalQuantity
      */
-    protected mixed $instance;
+    protected $instance;
 
     /**
      * Units that are included as part of the REST response
      */
-    public array $responseUnits = [];
+    public $responseUnits = [];
 
     /**
-     * Factory method for creating a new unit type
-     *
-     * @param mixed  $value
-     * @param string $unit
-     *
-     * @throws \Exception
-     *
-     * @return Unit
+     * @return mixed
      */
-    public static function make(mixed $value, string $unit): self
+    public function value()
     {
-        if ($value instanceof self) {
-            return $value;
-        }
-
-        return new static($value, $unit);
-    }
-
-    /**
-     * Return the value in an internal format
-     *
-     * @param int|null $round Optional value to round to
-     *
-     * @return float|null
-     */
-    public function internal(?int $round = null): ?float
-    {
-        return $this->toUnit($this->internalUnit, $round);
-    }
-
-    /**
-     * Return the value in the localized format
-     *
-     * @param int|null $round Optional value to round to
-     *
-     * @return float|null
-     */
-    public function local(?int $round = null): ?float
-    {
-        return $this->toUnit($this->localUnit, $round);
+        return $this->__toString();
     }
 
     /**
      * Just call toUnit() on the PhpUnitOfMeasure instance
      *
-     * @param string   $unit
-     * @param int|null $round Optional value to round to
+     * @param string $unit
      *
-     * @return float|null
+     * @return mixed
      */
-    public function toUnit(string $unit, ?int $round = null): ?float
+    public function toUnit($unit)
     {
-        $val = $this->instance->toUnit($unit);
-        if ($round === null) {
-            return $val;
-        }
-
-        return round($val, $round);
+        return $this->instance->toUnit($unit);
     }
 
     /**
@@ -102,7 +59,7 @@ class Unit implements ArrayAccess
     {
         $response = [];
         foreach ($this->responseUnits as $unit) {
-            $response[$unit] = round($this->instance->toUnit($unit), 2);
+            $response[$unit] = $this[$unit] ?? 0;
         }
 
         return $response;
@@ -115,7 +72,7 @@ class Unit implements ArrayAccess
      *
      * @return bool
      */
-    public function offsetExists($offset): bool
+    public function offsetExists($offset)
     {
         return $this->offsetGet($offset) !== null;
     }
@@ -123,15 +80,19 @@ class Unit implements ArrayAccess
     /**
      * Implements ArrayAccess
      *
-     * @param $unit
+     * @param $offset
      *
-     * @return float|null
+     * @return mixed
      */
-    public function offsetGet($unit): ?float
+    public function offsetGet($offset)
     {
-        $value = $this->instance->toUnit($unit);
-        if (!$value) {
-            return null;
+        try {
+            $value = $this->instance->toUnit($offset);
+            if (!$value) {
+                return;
+            }
+        } catch (UnknownUnitOfMeasure $e) {
+            return;
         }
 
         return round($value, 2);
@@ -163,6 +124,6 @@ class Unit implements ArrayAccess
      */
     public function __toString()
     {
-        return (string) $this->offsetGet($this->localUnit);
+        return (string) $this->offsetGet($this->unit);
     }
 }

app/Cron/Hourly/ClearExpiredSimbrief.php

@@ -12,7 +12,7 @@ use Illuminate\Support\Facades\Log;
  */
 class ClearExpiredSimbrief extends Listener
 {
-    private SimBriefService $simbriefSvc;
+    private $simbriefSvc;
 
     public function __construct(SimBriefService $simbriefSvc)
     {

app/Cron/Monthly/ApplyExpenses.php

@@ -13,7 +13,7 @@ use Illuminate\Support\Facades\Log;
  */
 class ApplyExpenses extends Listener
 {
-    private RecurringFinanceService $financeSvc;
+    private $financeSvc;
 
     /**
      * ApplyExpenses constructor.

app/Cron/Nightly/ApplyExpenses.php

@@ -13,7 +13,7 @@ use Illuminate\Support\Facades\Log;
  */
 class ApplyExpenses extends Listener
 {
-    private RecurringFinanceService $financeSvc;
+    private $financeSvc;
 
     /**
      * ApplyExpenses constructor.

app/Cron/Nightly/NewVersionCheck.php

@@ -9,7 +9,7 @@ use Illuminate\Support\Facades\Log;
 
 class NewVersionCheck extends Listener
 {
-    private VersionService $versionSvc;
+    private $versionSvc;
 
     /**
      * @param VersionService $versionSvc

app/Cron/Nightly/PilotLeave.php

@@ -12,7 +12,7 @@ use Illuminate\Support\Facades\Log;
  */
 class PilotLeave extends Listener
 {
-    private UserService $userSvc;
+    private $userSvc;
 
     /**
      * PilotLeave constructor.

app/Cron/Nightly/RecalculateBalances.php

@@ -6,14 +6,14 @@ use App\Contracts\Listener;
 use App\Events\CronNightly;
 use App\Models\Journal;
 use App\Repositories\JournalRepository;
-use Illuminate\Support\Facades\Log;
+use Log;
 
 /**
  * This recalculates the balances on all of the journals
  */
 class RecalculateBalances extends Listener
 {
-    private JournalRepository $journalRepo;
+    private $journalRepo;
 
     /**
      * Nightly constructor.

app/Cron/Nightly/RecalculateStats.php

@@ -13,8 +13,8 @@ use Illuminate\Support\Facades\Log;
  */
 class RecalculateStats extends Listener
 {
-    private AircraftService $aircraftSvc;
-    private UserService $userSvc;
+    private $aircraftSvc;
+    private $userSvc;
 
     public function __construct(AircraftService $aircraftSvc, UserService $userSvc)
     {

app/Database/factories/AcarsFactory.php

@@ -1,43 +1,23 @@
 <?php
 
-/** @noinspection PhpIllegalPsrClassPathInspection */
+use Faker\Generator as Faker;
 
-namespace App\Database\Factories;
-
-use App\Contracts\Factory;
-use App\Models\Acars;
-use DateTime;
-
-class AcarsFactory extends Factory
-{
-    /**
-     * The name of the factory's corresponding model.
-     *
-     * @var string
-     */
-    protected $model = Acars::class;
-
-    /**
-     * @return array <string, mixed>
-     */
-    public function definition(): array
-    {
-        return [
-            'id'          => null,
-            'pirep_id'    => null,
-            'log'         => $this->faker->text(100),
-            'lat'         => $this->faker->latitude,
-            'lon'         => $this->faker->longitude,
-            'distance'    => $this->faker->randomFloat(2, 0, 6000),
-            'heading'     => $this->faker->numberBetween(0, 359),
-            'altitude'    => $this->faker->numberBetween(20, 400),
-            'vs'          => $this->faker->numberBetween(-5000, 5000),
-            'gs'          => $this->faker->numberBetween(300, 500),
-            'transponder' => $this->faker->numberBetween(200, 9999),
-            'autopilot'   => $this->faker->text(10),
-            'fuel'        => $this->faker->randomFloat(2, 100, 1000),
-            'fuel_flow'   => $this->faker->randomFloat(2, 100, 1000),
-            'sim_time'    => $this->faker->dateTime('now', 'UTC')->format(DateTime::ATOM),
-        ];
-    }
-}
+$factory->define(App\Models\Acars::class, function (Faker $faker) {
+    return [
+        'id'          => null,
+        'pirep_id'    => null,
+        'log'         => $faker->text(100),
+        'lat'         => $faker->latitude,
+        'lon'         => $faker->longitude,
+        'distance'    => $faker->randomFloat(2, 0, 6000),
+        'heading'     => $faker->numberBetween(0, 359),
+        'altitude'    => $faker->numberBetween(20, 400),
+        'vs'          => $faker->numberBetween(-5000, 5000),
+        'gs'          => $faker->numberBetween(300, 500),
+        'transponder' => $faker->numberBetween(200, 9999),
+        'autopilot'   => $faker->text(10),
+        'fuel'        => $faker->randomFloat(2, 100, 1000),
+        'fuel_flow'   => $faker->randomFloat(2, 100, 1000),
+        'sim_time'    => $faker->dateTime('now', 'UTC'),
+    ];
+});

app/Database/factories/AircraftFactory.php

@@ -1,51 +1,33 @@
 <?php
 
-/** @noinspection PhpIllegalPsrClassPathInspection */
-
-namespace App\Database\Factories;
-
-use App\Contracts\Factory;
-use App\Models\Aircraft;
 use App\Models\Airport;
 use App\Models\Enums\AircraftState;
 use App\Models\Enums\AircraftStatus;
 use App\Models\Subfleet;
 use App\Support\ICAO;
-use DateTime;
+use Faker\Generator as Faker;
 
-class AircraftFactory extends Factory
-{
-    /**
-     * The name of the factory's corresponding model.
-     *
-     * @var string
-     */
-    protected $model = Aircraft::class;
-
-    /**
-     * Define the model's default state.
-     *
-     * @throws \Exception
-     *
-     * @return array<string, mixed>
-     */
-    public function definition(): array
-    {
-        return [
-            'id'           => null,
-            'subfleet_id'  => fn () => Subfleet::factory()->create()->id,
-            'airport_id'   => fn () => Airport::factory()->create()->id,
-            'iata'         => $this->faker->unique()->text(5),
-            'icao'         => $this->faker->unique()->text(5),
-            'name'         => $this->faker->text(50),
-            'registration' => $this->faker->unique()->text(10),
-            'hex_code'     => ICAO::createHexCode(),
-            'mtow'         => $this->faker->randomFloat(2, 0, 50000),
-            'zfw'          => $this->faker->randomFloat(2, 0, 50000),
-            'status'       => AircraftStatus::ACTIVE,
-            'state'        => AircraftState::PARKED,
-            'created_at'   => $this->faker->dateTimeBetween('-1 week')->format(DateTime::ATOM),
-            'updated_at'   => fn (array $pirep) => $pirep['created_at'],
-        ];
-    }
-}
+$factory->define(App\Models\Aircraft::class, function (Faker $faker) {
+    return [
+        'id'          => null,
+        'subfleet_id' => function () {
+            return factory(Subfleet::class)->create()->id;
+        },
+        'airport_id' => function () {
+            return factory(Airport::class)->create()->id;
+        },
+        'iata'         => $faker->unique()->text(5),
+        'icao'         => $faker->unique()->text(5),
+        'name'         => $faker->text(50),
+        'registration' => $faker->unique()->text(10),
+        'hex_code'     => ICAO::createHexCode(),
+        'mtow'         => $faker->randomFloat(2, 0, 50000),
+        'zfw'          => $faker->randomFloat(2, 0, 50000),
+        'status'       => AircraftStatus::ACTIVE,
+        'state'        => AircraftState::PARKED,
+        'created_at'   => $faker->dateTimeBetween('-1 week', 'now'),
+        'updated_at'   => function (array $pirep) {
+            return $pirep['created_at'];
+        },
+    ];
+});

app/Database/factories/AirlineFactory.php

@@ -1,41 +1,25 @@
 <?php
 
-/** @noinspection PhpIllegalPsrClassPathInspection */
-
-namespace App\Database\Factories;
-
-use App\Contracts\Factory;
-use App\Models\Airline;
+use Faker\Generator as Faker;
 use Hashids\Hashids;
 
-class AirlineFactory extends Factory
-{
-    /**
-     * The name of the factory's corresponding model.
-     *
-     * @var string
-     */
-    protected $model = Airline::class;
+/*
+ * Add any number of airports. Don't really care if they're real or not
+ */
+$factory->define(App\Models\Airline::class, function (Faker $faker) {
+    return [
+        'id'   => null,
+        'icao' => function (array $apt) {
+            $hashids = new Hashids(microtime(), 5);
+            $mt = str_replace('.', '', microtime(true));
 
-    /**
-     * Define the model's default state.
-     *
-     * @return array<string, mixed>
-     */
-    public function definition(): array
-    {
-        return [
-            'id'   => null,
-            'icao' => function (array $apt): string {
-                $hashids = new Hashids(microtime(), 5);
-                $mt = str_replace('.', '', microtime(true));
-
-                return $hashids->encode($mt);
-            },
-            'iata'    => fn (array $apt) => $apt['icao'],
-            'name'    => $this->faker->sentence(3),
-            'country' => $this->faker->country,
-            'active'  => 1,
-        ];
-    }
-}
+            return $hashids->encode($mt);
+        },
+        'iata' => function (array $apt) {
+            return $apt['icao'];
+        },
+        'name'    => $faker->sentence(3),
+        'country' => $faker->country,
+        'active'  => 1,
+    ];
+});

app/Database/factories/AirportFactory.php

@@ -1,31 +1,12 @@
 <?php
 
-/** @noinspection PhpIllegalPsrClassPathInspection */
+use Faker\Generator as Faker;
 
-namespace App\Database\Factories;
-
-use App\Contracts\Factory;
-use App\Models\Airport;
-
-class AirportFactory extends Factory
-{
-    /**
-     * The name of the factory's corresponding model.
-     *
-     * @var string
-     */
-    protected $model = Airport::class;
-
-    protected array $usedIcaos = [];
-
-    /**
-     * Generate a fake ICAO
-     *
-     * @throws \Exception
-     *
-     * @return string
-     */
-    protected function createFactoryICAO(): string
+/*
+ * Create an ICAO for use in the factory.
+ */
+if (!function_exists('createFactoryICAO')) {
+    function createFactoryICAO(): string
     {
         $characters = 'ABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789';
         $max = strlen($characters) - 1;
@@ -39,35 +20,37 @@ class AirportFactory extends Factory
 
         return $string;
     }
-
-    /**
-     * Define the model's default state.
-     *
-     * @return array<string, mixed>
-     */
-    public function definition(): array
-    {
-        return [
-            'id' => function () {
-                do {
-                    $airport = $this->createFactoryICAO();
-                } while (in_array($airport, $this->usedIcaos, true));
-
-                return $airport;
-            },
-            'icao'                 => fn (array $apt) => $apt['id'],
-            'iata'                 => fn (array $apt) => $apt['id'],
-            'name'                 => $this->faker->sentence(3),
-            'country'              => $this->faker->country,
-            'timezone'             => $this->faker->timezone,
-            'lat'                  => $this->faker->latitude,
-            'lon'                  => $this->faker->longitude,
-            'hub'                  => false,
-            'notes'                => null,
-            'ground_handling_cost' => $this->faker->randomFloat(2, 0, 500),
-            'fuel_100ll_cost'      => $this->faker->randomFloat(2, 1, 10),
-            'fuel_jeta_cost'       => $this->faker->randomFloat(2, 1, 10),
-            'fuel_mogas_cost'      => $this->faker->randomFloat(2, 1, 10),
-        ];
-    }
 }
+
+/*
+ * Add any number of airports. Don't really care if they're real or not
+ */
+$factory->define(App\Models\Airport::class, function (Faker $faker) {
+    $usedIcaos = [];
+
+    return [
+        'id' => function () use ($usedIcaos) {
+            do {
+                $airport = createFactoryICAO();
+            } while (in_array($airport, $usedIcaos, true));
+
+            return $airport;
+        },
+        'icao' => function (array $apt) {
+            return $apt['id'];
+        },
+        'iata' => function (array $apt) {
+            return $apt['id'];
+        },
+        'name'                 => $faker->sentence(3),
+        'country'              => $faker->country,
+        'timezone'             => $faker->timezone,
+        'lat'                  => $faker->latitude,
+        'lon'                  => $faker->longitude,
+        'hub'                  => false,
+        'ground_handling_cost' => $faker->randomFloat(2, 0, 500),
+        'fuel_100ll_cost'      => $faker->randomFloat(2, 1, 10),
+        'fuel_jeta_cost'       => $faker->randomFloat(2, 1, 10),
+        'fuel_mogas_cost'      => $faker->randomFloat(2, 1, 10),
+    ];
+});

app/Database/factories/AwardFactory.php

@@ -1,34 +0,0 @@
-<?php
-
-/** @noinspection PhpIllegalPsrClassPathInspection */
-
-namespace App\Database\Factories;
-
-use App\Contracts\Factory;
-use App\Models\Award;
-
-class AwardFactory extends Factory
-{
-    /**
-     * The name of the factory's corresponding model.
-     *
-     * @var string
-     */
-    protected $model = Award::class;
-
-    /**
-     * Define the model's default state.
-     *
-     * @return array<string, mixed>
-     */
-    public function definition(): array
-    {
-        return [
-            'id'               => null,
-            'name'             => $this->faker->name,
-            'description'      => $this->faker->text(10),
-            'ref_model'        => null,
-            'ref_model_params' => null,
-        ];
-    }
-}

app/Database/factories/AwardsFactory.php

@@ -0,0 +1,13 @@
+<?php
+
+use Faker\Generator as Faker;
+
+$factory->define(App\Models\Award::class, function (Faker $faker) {
+    return [
+        'id'               => null,
+        'name'             => $faker->name,
+        'description'      => $faker->text(10),
+        'ref_model'        => null,
+        'ref_model_params' => null,
+    ];
+});

app/Database/factories/ExpenseFactory.php

@@ -1,39 +1,18 @@
 <?php
 
-/** @noinspection PhpIllegalPsrClassPathInspection */
-
-namespace App\Database\Factories;
-
-use App\Contracts\Factory;
 use App\Models\Enums\ExpenseType;
-use App\Models\Expense;
+use Faker\Generator as Faker;
 
-class ExpenseFactory extends Factory
-{
-    /**
-     * The name of the factory's corresponding model.
-     *
-     * @var string
-     */
-    protected $model = Expense::class;
-
-    /**
-     * Define the model's default state.
-     *
-     * @return array<string, mixed>
-     */
-    public function definition(): array
-    {
-        return [
-            'id'           => null,
-            'airline_id'   => null,
-            'name'         => $this->faker->text(20),
-            'amount'       => $this->faker->randomFloat(2, 100, 1000),
-            'type'         => ExpenseType::FLIGHT,
-            'multiplier'   => false,
-            'ref_model'    => \App\Models\Expense::class,
-            'ref_model_id' => null,
-            'active'       => true,
-        ];
-    }
-}
+$factory->define(App\Models\Expense::class, function (Faker $faker) {
+    return [
+        'id'           => null,
+        'airline_id'   => null,
+        'name'         => $faker->text(20),
+        'amount'       => $faker->randomFloat(2, 100, 1000),
+        'type'         => ExpenseType::FLIGHT,
+        'multiplier'   => false,
+        'ref_model'    => \App\Models\Expense::class,
+        'ref_model_id' => null,
+        'active'       => true,
+    ];
+});

app/Database/factories/FareFactory.php

@@ -1,35 +1,16 @@
 <?php
 
-/** @noinspection PhpIllegalPsrClassPathInspection */
+use Faker\Generator as Faker;
 
-namespace App\Database\Factories;
-
-use App\Contracts\Factory;
-use App\Models\Fare;
-
-class FareFactory extends Factory
-{
-    /**
-     * The name of the factory's corresponding model.
-     *
-     * @var string
-     */
-    protected $model = Fare::class;
-
-    /**
-     * Define the model's default state.
-     *
-     * @return array<string, mixed>
-     */
-    public function definition(): array
-    {
-        return [
-            'id'       => null,
-            'code'     => $this->faker->unique()->text(50),
-            'name'     => $this->faker->text(50),
-            'price'    => $this->faker->randomFloat(2, 100, 1000),
-            'cost'     => fn (array $fare) => round($fare['price'] / 2),
-            'capacity' => $this->faker->randomFloat(0, 20, 500),
-        ];
-    }
-}
+$factory->define(App\Models\Fare::class, function (Faker $faker) {
+    return [
+        'id'    => null,
+        'code'  => $faker->unique()->text(50),
+        'name'  => $faker->text(50),
+        'price' => $faker->randomFloat(2, 100, 1000),
+        'cost'  => function (array $fare) {
+            return round($fare['price'] / 2);
+        },
+        'capacity' => $faker->randomFloat(0, 20, 500),
+    ];
+});

app/Database/factories/FlightFactory.php

@@ -1,58 +1,44 @@
 <?php
+/**
+ * Create flights
+ */
+use Faker\Generator as Faker;
 
-/** @noinspection PhpIllegalPsrClassPathInspection */
-
-namespace App\Database\Factories;
-
-use App\Contracts\Factory;
-use App\Models\Flight;
-use DateTime;
-
-class FlightFactory extends Factory
-{
-    /**
-     * The name of the factory's corresponding model.
-     *
-     * @var string
-     */
-    protected $model = Flight::class;
-
-    /**
-     * Define the model's default state.
-     *
-     * @return array<string, mixed>
-     */
-    public function definition(): array
-    {
-        return [
-            'id'            => $this->faker->unique()->numberBetween(10, 10000000),
-            'airline_id'    => fn () => \App\Models\Airline::factory()->create()->id,
-            'flight_number' => $this->faker->unique()->numberBetween(10, 1000000),
-            'route_code'    => $this->faker->randomElement(['', $this->faker->text(5)]),
-            'route_leg'     => $this->faker->randomElement(
-                ['', $this->faker->numberBetween(0, 1000)]
-            ),
-            'dpt_airport_id'       => static fn () => \App\Models\Airport::factory()->create()->id,
-            'arr_airport_id'       => static fn () => \App\Models\Airport::factory()->create()->id,
-            'alt_airport_id'       => static fn () => \App\Models\Airport::factory()->create()->id,
-            'distance'             => $this->faker->numberBetween(1, 1000),
-            'route'                => null,
-            'level'                => 0,
-            'dpt_time'             => $this->faker->time(),
-            'arr_time'             => $this->faker->time(),
-            'flight_time'          => $this->faker->numberBetween(60, 360),
-            'load_factor'          => $this->faker->randomElement([15, 20, 50, 90, 100]),
-            'load_factor_variance' => $this->faker->randomElement([15, 20, 50, 90, 100]),
-            'has_bid'              => false,
-            'active'               => true,
-            'visible'              => true,
-            'days'                 => 0,
-            'start_date'           => null,
-            'end_date'             => null,
-            'created_at'           => $this->faker->dateTimeBetween('-1 week')->format(
-                DateTime::ATOM
-            ),
-            'updated_at' => static fn (array $flight) => $flight['created_at'],
-        ];
-    }
-}
+$factory->define(App\Models\Flight::class, function (Faker $faker) {
+    return [
+        'id'         => $faker->unique()->numberBetween(10, 10000000),
+        'airline_id' => function () {
+            return factory(\App\Models\Airline::class)->create()->id;
+        },
+        'flight_number'  => $faker->unique()->numberBetween(10, 1000000),
+        'route_code'     => $faker->randomElement(['', $faker->text(5)]),
+        'route_leg'      => $faker->randomElement(['', $faker->numberBetween(0, 1000)]),
+        'dpt_airport_id' => function () {
+            return factory(\App\Models\Airport::class)->create()->id;
+        },
+        'arr_airport_id' => function () {
+            return factory(\App\Models\Airport::class)->create()->id;
+        },
+        'alt_airport_id' => function () {
+            return factory(\App\Models\Airport::class)->create()->id;
+        },
+        'distance'             => $faker->numberBetween(1, 1000),
+        'route'                => null,
+        'level'                => 0,
+        'dpt_time'             => $faker->time(),
+        'arr_time'             => $faker->time(),
+        'flight_time'          => $faker->numberBetween(60, 360),
+        'load_factor'          => $faker->randomElement([15, 20, 50, 90, 100]),
+        'load_factor_variance' => $faker->randomElement([15, 20, 50, 90, 100]),
+        'has_bid'              => false,
+        'active'               => true,
+        'visible'              => true,
+        'days'                 => 0,
+        'start_date'           => null,
+        'end_date'             => null,
+        'created_at'           => $faker->dateTimeBetween('-1 week', 'now'),
+        'updated_at'           => static function (array $flight) {
+            return $flight['created_at'];
+        },
+    ];
+});

app/Database/factories/JournalFactory.php

@@ -1,30 +1,9 @@
 <?php
 
-/** @noinspection PhpIllegalPsrClassPathInspection */
+use Faker\Generator as Faker;
 
-namespace App\Database\Factories;
-
-use App\Contracts\Factory;
-use App\Models\Journal;
-
-class JournalFactory extends Factory
-{
-    /**
-     * The name of the factory's corresponding model.
-     *
-     * @var string
-     */
-    protected $model = Journal::class;
-
-    /**
-     * Define the model's default state.
-     *
-     * @return array<string, mixed>
-     */
-    public function definition(): array
-    {
-        return [
-            'currency' => 'USD',
-        ];
-    }
-}
+$factory->define(App\Models\Journal::class, function (Faker $faker) {
+    return [
+        'currency' => 'USD',
+    ];
+});

app/Database/factories/JournalTransactionsFactory.php

@@ -1,39 +1,20 @@
 <?php
 
-/** @noinspection PhpIllegalPsrClassPathInspection */
-
-namespace App\Database\Factories;
-
-use App\Contracts\Factory;
 use App\Models\Journal;
-use App\Models\JournalTransaction;
 use Carbon\Carbon;
+use Faker\Generator as Faker;
 use Ramsey\Uuid\Uuid;
 
-class JournalTransactionsFactory extends Factory
-{
-    /**
-     * The name of the factory's corresponding model.
-     *
-     * @var string
-     */
-    protected $model = JournalTransaction::class;
-
-    /**
-     * Define the model's default state.
-     *
-     * @return array<string, mixed>
-     */
-    public function definition(): array
-    {
-        return [
-            'transaction_group' => Uuid::uuid4()->toString(),
-            'journal_id'        => fn () => Journal::factory()->create()->id,
-            'credit'            => $this->faker->numberBetween(100, 10000),
-            'debit'             => $this->faker->numberBetween(100, 10000),
-            'currency'          => 'USD',
-            'memo'              => $this->faker->sentence(6),
-            'post_date'         => Carbon::now('UTC')->toDateTimeString(),
-        ];
-    }
-}
+$factory->define(App\Models\JournalTransaction::class, function (Faker $faker) {
+    return [
+        'transaction_group' => Uuid::uuid4()->toString(),
+        'journal_id'        => function () {
+            return factory(Journal::class)->create()->id;
+        },
+        'credit'    => $faker->numberBetween(100, 10000),
+        'debit'     => $faker->numberBetween(100, 10000),
+        'currency'  => 'USD',
+        'memo'      => $faker->sentence(6),
+        'post_date' => Carbon::now('UTC'),
+    ];
+});

app/Database/factories/NavdataFactory.php

@@ -1,36 +1,15 @@
 <?php
 
-/** @noinspection PhpIllegalPsrClassPathInspection */
-
-namespace App\Database\Factories;
-
-use App\Contracts\Factory;
 use App\Models\Enums\NavaidType;
-use App\Models\Navdata;
+use Faker\Generator as Faker;
 
-class NavdataFactory extends Factory
-{
-    /**
-     * The name of the factory's corresponding model.
-     *
-     * @var string
-     */
-    protected $model = Navdata::class;
-
-    /**
-     * Define the model's default state.
-     *
-     * @return array<string, mixed>
-     */
-    public function definition(): array
-    {
-        return [
-            'id'   => str_replace(' ', '', str_replace('.', '', $this->faker->unique()->text(5))),
-            'name' => str_replace('.', '', $this->faker->unique()->word),
-            'type' => $this->faker->randomElement([NavaidType::VOR, NavaidType::NDB]),
-            'lat'  => $this->faker->latitude,
-            'lon'  => $this->faker->longitude,
-            'freq' => $this->faker->randomFloat(2, 100, 1000),
-        ];
-    }
-}
+$factory->define(App\Models\Navdata::class, function (Faker $faker) {
+    return [
+        'id'   => str_replace(' ', '', str_replace('.', '', $faker->unique()->text(5))),
+        'name' => str_replace('.', '', $faker->unique()->word),
+        'type' => $faker->randomElement([NavaidType::VOR, NavaidType::NDB]),
+        'lat'  => $faker->latitude,
+        'lon'  => $faker->longitude,
+        'freq' => $faker->randomFloat(2, 100, 1000),
+    ];
+});

app/Database/factories/NewsFactory.php

@@ -1,33 +1,14 @@
 <?php
 
-/** @noinspection PhpIllegalPsrClassPathInspection */
+use Faker\Generator as Faker;
 
-namespace App\Database\Factories;
-
-use App\Contracts\Factory;
-use App\Models\News;
-
-class NewsFactory extends Factory
-{
-    /**
-     * The name of the factory's corresponding model.
-     *
-     * @var string
-     */
-    protected $model = News::class;
-
-    /**
-     * Define the model's default state.
-     *
-     * @return array<string, mixed>
-     */
-    public function definition(): array
-    {
-        return [
-            'id'      => null,
-            'user_id' => fn () => \App\Models\User::factory()->create()->id,
-            'subject' => $this->faker->text(),
-            'body'    => $this->faker->sentence,
-        ];
-    }
-}
+$factory->define(App\Models\News::class, function (Faker $faker) {
+    return [
+        'id'      => null,
+        'user_id' => function () {
+            return factory(\App\Models\User::class)->create()->id;
+        },
+        'subject' => $faker->text(),
+        'body'    => $faker->sentence,
+    ];
+});

app/Database/factories/PirepFactory.php

@@ -1,74 +1,69 @@
 <?php
 
-/** @noinspection PhpIllegalPsrClassPathInspection */
-
-namespace App\Database\Factories;
-
-use App\Contracts\Factory;
-use App\Models\Airline;
 use App\Models\Enums\PirepSource;
 use App\Models\Enums\PirepState;
 use App\Models\Enums\PirepStatus;
-use App\Models\Flight;
-use App\Models\Pirep;
 use Carbon\Carbon;
+use Faker\Generator as Faker;
 
-class PirepFactory extends Factory
-{
-    /**
-     * The name of the factory's corresponding model.
-     *
-     * @var string
-     */
-    protected $model = Pirep::class;
+/*
+ * Create a new PIREP
+ */
+$factory->define(App\Models\Pirep::class, function (Faker $faker) {
+    $airline = factory(\App\Models\Airline::class)->create();
+    $flight = factory(\App\Models\Flight::class)->create([
+        'airline_id' => $airline->id,
+    ]);
 
-    /**
-     * Define the model's default state.
-     *
-     * @return array<string, mixed>
-     */
-    public function definition(): array
-    {
-        /** @var Airline $airline */
-        $airline = Airline::factory()->create();
-
-        /** @var Flight $flight */
-        $flight = Flight::factory()->create(['airline_id' => $airline->id]);
-
-        return [
-            'id'                  => $this->faker->unique()->numberBetween(10, 10000000),
-            'airline_id'          => fn () => $airline->id,
-            'user_id'             => fn () => \App\Models\User::factory()->create()->id,
-            'aircraft_id'         => fn () => \App\Models\Aircraft::factory()->create()->id,
-            'flight_id'           => fn () => $flight->id,
-            'flight_number'       => fn () => $flight->flight_number,
-            'route_code'          => null,
-            'route_leg'           => null,
-            'dpt_airport_id'      => fn () => $flight->dpt_airport_id,
-            'arr_airport_id'      => fn () => $flight->arr_airport_id,
-            'level'               => $this->faker->numberBetween(20, 400),
-            'distance'            => $this->faker->randomFloat(2, 0, 6000),
-            'planned_distance'    => $this->faker->randomFloat(2, 0, 6000),
-            'flight_time'         => $this->faker->numberBetween(60, 360),
-            'planned_flight_time' => $this->faker->numberBetween(60, 360),
-            'zfw'                 => $this->faker->randomFloat(2),
-            'block_fuel'          => $this->faker->randomFloat(2, 0, 1000),
-            'fuel_used'           => fn (array $pirep) => round($pirep['block_fuel'] * .9, 2),
-            'block_on_time'       => Carbon::now('UTC'),
-            'block_off_time'      => fn (array $pirep) => $pirep['block_on_time']->subMinutes(
-                $pirep['flight_time']
-            ),
-            'route'  => $this->faker->text(200),
-            'notes'  => $this->faker->text(200),
-            'source' => $this->faker->randomElement(
-                [PirepSource::MANUAL, PirepSource::ACARS]
-            ),
-            'source_name'  => 'TestFactory',
-            'state'        => PirepState::PENDING,
-            'status'       => PirepStatus::SCHEDULED,
-            'submitted_at' => Carbon::now('UTC')->toDateTimeString(),
-            'created_at'   => Carbon::now('UTC')->toDateTimeString(),
-            'updated_at'   => fn (array $pirep) => $pirep['created_at'],
-        ];
-    }
-}
+    return [
+        'id'         => $faker->unique()->numberBetween(10, 10000000),
+        'airline_id' => function () use ($airline) {
+            return $airline->id;
+        },
+        'user_id' => function () {
+            return factory(\App\Models\User::class)->create()->id;
+        },
+        'aircraft_id' => function () {
+            return factory(\App\Models\Aircraft::class)->create()->id;
+        },
+        'flight_id' => function () use ($flight) {
+            return $flight->id;
+        },
+        'flight_number' => function () use ($flight) {
+            return $flight->flight_number;
+        },
+        'route_code'     => null,
+        'route_leg'      => null,
+        'dpt_airport_id' => function () use ($flight) {
+            return $flight->dpt_airport_id;
+        },
+        'arr_airport_id' => function () use ($flight) {
+            return $flight->arr_airport_id;
+        },
+        'level'               => $faker->numberBetween(20, 400),
+        'distance'            => $faker->randomFloat(2, 0, 6000),
+        'planned_distance'    => $faker->randomFloat(2, 0, 6000),
+        'flight_time'         => $faker->numberBetween(60, 360),
+        'planned_flight_time' => $faker->numberBetween(60, 360),
+        'zfw'                 => $faker->randomFloat(2),
+        'block_fuel'          => $faker->randomFloat(2, 0, 1000),
+        'fuel_used'           => function (array $pirep) {
+            return round($pirep['block_fuel'] * .9, 2); // 90% of the fuel loaded was used
+        },
+        'block_on_time'  => Carbon::now('UTC'),
+        'block_off_time' => function (array $pirep) {
+            return $pirep['block_on_time']->subMinutes($pirep['flight_time']);
+        },
+        'route'        => $faker->text(200),
+        'notes'        => $faker->text(200),
+        'source'       => $faker->randomElement([PirepSource::MANUAL, PirepSource::ACARS]),
+        'source_name'  => 'TestFactory',
+        'state'        => PirepState::PENDING,
+        'status'       => PirepStatus::SCHEDULED,
+        'submitted_at' => Carbon::now('UTC')->toDateTimeString(),
+        'created_at'   => Carbon::now('UTC')->toDateTimeString(),
+        'updated_at'   => function (array $pirep) {
+            return $pirep['created_at'];
+        },
+    ];
+});

app/Database/factories/RankFactory.php

@@ -1,37 +1,23 @@
 <?php
 
-/** @noinspection PhpIllegalPsrClassPathInspection */
+use Faker\Generator as Faker;
 
-namespace App\Database\Factories;
-
-use App\Contracts\Factory;
-use App\Models\Rank;
-
-class RankFactory extends Factory
-{
-    /**
-     * The name of the factory's corresponding model.
-     *
-     * @var string
-     */
-    protected $model = Rank::class;
-
-    /**
-     * Define the model's default state.
-     *
-     * @return array<string, mixed>
-     */
-    public function definition(): array
-    {
-        return [
-            'id'                   => null,
-            'name'                 => $this->faker->unique()->text(50),
-            'hours'                => $this->faker->numberBetween(10, 50),
-            'acars_base_pay_rate'  => $this->faker->numberBetween(10, 100),
-            'manual_base_pay_rate' => $this->faker->numberBetween(10, 100),
-            'auto_approve_acars'   => 0,
-            'auto_approve_manual'  => 0,
-            'auto_promote'         => 0,
-        ];
-    }
-}
+/*
+ * id: 2
+    name: Junior First Officer
+    hours: 10
+    auto_approve_acars: 1
+    auto_approve_manual: 1
+ */
+$factory->define(App\Models\Rank::class, function (Faker $faker) {
+    return [
+        'id'                   => null,
+        'name'                 => $faker->unique()->text(50),
+        'hours'                => $faker->numberBetween(10, 50),
+        'acars_base_pay_rate'  => $faker->numberBetween(10, 100),
+        'manual_base_pay_rate' => $faker->numberBetween(10, 100),
+        'auto_approve_acars'   => 0,
+        'auto_approve_manual'  => 0,
+        'auto_promote'         => 0,
+    ];
+});

app/Database/factories/RoleFactory.php

@@ -1,34 +1,13 @@
 <?php
 
-/** @noinspection PhpIllegalPsrClassPathInspection */
+use Faker\Generator as Faker;
 
-namespace App\Database\Factories;
-
-use App\Contracts\Factory;
-use App\Models\Role;
-
-class RoleFactory extends Factory
-{
-    /**
-     * The name of the factory's corresponding model.
-     *
-     * @var string
-     */
-    protected $model = Role::class;
-
-    /**
-     * Define the model's default state.
-     *
-     * @return array<string, mixed>
-     */
-    public function definition(): array
-    {
-        return [
-            'id'                      => null,
-            'name'                    => $this->faker->name,
-            'display_name'            => $this->faker->name,
-            'read_only'               => false,
-            'disable_activity_checks' => $this->faker->boolean(),
-        ];
-    }
-}
+$factory->define(App\Models\Role::class, function (Faker $faker) {
+    return [
+        'id'                      => null,
+        'name'                    => $faker->name,
+        'display_name'            => $faker->name,
+        'read_only'               => false,
+        'disable_activity_checks' => $faker->boolean(),
+    ];
+});

app/Database/factories/SimBriefFactory.php

@@ -1,38 +0,0 @@
-<?php
-
-/** @noinspection PhpIllegalPsrClassPathInspection */
-
-namespace App\Database\Factories;
-
-use App\Contracts\Factory;
-use App\Models\SimBrief;
-use Carbon\Carbon;
-
-class SimBriefFactory extends Factory
-{
-    /**
-     * The name of the factory's corresponding model.
-     *
-     * @var string
-     */
-    protected $model = SimBrief::class;
-
-    /**
-     * Define the model's default state.
-     *
-     * @return array<string, mixed>
-     */
-    public function definition()
-    {
-        return [
-            'id'         => $this->faker->unique()->numberBetween(10, 10000000),
-            'user_id'    => null,
-            'flight_id'  => null,
-            'pirep_id'   => null,
-            'acars_xml'  => '',
-            'ofp_xml'    => '',
-            'created_at' => Carbon::now('UTC')->toDateTimeString(),
-            'updated_at' => fn (array $sb) => $sb['created_at'],
-        ];
-    }
-}

app/Database/factories/SimbriefFactory.php

@@ -0,0 +1,21 @@
+<?php
+
+/** @var \Illuminate\Database\Eloquent\Factory $factory */
+
+use Carbon\Carbon;
+use Faker\Generator as Faker;
+
+$factory->define(App\Models\SimBrief::class, function (Faker $faker) {
+    return [
+        'id'         => $faker->unique()->numberBetween(10, 10000000),
+        'user_id'    => null,
+        'flight_id'  => null,
+        'pirep_id'   => null,
+        'acars_xml'  => '',
+        'ofp_xml'    => '',
+        'created_at' => Carbon::now('UTC')->toDateTimeString(),
+        'updated_at' => function (array $sb) {
+            return $sb['created_at'];
+        },
+    ];
+});

app/Database/factories/SubfleetFactory.php

@@ -1,34 +1,15 @@
 <?php
 
-/** @noinspection PhpIllegalPsrClassPathInspection */
+use Faker\Generator as Faker;
 
-namespace App\Database\Factories;
-
-use App\Contracts\Factory;
-use App\Models\Subfleet;
-
-class SubfleetFactory extends Factory
-{
-    /**
-     * The name of the factory's corresponding model.
-     *
-     * @var string
-     */
-    protected $model = Subfleet::class;
-
-    /**
-     * Define the model's default state.
-     *
-     * @return array<string, mixed>
-     */
-    public function definition(): array
-    {
-        return [
-            'id'                         => null,
-            'airline_id'                 => fn () => \App\Models\Airline::factory()->create()->id,
-            'name'                       => $this->faker->unique()->text(50),
-            'type'                       => $this->faker->unique()->text(7),
-            'ground_handling_multiplier' => $this->faker->numberBetween(50, 200),
-        ];
-    }
-}
+$factory->define(App\Models\Subfleet::class, function (Faker $faker) {
+    return [
+        'id'         => null,
+        'airline_id' => function () {
+            return factory(\App\Models\Airline::class)->create()->id;
+        },
+        'name'                       => $faker->unique()->text(50),
+        'type'                       => $faker->unique()->text(7),
+        'ground_handling_multiplier' => $faker->numberBetween(50, 200),
+    ];
+});

app/Database/factories/UserFactory.php

@@ -1,54 +1,28 @@
 <?php
 
-/** @noinspection PhpIllegalPsrClassPathInspection */
-
-namespace App\Database\Factories;
-
-use App\Contracts\Factory;
 use App\Models\Airline;
 use App\Models\Enums\UserState;
-use App\Models\User;
+use Faker\Generator as Faker;
 use Illuminate\Support\Facades\Hash;
 
-class UserFactory extends Factory
-{
-    /**
-     * The name of the factory's corresponding model.
-     *
-     * @var string
-     */
-    protected $model = User::class;
+$factory->define(App\Models\User::class, function (Faker $faker) {
+    static $password;
 
-    /**
-     * @var string
-     */
-    private static string $password;
-
-    /**
-     * Define the model's default state.
-     *
-     * @return array<string, mixed>
-     */
-    public function definition(): array
-    {
-        if (empty(self::$password)) {
-            self::$password = Hash::make('secret');
-        }
-
-        return [
-            'id'             => null,
-            'pilot_id'       => null,
-            'name'           => $this->faker->name,
-            'email'          => $this->faker->safeEmail,
-            'password'       => self::$password,
-            'api_key'        => $this->faker->sha1,
-            'airline_id'     => fn () => Airline::factory()->create()->id,
-            'rank_id'        => 1,
-            'flights'        => $this->faker->numberBetween(0, 1000),
-            'flight_time'    => $this->faker->numberBetween(0, 10000),
-            'transfer_time'  => $this->faker->numberBetween(0, 10000),
-            'state'          => UserState::ACTIVE,
-            'remember_token' => $this->faker->unique()->text(5),
-        ];
-    }
-}
+    return [
+        'id'         => null,
+        'pilot_id'   => null,
+        'name'       => $faker->name,
+        'email'      => $faker->safeEmail,
+        'password'   => $password ?: $password = Hash::make('secret'),
+        'api_key'    => $faker->sha1,
+        'airline_id' => function () {
+            return factory(Airline::class)->create()->id;
+        },
+        'rank_id'        => 1,
+        'flights'        => $faker->numberBetween(0, 1000),
+        'flight_time'    => $faker->numberBetween(0, 10000),
+        'transfer_time'  => $faker->numberBetween(0, 10000),
+        'state'          => UserState::ACTIVE,
+        'remember_token' => $faker->unique()->text(5),
+    ];
+});

app/Database/migrations/2017_06_07_014930_create_settings_table.php

@@ -4,7 +4,8 @@ use App\Contracts\Migration;
 use App\Services\Installer\SeederService;
 use Illuminate\Database\Schema\Blueprint;
 
-return new class() extends Migration {
+class CreateSettingsTable extends Migration
+{
     private $seederSvc;
 
     public function __construct()
@@ -49,4 +50,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('settings');
     }
-};
+}

app/Database/migrations/2017_06_08_0000_create_users_table.php

@@ -5,7 +5,8 @@ use App\Models\Pirep;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
-return new class() extends Migration {
+class CreateUsersTable extends Migration
+{
     /**
      * Run the migrations.
      *
@@ -54,4 +55,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('users');
     }
-};
+}

app/Database/migrations/2017_06_08_0001_roles_permissions_tables.php

@@ -2,9 +2,9 @@
 
 use App\Contracts\Migration;
 use Illuminate\Database\Schema\Blueprint;
-use Illuminate\Support\Facades\Schema;
 
-return new class() extends Migration {
+class RolesPermissionsTables extends Migration
+{
     public function up()
     {
         // Create table for storing roles
@@ -82,4 +82,4 @@ return new class() extends Migration {
         Schema::dropIfExists('role_user');
         Schema::dropIfExists('roles');
     }
-};
+}

app/Database/migrations/2017_06_08_0005_create_password_resets_table.php

@@ -4,7 +4,8 @@ use App\Contracts\Migration;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
-return new class() extends Migration {
+class CreatePasswordResetsTable extends Migration
+{
     public function up()
     {
         Schema::create('password_resets', function (Blueprint $table) {
@@ -18,4 +19,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('password_resets');
     }
-};
+}

app/Database/migrations/2017_06_08_0006_create_sessions_table.php

@@ -4,7 +4,8 @@ use App\Contracts\Migration;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
-return new class() extends Migration {
+class CreateSessionsTable extends Migration
+{
     public function up()
     {
         Schema::create('sessions', function (Blueprint $table) {
@@ -21,4 +22,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('sessions');
     }
-};
+}

app/Database/migrations/2017_06_08_191703_create_airlines_table.php

@@ -3,7 +3,8 @@
 use App\Contracts\Migration;
 use Illuminate\Database\Schema\Blueprint;
 
-return new class() extends Migration {
+class CreateAirlinesTable extends Migration
+{
     public function up()
     {
         Schema::create('airlines', function (Blueprint $table) {
@@ -30,4 +31,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('airlines');
     }
-};
+}

app/Database/migrations/2017_06_09_010621_create_aircrafts_table.php

@@ -5,7 +5,8 @@ use App\Models\Enums\AircraftState;
 use App\Models\Enums\AircraftStatus;
 use Illuminate\Database\Schema\Blueprint;
 
-return new class() extends Migration {
+class CreateAircraftsTable extends Migration
+{
     public function up()
     {
         Schema::create('aircraft', function (Blueprint $table) {
@@ -33,4 +34,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('aircraft');
     }
-};
+}

app/Database/migrations/2017_06_10_040335_create_fares_table.php

@@ -3,7 +3,8 @@
 use App\Contracts\Migration;
 use Illuminate\Database\Schema\Blueprint;
 
-return new class() extends Migration {
+class CreateFaresTable extends Migration
+{
     public function up()
     {
         Schema::create('fares', function (Blueprint $table) {
@@ -23,4 +24,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('fares');
     }
-};
+}

app/Database/migrations/2017_06_11_135707_create_airports_table.php

@@ -3,7 +3,8 @@
 use App\Contracts\Migration;
 use Illuminate\Database\Schema\Blueprint;
 
-return new class() extends Migration {
+class CreateAirportsTable extends Migration
+{
     public function up()
     {
         Schema::create('airports', function (Blueprint $table) {
@@ -32,4 +33,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('airports');
     }
-};
+}

app/Database/migrations/2017_06_17_214650_create_flight_tables.php

@@ -5,7 +5,8 @@ use App\Contracts\Model;
 use App\Models\Enums\FlightType;
 use Illuminate\Database\Schema\Blueprint;
 
-return new class() extends Migration {
+class CreateFlightTables extends Migration
+{
     public function up()
     {
         Schema::create('flights', function (Blueprint $table) {
@@ -97,4 +98,4 @@ return new class() extends Migration {
         Schema::drop('flight_subfleet');
         Schema::drop('flights');
     }
-};
+}

app/Database/migrations/2017_06_21_165410_create_ranks_table.php

@@ -3,7 +3,8 @@
 use App\Contracts\Migration;
 use Illuminate\Database\Schema\Blueprint;
 
-return new class() extends Migration {
+class CreateRanksTable extends Migration
+{
     public function up()
     {
         Schema::create('ranks', function (Blueprint $table) {
@@ -28,4 +29,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('ranks');
     }
-};
+}

app/Database/migrations/2017_06_23_011011_create_subfleet_tables.php

@@ -3,7 +3,8 @@
 use App\Contracts\Migration;
 use Illuminate\Database\Schema\Blueprint;
 
-return new class() extends Migration {
+class CreateSubfleetTables extends Migration
+{
     public function up()
     {
         Schema::create('subfleets', function (Blueprint $table) {
@@ -50,4 +51,4 @@ return new class() extends Migration {
         Schema::dropIfExists('subfleet_fare');
         Schema::dropIfExists('subfleet_rank');
     }
-};
+}

app/Database/migrations/2017_06_28_195426_create_pirep_tables.php

@@ -7,7 +7,8 @@ use App\Models\Enums\PirepState;
 use App\Models\Enums\PirepStatus;
 use Illuminate\Database\Schema\Blueprint;
 
-return new class() extends Migration {
+class CreatePirepTables extends Migration
+{
     /**
      * Run the migrations.
      *
@@ -105,4 +106,4 @@ return new class() extends Migration {
         Schema::dropIfExists('pirep_fields');
         Schema::dropIfExists('pirep_field_values');
     }
-};
+}

app/Database/migrations/2017_12_12_174519_create_bids_table.php

@@ -5,7 +5,8 @@ use App\Contracts\Model;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
-return new class() extends Migration {
+class CreateBidsTable extends Migration
+{
     public function up()
     {
         Schema::create('bids', function (Blueprint $table) {
@@ -23,4 +24,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('bids');
     }
-};
+}

app/Database/migrations/2017_12_14_225241_create_jobs_table.php

@@ -4,7 +4,8 @@ use App\Contracts\Migration;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
-return new class() extends Migration {
+class CreateJobsTable extends Migration
+{
     public function up()
     {
         Schema::create('jobs', function (Blueprint $table) {
@@ -22,4 +23,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('jobs');
     }
-};
+}

app/Database/migrations/2017_12_14_225337_create_failed_jobs_table.php

@@ -4,7 +4,8 @@ use App\Contracts\Migration;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
-return new class() extends Migration {
+class CreateFailedJobsTable extends Migration
+{
     public function up()
     {
         Schema::create('failed_jobs', function (Blueprint $table) {
@@ -21,4 +22,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('failed_jobs');
     }
-};
+}

app/Database/migrations/2017_12_20_004147_create_navdata_tables.php

@@ -4,7 +4,8 @@ use App\Contracts\Migration;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
-return new class() extends Migration {
+class CreateNavdataTables extends Migration
+{
     public function up()
     {
         /*
@@ -29,4 +30,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('navdata');
     }
-};
+}

app/Database/migrations/2017_12_20_005147_create_acars_tables.php

@@ -5,7 +5,8 @@ use App\Contracts\Model;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
-return new class() extends Migration {
+class CreateAcarsTables extends Migration
+{
     public function up()
     {
         Schema::create('acars', function (Blueprint $table) {
@@ -42,4 +43,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('acars');
     }
-};
+}

app/Database/migrations/2018_01_03_014930_create_stats_table.php

@@ -4,7 +4,8 @@ use App\Contracts\Migration;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
-return new class() extends Migration {
+class CreateStatsTable extends Migration
+{
     public function up()
     {
         Schema::create('stats', function (Blueprint $table) {
@@ -23,4 +24,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('stats');
     }
-};
+}

app/Database/migrations/2018_01_08_142204_create_news_table.php

@@ -4,7 +4,8 @@ use App\Contracts\Migration;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
-return new class() extends Migration {
+class CreateNewsTable extends Migration
+{
     public function up()
     {
         Schema::create('news', function (Blueprint $table) {
@@ -20,4 +21,4 @@ return new class() extends Migration {
     {
         Schema::drop('news');
     }
-};
+}

app/Database/migrations/2018_01_28_180522_create_awards_table.php

@@ -5,7 +5,8 @@ use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 use Modules\Awards\Awards\PilotFlightAwards;
 
-return new class() extends Migration {
+class CreateAwardsTable extends Migration
+{
     public function up()
     {
         Schema::create('awards', function (Blueprint $table) {
@@ -51,4 +52,4 @@ return new class() extends Migration {
         Schema::dropIfExists('awards');
         Schema::dropIfExists('user_awards');
     }
-};
+}

app/Database/migrations/2018_02_26_185121_create_expenses_table.php

@@ -4,7 +4,8 @@ use App\Contracts\Migration;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
-return new class() extends Migration {
+class CreateExpensesTable extends Migration
+{
     public function up()
     {
         Schema::create('expenses', function (Blueprint $table) {
@@ -33,4 +34,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('expenses');
     }
-};
+}

app/Database/migrations/2018_02_28_231807_create_journal_transactions_table.php

@@ -4,7 +4,8 @@ use App\Contracts\Migration;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
-return new class() extends Migration {
+class CreateJournalTransactionsTable extends Migration
+{
     public function up()
     {
         Schema::create('journal_transactions', function (Blueprint $table) {
@@ -32,4 +33,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('journal_transactions');
     }
-};
+}

app/Database/migrations/2018_02_28_231813_create_journals_table.php

@@ -4,7 +4,8 @@ use App\Contracts\Migration;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
-return new class() extends Migration {
+class CreateJournalsTable extends Migration
+{
     public function up()
     {
         Schema::create('journals', function (Blueprint $table) {
@@ -22,4 +23,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('journals');
     }
-};
+}

app/Database/migrations/2018_02_28_232438_create_ledgers_table.php

@@ -4,7 +4,8 @@ use App\Contracts\Migration;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
-return new class() extends Migration {
+class CreateLedgersTable extends Migration
+{
     /**
      * Run the migrations.
      *
@@ -29,4 +30,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('ledgers');
     }
-};
+}

app/Database/migrations/2018_04_01_193443_create_files_table.php

@@ -5,7 +5,8 @@ use App\Contracts\Model;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
-return new class() extends Migration {
+class CreateFilesTable extends Migration
+{
     /**
      * Create the files table. Acts as a morphable
      *
@@ -39,4 +40,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('files');
     }
-};
+}

app/Database/migrations/2019_06_19_220910_add_readonly_to_roles.php

@@ -5,7 +5,8 @@ use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Facades\Schema;
 
-return new class() extends Migration {
+class AddReadonlyToRoles extends Migration
+{
     public function up(): void
     {
         Schema::table('roles', static function (Blueprint $table) {
@@ -24,4 +25,4 @@ return new class() extends Migration {
             $table->dropColumn('read_only');
         });
     }
-};
+}

app/Database/migrations/2019_07_16_141152_users_add_pilot_id.php

@@ -12,7 +12,8 @@ use Illuminate\Support\Facades\Schema;
  * 3. Iterate through all of the users and set their `id` to the `pilot_id`
  * 4. Change the other tables column types that reference `user_id`
  */
-return new class() extends Migration {
+class UsersAddPilotId extends Migration
+{
     public function up()
     {
         Schema::table('users', static function (Blueprint $table) {
@@ -32,4 +33,4 @@ return new class() extends Migration {
             $table->dropColumn('pilot_id');
         });
     }
-};
+}

app/Database/migrations/2019_08_30_132224_create_notifications_table.php

@@ -4,7 +4,8 @@ use App\Contracts\Migration;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
-return new class() extends Migration {
+class CreateNotificationsTable extends Migration
+{
     public function up()
     {
         Schema::create('notifications', function (Blueprint $table) {
@@ -21,4 +22,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('notifications');
     }
-};
+}

app/Database/migrations/2019_09_16_141152_pireps_change_state_type.php

@@ -9,7 +9,8 @@ use Illuminate\Support\Facades\Schema;
 /**
  * Change the PIREP state column to be a TINYINT
  */
-return new class() extends Migration {
+class PirepsChangeStateType extends Migration
+{
     public function up()
     {
         // Migrate the old rejected state
@@ -22,4 +23,4 @@ return new class() extends Migration {
             $table->unsignedSmallInteger('state')->change();
         });
     }
-};
+}

app/Database/migrations/2019_10_30_141152_pireps_add_flight_id.php

@@ -8,11 +8,12 @@ use Illuminate\Support\Facades\Schema;
 /**
  * Add a `flight_id` column to the PIREPs table
  */
-return new class() extends Migration {
+class PirepsAddFlightId extends Migration
+{
     public function up()
     {
         Schema::table('pireps', function (Blueprint $table) {
             $table->string('flight_id', Model::ID_MAX_LENGTH)->nullable()->after('aircraft_id');
         });
     }
-};
+}

app/Database/migrations/2020_02_12_141152_expenses_add_flight_type.php

@@ -7,7 +7,8 @@ use Illuminate\Support\Facades\Schema;
 /**
  * Add a `flight_type` column to the expenses table
  */
-return new class() extends Migration {
+class ExpensesAddFlightType extends Migration
+{
     public function up()
     {
         Schema::table('expenses', function (Blueprint $table) {
@@ -23,4 +24,4 @@ return new class() extends Migration {
             $table->dropColumn('flight_type');
         });
     }
-};
+}

app/Database/migrations/2020_02_26_044305_modify_airports_coordinates.php

@@ -6,7 +6,8 @@ use Illuminate\Support\Facades\Schema;
 /**
  * Turn the airport coordinates and other lat/lon coords into decimal type
  */
-return new class() extends Migration {
+class ModifyAirportsCoordinates extends Migration
+{
     /**
      * Run the migrations.
      *
@@ -24,4 +25,4 @@ return new class() extends Migration {
             $table->decimal('lon', 11, 5)->change()->default(0.0)->nullable();
         });
     }
-};
+}

app/Database/migrations/2020_03_04_044305_flight_field_nullable.php

@@ -6,11 +6,12 @@ use Illuminate\Support\Facades\Schema;
 /**
  * Allow the flight field value to be nullable
  */
-return new class() extends Migration {
+class FlightFieldNullable extends Migration
+{
     public function up()
     {
         Schema::table('flight_field_values', function ($table) {
             $table->text('value')->change()->nullable();
         });
     }
-};
+}

app/Database/migrations/2020_03_05_141152_flights_add_load_factor.php

@@ -7,7 +7,8 @@ use Illuminate\Support\Facades\Schema;
 /**
  * Add a `load_factor` and `load_factor_variance` columns to the expenses table
  */
-return new class() extends Migration {
+class FlightsAddLoadFactor extends Migration
+{
     public function up()
     {
         Schema::table('flights', function (Blueprint $table) {
@@ -28,4 +29,4 @@ return new class() extends Migration {
             $table->dropColumn('load_factor_variance');
         });
     }
-};
+}

app/Database/migrations/2020_03_06_141152_flights_add_pilot_pay.php

@@ -7,7 +7,8 @@ use Illuminate\Support\Facades\Schema;
 /**
  * Add a `pilot_pay` column for a fixed amount to pay to a pilot for a flight
  */
-return new class() extends Migration {
+class FlightsAddPilotPay extends Migration
+{
     public function up()
     {
         Schema::table('flights', function (Blueprint $table) {
@@ -23,4 +24,4 @@ return new class() extends Migration {
             $table->dropColumn('pilot_pay');
         });
     }
-};
+}

app/Database/migrations/2020_03_06_141153_fares_add_type.php

@@ -8,7 +8,8 @@ use Illuminate\Support\Facades\Schema;
 /**
  * Add a `pilot_pay` column for a fixed amount to pay to a pilot for a flight
  */
-return new class() extends Migration {
+class FaresAddType extends Migration
+{
     public function up()
     {
         Schema::table('fares', function (Blueprint $table) {
@@ -18,4 +19,4 @@ return new class() extends Migration {
                 ->after('capacity');
         });
     }
-};
+}

app/Database/migrations/2020_03_09_141152_increase_id_lengths.php

@@ -7,7 +7,8 @@ use Illuminate\Support\Facades\Schema;
 /**
  * Increase string ID lengths because of collisions
  */
-return new class() extends Migration {
+class IncreaseIdLengths extends Migration
+{
     public function up()
     {
         $tables = [
@@ -32,4 +33,4 @@ return new class() extends Migration {
             });
         }
     }
-};
+}

app/Database/migrations/2020_03_09_141153_remove_subfleet_type_index.php

@@ -7,11 +7,12 @@ use Illuminate\Support\Facades\Schema;
 /**
  * Remove the unique index from subfleets.type
  */
-return new class() extends Migration {
+class RemoveSubfleetTypeIndex extends Migration
+{
     public function up()
     {
         Schema::table('subfleets', function (Blueprint $table) {
             $table->dropUnique(['type']);
         });
     }
-};
+}

app/Database/migrations/2020_03_11_141153_add_simbrief_table.php

@@ -7,7 +7,8 @@ use Illuminate\Support\Facades\Schema;
 /**
  * Add a table to store the Simbrief data
  */
-return new class() extends Migration {
+class AddSimbriefTable extends Migration
+{
     public function up()
     {
         Schema::create('simbrief', function (Blueprint $table) {
@@ -30,4 +31,4 @@ return new class() extends Migration {
     {
         Schema::drop('simbrief');
     }
-};
+}

app/Database/migrations/2020_03_27_174238_create_pages.php

@@ -9,7 +9,8 @@ use Illuminate\Support\Facades\Schema;
  * Create the pages
  * https://github.com/nabeelio/phpvms/issues/641
  */
-return new class() extends Migration {
+class CreatePages extends Migration
+{
     public function up()
     {
         Schema::create('pages', function (Blueprint $table) {
@@ -31,4 +32,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('pages');
     }
-};
+}

app/Database/migrations/2020_03_28_174238_airline_remove_nullable.php

@@ -4,11 +4,12 @@ use App\Contracts\Migration;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
-return new class() extends Migration {
+class AirlineRemoveNullable extends Migration
+{
     public function up()
     {
         Schema::table('airlines', function (Blueprint $table) {
             $table->dropUnique(['iata']);
         });
     }
-};
+}

app/Database/migrations/2020_03_28_174238_page_icon_nullable.php

@@ -4,11 +4,12 @@ use App\Contracts\Migration;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
-return new class() extends Migration {
+class PageIconNullable extends Migration
+{
     public function up()
     {
         Schema::table('pages', function (Blueprint $table) {
             $table->string('icon')->change()->nullable();
         });
     }
-};
+}

app/Database/migrations/2020_06_09_141153_pages_add_link.php

@@ -8,7 +8,8 @@ use Illuminate\Support\Facades\Schema;
  * Add a `link` column and make the body optional. Also add a "new_window" bool
  * which determines if we open this link in a new window or not
  */
-return new class() extends Migration {
+class PagesAddLink extends Migration
+{
     public function up()
     {
         Schema::table('pages', function (Blueprint $table) {
@@ -29,4 +30,4 @@ return new class() extends Migration {
             $table->dropColumn('new_window');
         });
     }
-};
+}

app/Database/migrations/2020_07_21_141153_create_user_fields.php

@@ -8,7 +8,8 @@ use Illuminate\Support\Facades\Schema;
 /**
  * Add two tables for holding user fields and the values
  */
-return new class() extends Migration {
+class CreateUserFields extends Migration
+{
     public function up()
     {
         /*
@@ -38,4 +39,4 @@ return new class() extends Migration {
             $table->index(['user_field_id', 'user_id']);
         });
     }
-};
+}

app/Database/migrations/2020_09_03_141152_aircraft_add_mtow.php

@@ -7,7 +7,8 @@ use Illuminate\Support\Facades\Schema;
 /**
  * Add a `mtow` column for the max takeoff weight
  */
-return new class() extends Migration {
+class AircraftAddMtow extends Migration
+{
     public function up()
     {
         Schema::table('aircraft', function (Blueprint $table) {
@@ -24,4 +25,4 @@ return new class() extends Migration {
             $table->dropColumn('mtow');
         });
     }
-};
+}

app/Database/migrations/2020_09_30_081536_create_modules_table.php

@@ -4,7 +4,8 @@ use App\Contracts\Migration;
 use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Support\Facades\Schema;
 
-return new class() extends Migration {
+class CreateModulesTable extends Migration
+{
     public function up()
     {
         Schema::create('modules', function (Blueprint $table) {
@@ -25,4 +26,4 @@ return new class() extends Migration {
     {
         Schema::dropIfExists('modules');
     }
-};
+}

app/Database/migrations/2020_11_20_044305_modify_pages_size.php

@@ -7,11 +7,12 @@ use Illuminate\Support\Facades\Schema;
 /**
  * Change the pages body column type to a Medium Text, max size of 16MB
  */
-return new class() extends Migration {
+class ModifyPagesSize extends Migration
+{
     public function up()
     {
         Schema::table('pages', function (Blueprint $table) {
             $table->mediumText('body')->change()->nullable();
         });
     }
-};
+}

app/Database/migrations/2020_11_26_044305_modify_download_link_size.php

@@ -7,7 +7,8 @@ use Illuminate\Support\Facades\Schema;
 /**
  * Change the downloads link size
  */
-return new class() extends Migration {
+class ModifyDownloadLinkSize extends Migration
+{
     public function up()
     {
         Schema::table('files', function (Blueprint $table) {
@@ -15,4 +16,4 @@ return new class() extends Migration {
             $table->mediumText('path')->change()->nullable();
         });
     }
-};
+}

app/Database/migrations/2021_01_17_044305_add_hub_to_subfleets.php

@@ -7,7 +7,8 @@ use Illuminate\Support\Facades\Schema;
 /**
  * Add a hub to the subfleet is
  */
-return new class() extends Migration {
+class AddHubToSubfleets extends Migration
+{
     public function up()
     {
         Schema::table('subfleets', function (Blueprint $table) {
@@ -16,4 +17,4 @@ return new class() extends Migration {
                 ->after('airline_id');
         });
     }
-};
+}

app/Database/migrations/2021_02_10_044305_change_acars_vs_type.php

@@ -7,11 +7,12 @@ use Illuminate\Support\Facades\Schema;
 /**
  * Change the vertical speed for the acars table to a double
  */
-return new class() extends Migration {
+class ChangeAcarsVsType extends Migration
+{
     public function up()
     {
         Schema::table('acars', function (Blueprint $table) {
             $table->float('vs')->change()->default(0.0)->nullable();
         });
     }
-};
+}

app/Database/migrations/2021_02_11_044305_update_sessions_table.php

@@ -7,7 +7,8 @@ use Illuminate\Support\Facades\Schema;
 /**
  * Bring the sessions table in line with the latest
  */
-return new class() extends Migration {
+class UpdateSessionsTable extends Migration
+{
     public function up()
     {
         Schema::table('sessions', function (Blueprint $table) {
@@ -15,4 +16,4 @@ return new class() extends Migration {
             $table->index('last_activity');
         });
     }
-};
+}