diff --git a/app/Http/Controllers/Admin/FlightController.php b/app/Http/Controllers/Admin/FlightController.php index c5358cb2..6419af17 100644 --- a/app/Http/Controllers/Admin/FlightController.php +++ b/app/Http/Controllers/Admin/FlightController.php @@ -72,7 +72,7 @@ class FlightController extends BaseController return view('admin.flights.create', [ 'flight' => null, 'airlines' => $this->airlineRepo->selectBoxList(), - 'airports' => $this->airportRepo->selectBoxList(), + 'airports' => $this->airportRepo->selectBoxList(true, false), ]); } diff --git a/app/Http/Controllers/Admin/PirepFieldController.php b/app/Http/Controllers/Admin/PirepFieldController.php index 3866e8a3..0bc77aeb 100644 --- a/app/Http/Controllers/Admin/PirepFieldController.php +++ b/app/Http/Controllers/Admin/PirepFieldController.php @@ -31,7 +31,7 @@ class PirepFieldController extends BaseController $this->pirepFieldRepo->pushCriteria(new RequestCriteria($request)); $fields = $this->pirepFieldRepo->all(); - return view('admin.pirepFields.index', [ + return view('admin.pirepfields.index', [ 'fields' => $fields, ]); } @@ -43,7 +43,7 @@ class PirepFieldController extends BaseController */ public function create() { - return view('admin.pirepFields.create'); + return view('admin.pirepfields.create'); } /** @@ -60,7 +60,7 @@ class PirepFieldController extends BaseController $field = $this->pirepFieldRepo->create($input); Flash::success('PirepField saved successfully.'); - return redirect(route('admin.pirepFields.index')); + return redirect(route('admin.pirepfields.index')); } /** @@ -76,10 +76,10 @@ class PirepFieldController extends BaseController if (empty($field)) { Flash::error('PirepField not found'); - return redirect(route('admin.pirepFields.index')); + return redirect(route('admin.pirepfields.index')); } - return view('admin.pirepFields.show', [ + return view('admin.pirepfields.show', [ 'field' => $field, ]); } @@ -97,10 +97,10 @@ class PirepFieldController extends BaseController if (empty($field)) { Flash::error('PirepField not found'); - return redirect(route('admin.pirepFields.index')); + return redirect(route('admin.pirepfields.index')); } - return view('admin.pirepFields.edit', [ + return view('admin.pirepfields.edit', [ 'field' => $field, ]); } @@ -119,13 +119,13 @@ class PirepFieldController extends BaseController if (empty($field)) { Flash::error('PirepField not found'); - return redirect(route('admin.pirepFields.index')); + return redirect(route('admin.pirepfields.index')); } $field = $this->pirepFieldRepo->update($request->all(), $id); Flash::success('PirepField updated successfully.'); - return redirect(route('admin.pirepFields.index')); + return redirect(route('admin.pirepfields.index')); } /** @@ -141,12 +141,12 @@ class PirepFieldController extends BaseController if (empty($field)) { Flash::error('PirepField not found'); - return redirect(route('admin.pirepFields.index')); + return redirect(route('admin.pirepfields.index')); } $this->pirepFieldRepo->delete($id); Flash::success('PirepField deleted successfully.'); - return redirect(route('admin.pirepFields.index')); + return redirect(route('admin.pirepfields.index')); } } diff --git a/app/Models/Acars.php b/app/Models/Acars.php index 3a895335..4e270054 100644 --- a/app/Models/Acars.php +++ b/app/Models/Acars.php @@ -7,9 +7,9 @@ use App\Models\Traits\HashId; class Acars extends BaseModel { use HashId; - public $incrementing = false; public $table = 'acars'; + public $incrementing = false; public $fillable = [ 'pirep_id', @@ -37,6 +37,10 @@ class Acars extends BaseModel 'fuel_flow' => 'float', ]; + public static $rules = [ + 'pirep_id' => 'required', + ]; + /** * FKs */ diff --git a/app/Models/Aircraft.php b/app/Models/Aircraft.php index bf012945..cbd77ed0 100644 --- a/app/Models/Aircraft.php +++ b/app/Models/Aircraft.php @@ -6,8 +6,6 @@ class Aircraft extends BaseModel { public $table = 'aircraft'; - protected $dates = ['deleted_at']; - public $fillable = [ 'subfleet_id', 'airport_id', diff --git a/app/Models/Airline.php b/app/Models/Airline.php index 34f7afd0..403cf2d6 100644 --- a/app/Models/Airline.php +++ b/app/Models/Airline.php @@ -34,9 +34,11 @@ class Airline extends BaseModel * @var array */ public static $rules = [ - 'iata' => 'required|max:5', - 'icao' => 'required|max:5', - 'name' => 'required', + 'country' => 'nullable', + 'iata' => 'nullable|max:5', + 'icao' => 'required|max:5', + 'logo' => 'nullable', + 'name' => 'required', ]; /** @@ -46,4 +48,21 @@ class Airline extends BaseModel return $this->icao; } + protected static function boot() + { + parent::boot(); + + /** + * IATA and ICAO should be in all caps + */ + static::creating(function (Airline $model) { + if (!empty($model->iata)) { + $model->iata = strtoupper($model->iata); + } + + if (!empty($model->icao)) { + $model->icao = strtoupper($model->icao); + } + }); + } } diff --git a/app/Models/Airport.php b/app/Models/Airport.php index b4cad11e..6dbb145b 100644 --- a/app/Models/Airport.php +++ b/app/Models/Airport.php @@ -38,14 +38,12 @@ class Airport extends BaseModel /** * Validation rules - * - * @var array */ public static $rules = [ - 'icao' => 'required', - 'name' => 'required', - 'lat' => 'required', - 'lon' => 'required', + 'icao' => 'required', + 'name' => 'required', + 'lat' => 'required', + 'lon' => 'required', ]; /** diff --git a/app/Models/Setting.php b/app/Models/Setting.php index 8d772003..2687b55c 100644 --- a/app/Models/Setting.php +++ b/app/Models/Setting.php @@ -35,5 +35,4 @@ class Setting extends BaseModel } }); } - } diff --git a/resources/views/admin/aircraft/fields.blade.php b/resources/views/admin/aircraft/fields.blade.php index 0fe2eed8..4215238e 100644 --- a/resources/views/admin/aircraft/fields.blade.php +++ b/resources/views/admin/aircraft/fields.blade.php @@ -1,7 +1,7 @@
{{ $errors->first('airline_id') }}
{{ $errors->first('flight_number') }}
{{ $errors->first('dpt_airport_id') }}
{{ $errors->first('arr_airport_id') }}