From 2e0f0b00efb675844d4937882f7a3160c578475f Mon Sep 17 00:00:00 2001
From: Nabeel Shahzad <nabeel@nabeel.sh>
Date: Thu, 21 Jan 2021 09:21:39 -0500
Subject: [PATCH] Ensure simbrief id is assigned

---
 app/Repositories/FlightRepository.php |  2 +-
 app/Services/PirepService.php         | 16 ++++++++++++----
 2 files changed, 13 insertions(+), 5 deletions(-)

diff --git a/app/Repositories/FlightRepository.php b/app/Repositories/FlightRepository.php
index f74e1e18..0ab45f75 100644
--- a/app/Repositories/FlightRepository.php
+++ b/app/Repositories/FlightRepository.php
@@ -94,7 +94,7 @@ class FlightRepository extends Repository implements CacheableInterface
             $where['flight_number'] = $request->input('flight_number');
         }
 
-        if ($request->filled('flight_type')) {
+        if ($request->filled('flight_type') && $request->input('flight_type') !== '0') {
             $where['flight_type'] = $request->input('flight_type');
         }
 
diff --git a/app/Services/PirepService.php b/app/Services/PirepService.php
index d7cad0e6..e3263b1a 100644
--- a/app/Services/PirepService.php
+++ b/app/Services/PirepService.php
@@ -24,6 +24,7 @@ use App\Models\Enums\PirepStatus;
 use App\Models\Navdata;
 use App\Models\Pirep;
 use App\Models\PirepFieldValue;
+use App\Models\SimBrief;
 use App\Models\User;
 use App\Repositories\AircraftRepository;
 use App\Repositories\AirportRepository;
@@ -40,17 +41,16 @@ class PirepService extends Service
     private $airportSvc;
     private $geoSvc;
     private $pirepRepo;
+    private $simbriefSvc;
     private $userSvc;
 
     /**
      * @param AircraftRepository $aircraftRepo
      * @param GeoService         $geoSvc
-     * @param PirepRepository    $pirepRepo
-     * @param UserService        $userSvc
      * @param AirportRepository  $airportRepo
      * @param AirportService     $airportSvc
-     * @param GeoService         $geoSvc
      * @param PirepRepository    $pirepRepo
+     * @param SimBriefService    $simbriefSvc
      * @param UserService        $userSvc
      */
     public function __construct(
@@ -59,14 +59,16 @@ class PirepService extends Service
         AircraftRepository $aircraftRepo,
         GeoService $geoSvc,
         PirepRepository $pirepRepo,
+        SimBriefService $simbriefSvc,
         UserService $userSvc
     ) {
         $this->airportRepo = $airportRepo;
         $this->airportSvc = $airportSvc;
         $this->aircraftRepo = $aircraftRepo;
         $this->geoSvc = $geoSvc;
-        $this->userSvc = $userSvc;
         $this->pirepRepo = $pirepRepo;
+        $this->simbriefSvc = $simbriefSvc;
+        $this->userSvc = $userSvc;
     }
 
     /**
@@ -149,6 +151,12 @@ class PirepService extends Service
 
         $pirep->save();
 
+        // Check of there is a simbrief_id
+        if (array_key_exists('simbrief_id', $attrs)) {
+            $simbrief = SimBrief::find($attrs['simbrief_id']);
+            $this->simbriefSvc->attachSimbriefToPirep($pirep, $simbrief);
+        }
+
         return $pirep;
     }