FGFS-4.1/phpvms
3
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Extra logging around fares

Browse Source
This commit is contained in:
Nabeel Shahzad
2020-10-13 18:10:31 -04:00
parent d001704880
commit bf878a98a4
2 changed files with 12 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
app/Http/Controllers/Admin/FareController.php
View File

@@ -12,6 +12,7 @@ use App\Repositories\FareRepository;
use App\Services\ExportService; use App\Services\ExportService;
use App\Services\ImportService; use App\Services\ImportService;
use Illuminate\Http\Request; use Illuminate\Http\Request;
use Illuminate\Support\Facades\Log;
use Laracasts\Flash\Flash; use Laracasts\Flash\Flash;
use Prettus\Repository\Criteria\RequestCriteria; use Prettus\Repository\Criteria\RequestCriteria;
@@ -162,6 +163,8 @@ class FareController extends Controller
return redirect(route('admin.fares.index')); return redirect(route('admin.fares.index'));
} }
Log::info('Deleting fare "'.$fare->name.'", id='.$fare->id);
$this->fareRepo->delete($id); $this->fareRepo->delete($id);
Flash::success('Fare deleted successfully.'); Flash::success('Fare deleted successfully.');

10
app/Services/FareService.php
View File

@@ -9,6 +9,7 @@ use App\Models\Pirep;
use App\Models\PirepFare; use App\Models\PirepFare;
use App\Models\Subfleet; use App\Models\Subfleet;
use App\Support\Math; use App\Support\Math;
use Illuminate\Support\Facades\Log;
use function count; use function count;
use Illuminate\Support\Collection; use Illuminate\Support\Collection;
@@ -106,6 +107,8 @@ class FareService extends Service
*/ */
public function setForFlight(Flight $flight, Fare $fare, array $override = []): Flight public function setForFlight(Flight $flight, Fare $fare, array $override = []): Flight
{ {
Log::info('Setting fare "'.$fare->name.'" to flight "'.$flight->ident.'"');
$flight->fares()->syncWithoutDetaching([$fare->id]); $flight->fares()->syncWithoutDetaching([$fare->id]);
// modify any pivot values? // modify any pivot values?
@@ -145,6 +148,8 @@ class FareService extends Service
*/ */
public function delFareFromFlight(Flight $flight, Fare $fare) public function delFareFromFlight(Flight $flight, Fare $fare)
{ {
Log::info('Removing fare "'.$fare->name.'" to flight "'.$flight->ident.'"');
$flight->fares()->detach($fare->id); $flight->fares()->detach($fare->id);
$flight->refresh(); $flight->refresh();
@@ -162,6 +167,8 @@ class FareService extends Service
*/ */
public function setForSubfleet(Subfleet $subfleet, Fare $fare, array $override = []): Subfleet public function setForSubfleet(Subfleet $subfleet, Fare $fare, array $override = []): Subfleet
{ {
Log::info('Setting fare "'.$fare->name.'" to subfleet "'.$subfleet->name.'"');
$subfleet->fares()->syncWithoutDetaching([$fare->id]); $subfleet->fares()->syncWithoutDetaching([$fare->id]);
// modify any pivot values? // modify any pivot values?
@@ -203,6 +210,8 @@ class FareService extends Service
*/ */
public function delFareFromSubfleet(Subfleet &$subfleet, Fare &$fare) public function delFareFromSubfleet(Subfleet &$subfleet, Fare &$fare)
{ {
Log::info('Removing fare "'.$fare->name.'" from subfleet "'.$subfleet->name.'"');
$subfleet->fares()->detach($fare->id); $subfleet->fares()->detach($fare->id);
$subfleet->refresh(); $subfleet->refresh();
@@ -219,7 +228,6 @@ class FareService extends Service
*/ */
public function getForPirep(Pirep $pirep) public function getForPirep(Pirep $pirep)
{ {
$fares = [];
$found_fares = PirepFare::where('pirep_id', $pirep->id)->get(); $found_fares = PirepFare::where('pirep_id', $pirep->id)->get();
return $found_fares; return $found_fares;