Add fare type for pax/cargo/mixed flights #621 (#623)

nabeelio · Mar 6, 2020 · 632c578 · 632c578
1 parent 9f3ddd5
commit 632c578
Show file tree

Hide file tree

Showing 12 changed files with 148 additions and 44 deletions.
diff --git a/app/Database/migrations/2020_03_06_141153_fares_add_type.php b/app/Database/migrations/2020_03_06_141153_fares_add_type.php
@@ -0,0 +1,34 @@
+<?php
+
+use App\Models\Enums\FareType;
+use Illuminate\Database\Migrations\Migration;
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Support\Facades\Schema;
+
+class FaresAddType extends Migration
+{
+    /**
+     * Add a `pilot_pay` column for a fixed amount to pay to a pilot for a flight
+     */
+    public function up()
+    {
+        Schema::table('fares', function (Blueprint $table) {
+            $table->unsignedTinyInteger('type')
+                ->default(FareType::PASSENGER)
+                ->nullable()
+                ->after('capacity');
+        });
+    }
+
+    /**
+     * Reverse the migrations.
+     *
+     * @return void
+     */
+    public function down()
+    {
+        Schema::table('fares', function (Blueprint $table) {
+            $table->dropColumn('type');
+        });
+    }
+}
diff --git a/app/Http/Controllers/Admin/FareController.php b/app/Http/Controllers/Admin/FareController.php
@@ -6,6 +6,7 @@
 use App\Http\Controllers\Admin\Traits\Importable;
 use App\Http\Requests\CreateFareRequest;
 use App\Http\Requests\UpdateFareRequest;
+use App\Models\Enums\FareType;
 use App\Models\Enums\ImportExportType;
 use App\Repositories\FareRepository;
 use App\Services\ExportService;
