diff --git a/app/Http/Controllers/Frontend/PirepController.php b/app/Http/Controllers/Frontend/PirepController.php index 4be10783..0ebfde17 100644 --- a/app/Http/Controllers/Frontend/PirepController.php +++ b/app/Http/Controllers/Frontend/PirepController.php @@ -8,6 +8,7 @@ use App\Http\Requests\CreatePirepRequest; use App\Http\Requests\UpdatePirepRequest; use App\Models\Enums\PirepSource; use App\Models\Enums\PirepState; +use App\Models\Enums\PirepStatus; use App\Models\Pirep; use App\Repositories\AircraftRepository; use App\Repositories\AirlineRepository; @@ -21,11 +22,10 @@ use App\Services\PirepService; use App\Services\UserService; use App\Support\Units\Time; use Carbon\Carbon; -use Flash; use Illuminate\Http\Request; use Illuminate\Support\Facades\Auth; use Illuminate\Support\Facades\Log; -use PirepStatus; +use Laracasts\Flash\Flash; /** * Class PirepController @@ -81,15 +81,14 @@ class PirepController extends Controller /** * Dropdown with aircraft grouped by subfleet * - * @param null $user * @param mixed $add_blank * * @return array */ - public function aircraftList($user = null, $add_blank = false) + public function aircraftList($add_blank = false) { $aircraft = []; - $subfleets = $this->userSvc->getAllowableSubfleets($user); + $subfleets = $this->userSvc->getAllowableSubfleets(Auth::user()); if ($add_blank) { $aircraft[''] = ''; @@ -236,13 +235,11 @@ class PirepController extends Controller */ public function create() { - $user = Auth::user(); - return view('pireps.create', [ 'aircraft' => null, 'read_only' => false, 'airline_list' => $this->airlineRepo->selectBoxList(true), - 'aircraft_list' => $this->aircraftList($user, true), + 'aircraft_list' => $this->aircraftList(true), 'airport_list' => $this->airportRepo->selectBoxList(true), 'pirep_fields' => $this->pirepFieldRepo->all(), 'field_values' => [], @@ -354,7 +351,9 @@ class PirepController extends Controller } // Eager load the subfleet and fares under it - $pirep->aircraft->load('subfleet.fares'); + if ($pirep->aircraft) { + $pirep->aircraft->load('subfleet.fares'); + } $time = new Time($pirep->flight_time); $pirep->hours = $time->hours; @@ -379,7 +378,7 @@ class PirepController extends Controller return view('pireps.edit', [ 'pirep' => $pirep, 'aircraft' => $pirep->aircraft, - 'aircraft_list' => $this->aircraftList(), + 'aircraft_list' => $this->aircraftList(true), 'airline_list' => $this->airlineRepo->selectBoxList(), 'airport_list' => $this->airportRepo->selectBoxList(), 'pirep_fields' => $this->pirepFieldRepo->all(), @@ -447,6 +446,8 @@ class PirepController extends Controller * @param $id * @param Request $request * + * @throws \Exception + * * @return \Illuminate\Http\RedirectResponse|\Illuminate\Routing\Redirector */ public function submit($id, Request $request) diff --git a/resources/views/layouts/default/pireps/fields.blade.php b/resources/views/layouts/default/pireps/fields.blade.php index 7d8ad6c2..1209e8ea 100644 --- a/resources/views/layouts/default/pireps/fields.blade.php +++ b/resources/views/layouts/default/pireps/fields.blade.php @@ -104,7 +104,7 @@ flight reports that have been filed. You've been warned! {{ Form::hidden('minutes') }}

@else -
+
{{ Form::number('hours', null, [ 'class' => 'form-control', 'placeholder' => trans_choice('common.hour', 2),