Почему так странно работает findOrFail()?

Question

[Роми]

Внезапно я вспомнил про findOrFail() - и она действительно бросает исключение, если не находит модель :) ОДНАКО смотрите что в выводе:

{
    "message": "No query results for model [App\\Models\\User] 9999999",
    "exception": "Symfony\\Component\\HttpKernel\\Exception\\NotFoundHttpException",

и все бы тут ничего, но дело в том, что "message" оно берет из

Illuminate\Database\Eloquent\ModelNotFoundException

тогда как "exception" у нее, как видите, другой...

И самая главная проблема, что если я хочу изменить вывод в Handler через renderable(), то мне приходится указывать в качестве аргумента NotFoundHttpException у которого нет нужного мне метода getModel() (для отображения в message)

Это что, какой-то косяк разработчиков, или что?

Навскидку в голову ничего не пришло, как это можно поправить "мягким" способом, может чего посоветуете?

upd.: там вообще магия какая-то :) бо в самой функции вызывается именно

throw (new ModelNotFoundException)->setModel(
            get_class($this->model), $id
        );

откуда там взялось NotFoundHttpException - тот еще вопрос :D

Answer 1

[jazzus]

$this->renderable(function (NotFoundHttpException $e, $request) {
    if ($request->is('users/*')) {
        return response()->json([
            'message' => 'Юзер не найден'
        ], 404);
    }
});

Comments

[Роми]

бро, спасибо, это интересное решение, но к сожалению - не универсальное, как если бы можно было использовать getModel() - т.е. это просто годный костыль :) но таки годный ))

[jazzus]

Роми, да нет костыль как раз getModel, а универсальное решение это отдать шаблон 404 с сообщением страница не найдена, как делают все нормальные люди

[Роми]

jazzus, так оно и так, в общем-то, отдает же

"message": "No query results for model [App\\Models\\User] 9999999",

но соль в том, чтобы иметь возможность отдавать имя модели и при APP_DEBUG=false и в каком-то своем варианте. типа:

"message": "User 9999999 не найден"

или вроде того

[jazzus]

Роми, что юзер будет делать с названием модели и айдишником? Это техническая инфа для разработчика. Юзерфрендли в этом ноль. И запросы могут не иметь модели. А некоторые могут иметь несколько моделей. Короче бред на бреде.

[Роми]

jazzus, ну почему сразу бред-то? Просто хочется больше гибкости из коробки, это нормально.

Бред это как раз поведение Ларавеля в данном случае, когда в методе бросается один эксепшн, а на выходе почему-то совсем другой, вот это - бред, да. Это по ходу надо issue писать им на гитхаб, но мне вломы)))

[jazzus]

Роми, все в Ларавел логично и правильно. Сначала ModelNotFoundException затем NotFoundHttpException, который и рендерится при 404. Ты походу вертикально читаешь. Я уже писал, что запрос может не иметь модели. Так что иссую твою отправят в корзину.

[Роми]

jazzus, ну, я понял твою точку зрения, но как по мне, если в методе бросается одно исключение, а на выводе получается другое - это как-то неправильно.

[jazzus]

Роми, неправильно посылать юзеру название модели вместо нормального сообщения. А с findOrFail все логично - после ошибки с моделью вызывается 404 http, как и обещано в доке.

Answer 2

[Роми]

В общем, благодаря дискуссии в каментах с коллегой jazzus я пришел к выводу, что приемлемого решения этого вопроса на данный момент не существует.

Можно конечно отлавливать исключение ModelNotFoundException в месте вызова при помощи try-catch:

try {
            $user = User::findOrFail($id);
        } catch (ModelNotFoundException $e) {
            return response()->json(['model' => $e->getModel()], 404);
        }

но это такое себе...

Comments

[jazzus]

Действительно, проверять findOrFail еще надо додуматься. Решение всегда было. см доку и мой ответ. Как его использовать решает программист. Например запихиваешь в массив ключи/name урлов и сообщения, форычишь http ексепшен и получаешь нормальные сообщения, даже с переводами. id в реквесте также присутствует.

Answer 3

[iljaGolubev]

1. к вопросу "откуда там взялось NotFoundHttpException..."
src/Illuminate/Foundation/Exceptions/Handler.php
2. не хочешь стандартных исключенией - не юзай findOrFail
2.1 не можешь без findOrFail - измени поведение app/Exceptions/Handler.php