diff --git a/app/Exceptions/Handler.php b/app/Exceptions/Handler.php index 947e909a..63d101f2 100755 --- a/app/Exceptions/Handler.php +++ b/app/Exceptions/Handler.php @@ -132,7 +132,6 @@ class Handler extends ExceptionHandler { if ($request->expectsJson() || $request->is('api/*')) { $error = $this->createError(401, 'Unauthenticated'); - return response()->json($error, 401); } @@ -159,8 +158,8 @@ class Handler extends ExceptionHandler 'exception' => $e, 'SKIN_NAME' => config('phpvms.skin'), ], $status, $e->getHeaders()); - } else { - return $this->convertExceptionToResponse($e); } + + return $this->convertExceptionToResponse($e); } } diff --git a/app/Http/Controllers/Api/PirepController.php b/app/Http/Controllers/Api/PirepController.php index 8a46dc30..77f53c67 100644 --- a/app/Http/Controllers/Api/PirepController.php +++ b/app/Http/Controllers/Api/PirepController.php @@ -489,6 +489,7 @@ class PirepController extends Controller * @return \Illuminate\Http\JsonResponse * @throws \App\Exceptions\PirepCancelled * @throws \Symfony\Component\HttpKernel\Exception\BadRequestHttpException + * @throws \Exception */ public function route_post($id, RouteRequest $request) { @@ -506,6 +507,10 @@ class PirepController extends Controller $count = 0; $route = $request->post('route', []); + if (\count($route) === 0) { + return $this->message('No points to add'); + } + foreach ($route as $position) { $position['pirep_id'] = $id; $position['type'] = AcarsType::ROUTE; diff --git a/app/Http/Requests/Acars/RouteRequest.php b/app/Http/Requests/Acars/RouteRequest.php index baa3af0b..f50821b2 100644 --- a/app/Http/Requests/Acars/RouteRequest.php +++ b/app/Http/Requests/Acars/RouteRequest.php @@ -21,12 +21,12 @@ class RouteRequest extends FormRequest public function rules() { $rules = [ - 'route' => 'nullable|array', - /*'route.*.name' => 'required', + 'route' => 'required|array', + 'route.*.name' => 'required', 'route.*.order' => 'required|int', 'route.*.nav_type' => 'nullable|int', 'route.*.lat' => 'required|numeric', - 'route.*.lon' => 'required|numeric',*/ + 'route.*.lon' => 'required|numeric', ]; return $rules; diff --git a/resources/views/admin/pireps/field_values.blade.php b/resources/views/admin/pireps/field_values.blade.php index 06144b54..95b37d3f 100644 --- a/resources/views/admin/pireps/field_values.blade.php +++ b/resources/views/admin/pireps/field_values.blade.php @@ -1,9 +1,11 @@
| Value | Source | + @endif @foreach($pirep->fields as $field)
|---|