Only show flights if they're also marked as visible

This commit is contained in:
Nabeel Shahzad
2018-07-12 22:29:50 -05:00
parent 4ca58ed754
commit 0af85a3fff
4 changed files with 22 additions and 6 deletions

View File

@@ -40,7 +40,6 @@ class SetActiveFlights extends Listener
* @var Flight $flight
*/
foreach($flights as $flight) {
if (!$flight->active) {
continue;
}
@@ -54,6 +53,9 @@ class SetActiveFlights extends Listener
Log::info('Marking flight '.$flight->ident.' to '.($visible ? 'visible' : 'invisible'));
$flight->visible = $visible;
}
} else {
Log::info('Toggling flight '.$flight->ident.' to visible');
$flight->visible = true;
}
$flight->save();
@@ -72,7 +74,10 @@ class SetActiveFlights extends Listener
$flight->visible = true;
}
} else {
$flight->visible = true;
if ($flight->visible !== true) {
Log::info('Toggling flight '.$flight->ident.' to visible');
$flight->visible = true;
}
}
} else {
$flight->visible = false;

View File

@@ -44,7 +44,11 @@ class FlightController extends Controller
{
$user = Auth::user();
$where = ['active' => true];
$where = [
'active' => true,
'visible' => true,
];
if(setting('pilots.restrict_to_company')) {
$where['airline_id'] = Auth::user()->airline_id;
}
@@ -84,7 +88,11 @@ class FlightController extends Controller
$user = Auth::user();
try {
$where = ['active' => true];
$where = [
'active' => true,
'visible' => true,
];
if(setting('pilots.restrict_to_company')) {
$where['airline_id'] = Auth::user()->airline_id;
}

View File

@@ -52,8 +52,10 @@ class FlightController extends Controller
public function index(Request $request)
{
$where = [
'active' => true,
'active' => true,
'visible' => true,
];
if(setting('pilots.restrict_to_company')) {
$where['airline_id'] = Auth::user()->airline_id;
}

View File

@@ -71,7 +71,8 @@ class FlightRepository extends Repository implements CacheableInterface
$where = [];
if ($only_active === true) {
$where['active'] = $only_active;
$where['active'] = $only_active;
$where['visible'] = $only_active;
}
if ($request->filled('flight_id')) {