[5.x] Allow custom render callback for overridden exceptions

[FrittenKeeZ]

Upgrading from Statamic 4 and Laravel 10 to Statamic 5 and Laravel 11 I wanted to migrate this custom exception handler:

public function render($request, \Throwable $e)
{
    if ($e instanceof NotFoundHttpException) {
        $eventsMount = Collection::findByHandle('events')->mount();
        if (Str::before($request->path(), '/') === $eventsMount->slug()) {
            return response()->redirectTo($eventsMount->url());
        }
    }

    return parent::render($request, $e);
}

Following Laravel's documentation I couldn't understand why it didn't work, until I dived into the exception and saw it implements a render method which takes over.

This PR introduces a static callback you can use to implement your own handling:

NotFoundHttpException::$renderCallback = function (Request $request) {
    $eventsMount = Collection::findByHandle('events')->mount();
    if (Str::before($request->path(), '/') === $eventsMount->slug()) {
        return response()->redirectTo($eventsMount->url());
    }
};

[jasonvarga]

Thanks! I made a couple of tweaks.

• Changed to a renderUsing method which matches existing Laravel/Statamic conventions.
• The callback is moved to the start of render, so you can technically override the whole thing. You'll need to handle API situations though.
• The closure's $this will be the exception.
• If you don't return anything, the default behavior continues. (Yours was doing this already though).

NotFoundHttpException::renderUsing(function (Request $request) {
    //
});

Yours would look something like this:

NotFoundHttpException::renderUsing(function (Request $request) {
    if (Statamic::isCpRoute() || Statamic::isApiRoute() || $this->getStatusCode() !== 404) {
        return;
    }

    $eventsMount = Collection::findByHandle('events')->mount();
    if (Str::before($request->path(), '/') === $eventsMount->slug()) {
        return response()->redirectTo($eventsMount->url());
    }
};