API: Flight fields are an array when empty #618 (#619)

* Ensure 'fields' field is return as an object for flights #618 * Eager loading for flight fields
nabeelio · Mar 4, 2020 · 9ed2e3f · 9ed2e3f
1 parent 174b602
commit 9ed2e3f
Show file tree

Hide file tree

Showing 3 changed files with 19 additions and 4 deletions.
diff --git a/app/Http/Controllers/Api/FlightController.php b/app/Http/Controllers/Api/FlightController.php
@@ -88,7 +88,15 @@ public function search(Request $request)
             $this->flightRepo->pushCriteria(new WhereCriteria($request, $where));
             $this->flightRepo->pushCriteria(new RequestCriteria($request));
 
-            $flights = $this->flightRepo->paginate();
+            $flights = $this->flightRepo
+                ->with([
+                    'airline',
+                    'subfleets',
+                    'subfleets.aircraft',
+                    'subfleets.fares',
+                    'field_values',
+                ])
+                ->paginate();
         } catch (RepositoryException $e) {
             return response($e, 503);
         }

diff --git a/app/Http/Resources/Flight.php b/app/Http/Resources/Flight.php
@@ -3,18 +3,25 @@
 namespace App\Http\Resources;
 
 use App\Support\Units\Distance;
+use stdClass;
 
 class Flight extends Response
 {
     /**
      * Set the fields on the flight object
      *
-     * @return array
+     * @mixin \App\Models\Flight
      */
     private function setFields()
     {
+        /** @var \Illuminate\Support\Collection $field_values */
+        $field_values = $this->field_values;
+        if (empty($field_values) || $field_values->count() === 0) {
+            return new stdClass();
+        }
+
         $fields = [];
-        foreach ($this->field_values as $field) {
+        foreach ($field_values as $field) {
             $fields[$field->name] = $field->value;
         }
 

diff --git a/resources/views/admin/flights/fields.blade.php b/resources/views/admin/flights/fields.blade.php
@@ -235,7 +235,7 @@ class="required">*</span>
     <div class="checkbox">
       <label class="checkbox-inline">
         {{ Form::label('active', 'Active:') }}
-        {{ Form::hidden('active', 0, false) }}
+        <input name="active" type="hidden" value="0" />
         {{ Form::checkbox('active') }}
       </label>
     </div>