@@ -58,7 +59,9 @@ public function index(Request $request)
      */
     public function create()
     {
-        return view('admin.fares.create');
+        return view('admin.fares.create', [
+            'fare_types' => FareType::select(),
+        ]);
     }
 
     /**
@@ -94,7 +97,9 @@ public function show($id)
             return redirect(route('admin.fares.index'));
         }
 
-        return view('admin.fares.show')->with('fare', $fare);
+        return view('admin.fares.show', [
+            'fare' => $fare,
+        ]);
     }
 
     /**
@@ -112,7 +117,10 @@ public function edit($id)
             return redirect(route('admin.fares.index'));
         }
 
-        return view('admin.fares.edit')->with('fare', $fare);
+        return view('admin.fares.edit', [
+            'fare'       => $fare,
+            'fare_types' => FareType::select(),
+        ]);
     }
 
     /**

diff --git a/app/Http/Controllers/Admin/SubfleetController.php b/app/Http/Controllers/Admin/SubfleetController.php
@@ -7,6 +7,7 @@
 use App\Http\Requests\CreateSubfleetRequest;
 use App\Http\Requests\UpdateSubfleetRequest;
 use App\Models\Airline;
+use App\Models\Enums\FareType;
 use App\Models\Enums\FuelType;
 use App\Models\Enums\ImportExportType;
 use App\Models\Expense;
@@ -98,6 +99,7 @@ protected function getAvailFares($subfleet)
         foreach ($avail_fares as $fare) {
             $retval[$fare->id] = $fare->name.
                 ' (price: '.$fare->price.
+                ', type: '.FareType::label($fare->type).
                 ', cost: '.$fare->cost.
                 ', capacity: '.$fare->capacity.')';
         }
@@ -162,7 +164,9 @@ public function store(CreateSubfleetRequest $request)
      */
     public function show($id)
     {
-        $subfleet = $this->subfleetRepo->findWithoutFail($id);
+        $subfleet = $this->subfleetRepo
+            ->with(['fares'])
+            ->findWithoutFail($id);
 
         if (empty($subfleet)) {
             Flash::error('Subfleet not found');
@@ -185,7 +189,9 @@ public function show($id)
      */
     public function edit($id)
     {
-        $subfleet = $this->subfleetRepo->findWithoutFail($id);
+        $subfleet = $this->subfleetRepo
+            ->with(['fares', 'ranks'])
+            ->findWithoutFail($id);
 
         if (empty($subfleet)) {
             Flash::error('Subfleet not found');

diff --git a/app/Http/Resources/Fare.php b/app/Http/Resources/Fare.php
@@ -18,6 +18,7 @@ public function toArray($request)
             'price'    => $this->price,
             'cost'     => $this->cost,
             'capacity' => $this->capacity,
+            'type'     => $this->type,
             'notes'    => $this->notes,
             'active'   => $this->active,
         ];

diff --git a/app/Models/Enums/FareType.php b/app/Models/Enums/FareType.php
@@ -0,0 +1,16 @@
+<?php
+
+namespace App\Models\Enums;
+
+use App\Contracts\Enum;
+
+class FareType extends Enum
+{
+    public const PASSENGER = 0;
+    public const CARGO = 1;
+
+    public static $labels = [
+        self::PASSENGER => 'Passenger',
+        self::CARGO     => 'Cargo',
+    ];
+}
diff --git a/app/Models/Expense.php b/app/Models/Expense.php
@@ -78,4 +78,9 @@ public function airline()
     {
         return $this->belongsTo(Airline::class, 'airline_id');
     }
+
+    public function ref_model()
+    {
+        return $this->morphTo();
+    }
 }
diff --git a/app/Models/Fare.php b/app/Models/Fare.php
@@ -11,6 +11,7 @@
  * @property int     code
  * @property int     capacity
  * @property int     count Only when merged with pivot
+ * @property int     type
  * @property string  notes
  * @property bool    active
  */
@@ -32,12 +33,14 @@ class Fare extends Model
         'price'    => 'float',
         'cost'     => 'float',
         'capacity' => 'integer',
+        'type'     => 'integer',
         'active'   => 'boolean',
     ];
 
     public static $rules = [
         'code' => 'required',
         'name' => 'required',
+        'type' => 'required',
     ];
 
     /**

diff --git a/app/Services/Finance/PirepFinanceService.php b/app/Services/Finance/PirepFinanceService.php
@@ -7,6 +7,7 @@
 use App\Models\Aircraft;
 use App\Models\Airport;
 use App\Models\Enums\ExpenseType;
+use App\Models\Enums\FareType;
 use App\Models\Enums\PirepSource;
 use App\Models\Expense;
 use App\Models\Pirep;
@@ -124,13 +125,15 @@ public function payFaresForPirep($pirep): void
 
             Log::info('Finance: Calculate: C='.$credit->toAmount().', D='.$debit->toAmount());
 
+            $memo = FareType::label($fare->type).' fare: '.$fare->code.$fare->count
+                .'; price: '.$fare->price.', cost: '.$fare->cost;
+
             $this->journalRepo->post(
                 $pirep->airline->journal,
                 $credit,
                 $debit,
                 $pirep,
-                'Fares '.$fare->code.$fare->count
-                .'; price: '.$fare->price.', cost: '.$fare->cost,
+                $memo,
                 null,
                 'Fares',
                 'fare'

diff --git a/resources/views/admin/fares/fields.blade.php b/resources/views/admin/fares/fields.blade.php
@@ -9,7 +9,7 @@
   </div>
 </div>
 <div class="row">
-  <div class="form-group col-sm-6">
+  <div class="form-group col-sm-4">
     {{ Form::label('code', 'Code:') }}&nbsp;<span class="required">*</span>
     @component('admin.components.info')
       How this fare class will show up on a ticket
@@ -18,7 +18,7 @@
     <p class="text-danger">{{ $errors->first('code') }}</p>
   </div>
 
-  <div class="form-group col-sm-6">
+  <div class="form-group col-sm-4">
     {{ Form::label('name', 'Name:') }}&nbsp;<span class="required">*</span>
     @component('admin.components.info')
       The fare class name, E.g, "Economy" or "First"
@@ -27,10 +27,25 @@
     <p class="text-danger">{{ $errors->first('name') }}</p>
   </div>
 
+  <div class="form-group col-sm-4">
+    {{ Form::label('type', 'Fare Type:') }}&nbsp;<span class="required">*</span>
+    @component('admin.components.info')
+      If this is a passenger or cargo fare
+    @endcomponent
+    {{ Form::select('type', $fare_types, null , [
+          'id'    => 'type',
+          'class' => 'form-control select2'
+      ]) }}
+    <p class="text-danger">{{ $errors->first('type') }}</p>
+  </div>
+
+</div>
+<div class="row">
+
   <div class="form-group col-sm-6">
     {{ Form::label('price', 'Price:') }}
     @component('admin.components.info')
-      This is the price of a ticket for a passenger
+      This is the price of a ticket or price per {{ setting('units.weight') }}
     @endcomponent
     {{ Form::text('price', null, ['class' => 'form-control', 'placeholder' => 0]) }}
     <p class="text-danger">{{ $errors->first('price') }}</p>
@@ -39,27 +54,33 @@
   <div class="form-group col-sm-6">
     {{ Form::label('cost', 'Cost:') }}
     @component('admin.components.info')
-      The operating cost
+      The operating cost per unit (passenger or {{ setting('units.weight') }})
     @endcomponent
     {{ Form::number('cost', null, ['class' => 'form-control', 'placeholder' => 0, 'step' => '0.01']) }}
     <p class="text-danger">{{ $errors->first('cost') }}</p>
   </div>
-
+</div>
+<div class="row">
   <div class="form-group col-sm-6">
     {{ Form::label('capacity', 'Capacity:') }}
     @component('admin.components.info')
-      The number of seats available in this class.
+      Max seats or capacity available. This can be adjusted in the subfleet
     @endcomponent
     {{ Form::number('capacity', null, ['class' => 'form-control', 'min' => 0]) }}
     <p class="text-danger">{{ $errors->first('capacity') }}</p>
   </div>
 
   <div class="form-group col-sm-6">
     {{ Form::label('notes', 'Notes:') }}
+    @component('admin.components.info')
+      Notes for this fare
+    @endcomponent
     {{ Form::text('notes', null, ['class' => 'form-control']) }}
     <p class="text-danger">{{ $errors->first('notes') }}</p>
   </div>
 
+</div>
+<div class="row">
   <!-- Active Field -->
   <div class="form-group col-sm-12">
     {{ Form::label('active', 'Active:') }}
@@ -68,7 +89,8 @@
       {{ Form::checkbox('active', 1, null) }}
     </label>
   </div>
-
+</div>
+<div class="row">
   <!-- Submit Field -->
   <div class="form-group col-sm-12">
     <div class="pull-right">

diff --git a/resources/views/admin/subfleets/fares.blade.php b/resources/views/admin/subfleets/fares.blade.php
@@ -14,6 +14,7 @@
       <tr>
         <th>Name</th>
         <th style="text-align: center;">Code</th>
+        <th style="text-align: center;">Type</th>
         <th style="text-align: center;">Capacity (default)</th>
         <th style="text-align: center;">Price (default)</th>
         <th style="text-align: center;">Cost (default)</th>
@@ -24,16 +25,17 @@
     <tbody>
     @foreach($subfleet->fares as $atf)
       <tr>
-        <td class="sorting_1">{{ $atf->name }}</td>
-        <td style="text-align: center;">{{ $atf->code }}</td>
-        <td style="text-align: center;">
+        <td class="sorting_1 text-center">{{ $atf->name }}</td>
+        <td class="text-center">{{ $atf->code }}</td>
+        <td class="sorting_1 text-center">{{ \App\Models\Enums\FareType::label($atf->type)  }}</td>
+        <td class="text-center">
           <a href="#" data-pk="{{ $atf->id }}" data-name="capacity">{{ $atf->pivot->capacity }}</a>
           <span class="small background-color-grey-light">({{ $atf->capacity }})</span>
         </td>
-        <td style="text-align: center;">
+        <td class="text-center">
           <a href="#" data-pk="{{ $atf->id }}" data-name="price">{{ $atf->pivot->price }}</a>
           <span class="small background-color-grey-light">({{ $atf->price }})</span></td>
-        <td style="text-align: center;">
+        <td class="text-center">
           <a href="#" data-pk="{{ $atf->id }}" data-name="cost">{{ $atf->pivot->cost }}</a>
           <span class="small background-color-grey-light">({{ $atf->cost}})</span></td>
         <td style="text-align: right; width:3%;">

diff --git a/resources/views/admin/subfleets/fields.blade.php b/resources/views/admin/subfleets/fields.blade.php
@@ -1,32 +1,36 @@
 <div class="row">
-  <div class="col-sm-12">
-    @component('admin.components.info')
-      Subfleets are aircraft groups. The "type" is a short name. Airlines always
-      group aircraft together by feature, so 737s with winglets might have a type of
-      "B.738-WL". You can create as many as you want, you need at least one, though.
+  <div class="col-12">
+    <div class="row">
+      <div class="col-sm-12">
+        @component('admin.components.info')
+          Subfleets are aircraft groups. The "type" is a short name. Airlines always
+          group aircraft together by feature, so 737s with winglets might have a type of
+          "B.738-WL". You can create as many as you want, you need at least one, though.
 
-      Read more about subfleets <a href="{{ docs_link('finances') }}" target="_new">here</a>.
-    @endcomponent
-  </div>
-
-  <div class="form-group col-sm-4">
-    {{ Form::label('airline_id', 'Airline:') }}
-    {{ Form::select('airline_id', $airlines, null , ['class' => 'form-control select2']) }}
-    <p class="text-danger">{{ $errors->first('airline_id') }}</p>
-  </div>
+          Read more about subfleets <a href="{{ docs_link('finances') }}" target="_new">here</a>.
+        @endcomponent
+      </div>
+    </div>
+    <div class="row">
+      <div class="form-group col-sm-4">
+        {{ Form::label('airline_id', 'Airline:') }}
+        {{ Form::select('airline_id', $airlines, null , ['class' => 'form-control select2']) }}
+        <p class="text-danger">{{ $errors->first('airline_id') }}</p>
+      </div>
 
-  <div class="form-group col-sm-4">
-    {{ Form::label('type', 'Type:') }}
-    {{ Form::text('type', null, ['class' => 'form-control']) }}
-    <p class="text-danger">{{ $errors->first('type') }}</p>
-  </div>
+      <div class="form-group col-sm-4">
+        {{ Form::label('type', 'Type:') }}
+        {{ Form::text('type', null, ['class' => 'form-control']) }}
+        <p class="text-danger">{{ $errors->first('type') }}</p>
+      </div>
 
-  <div class="form-group col-sm-4">
-    {{ Form::label('name', 'Name:') }}
-    {{ Form::text('name', null, ['class' => 'form-control']) }}
-    <p class="text-danger">{{ $errors->first('name') }}</p>
+      <div class="form-group col-sm-4">
+        {{ Form::label('name', 'Name:') }}
+        {{ Form::text('name', null, ['class' => 'form-control']) }}
+        <p class="text-danger">{{ $errors->first('name') }}</p>
+      </div>
+    </div>
   </div>
-
 </div>
 <div class="row">
 

diff --git a/resources/views/layouts/default/pireps/fares.blade.php b/resources/views/layouts/default/pireps/fares.blade.php
@@ -7,7 +7,7 @@
       @foreach($aircraft->subfleet->fares as $fare)
         <div class="row">
           <div class="col">
-            {{Form::label('fare_'.$fare->id, $fare->name.' ('.$fare->code.')')}}
+            {{Form::label('fare_'.$fare->id, $fare->name.' ('. \App\Models\Enums\FareType::label($fare->type).', code '.$fare->code.')')}}
             <div class="input-group form-group">
               {{ Form::number('fare_'.$fare->id, null, ['class' => 'form-control', 'min' => 0]) }}
             </div>