Router – Invalid Presenter Exception

@d@rkWolf

application:
errorPresenter: Error

Výjimka se přehodí nejprve na ErrorPresenter, který záleží jak máš udělaný. Ten by měl optimálně výjimku zpracovat a pro běžné 404 nelogovat, ale poslat chybovou stránku. Pro 500 pak chybu logovat – může to být něco závažného.

Tak jak to máš udělané tak přijde 404, ale ErrorPresenter z ní udělá 500 tím že mapuje neexistující Error4xxPresenter – udělá tam špatnou cestu.

Řešil bych to v ErrorPresenteru nastavením natvrdo cesty k existujícímu Error4xxPresenteru pro všechny moduly stejný.

@d@rkWolf

Máš dva moduly, Admin se pozná tím, že začíná url za doménou admin, takže nejrychlejší řešení by mohlo být jednoduše identifikovat modul admin podle tohoto řetězce. Zkus na to jít nějak takhle:

final class ErrorPresenter implements Nette\Application\IPresenter
{
    use Nette\SmartObject;

    public function __construct(private ILogger $logger, private Nette\Http\Request $httpRequest)
    {}


    public function run(Nette\Application\Request $request): Nette\Application\Response
    {
        $exception = $request->getParameter('exception');

        if ($exception instanceof Nette\Application\BadRequestException) {
            $urlScript = $this->httpRequest->getUrl();
            $subpath = substr($urlScript->path, strlen($urlScript->basePath));  // tady máš url
            $module = str_starts_with($subpath, 'admin/') ? 'Admin' : 'Front';
            $sep = ':';
            return new Responses\ForwardResponse($request->setPresenterName($module . $sep . 'Error4xx'));
        }

        $this->logger->log($exception, ILogger::EXCEPTION);
        return new Responses\CallbackResponse(function (Http\IRequest $httpRequest, Http\IResponse $httpResponse): void {
            if (preg_match('#^text/html(?:;|$)#', (string) $httpResponse->getHeader('Content-Type'))) {
                require __DIR__ . '/../templates/Error/500.phtml';
            }
        });
    }
}

Budeš to muset ještě doladit, ale Tvoje trable by to mohlo snadno vyřešit.

Editoval m.brecher (6. 4. 2023 1:54)

d@rkWolf napsal(a):

jo, na to mělo být právě použité toto:

            // forward only to admin or front error4xx presenter
            if (!strstr($module, 'Admin')) {
                $module = ($sep === ':') ? 'Front' : 'Front:';
            }

Já jsem to nějak zkoušel a nějak mi z toho vylezlo toto, ale nejspíš mě tehdy nenapadly ani zdaleka všechny varianty toho, co mi tam může přijít… Asi mi vycházelo, že když to identifikuje modul Admin, bude v $module proměnné Admin pokud tam bude cokoliv jiného nebo nic, dá se tam Front.

Já jsem se trápil s ErrorPresenterem už docela dávno, a jestli si to ještě správně pamatuji, tak:

a) pokud BadRequestException vyhodí Presenter::error() tak informaci o modulu v Requestu máš,

b) pokud BadRequestException vyhodí už Router tak v Requestu informaci o aktuálním modulu nemáš, jsou to ty 404ky, kde Tracy hlásí „No route for HTTP request“.

Proto je lepší nespoléhat na modul v requestu a raději si v ErrorPresenteru znovu zanalyzovat url, což není nic těžkého.

Editoval m.brecher (17. 4. 2023 14:49)

@MarekBartoš

Jestli tě nezajímají routy, co nematchnuly vůbec, tak url vůbec analyzovat nemusíš :)
https://github.com/…resenter.php

Moc pěkný. Jestli jsem to správně pochopil tak:

a) si někde namapuješ skupiny presenterů (ideálně celé moduly) na errorPresentery (::addErrorPresenter())

b) výjimku přehodíš na DefaultErrorPresenter

c) DefaultErrorPresenter matchne request a forwardne výjimku na modulový/defaultní errorPresenter

d) modulový errorPresenter si 403, 404, 500 vyřeší každý po svém, lze tedy i stránky 500 vykreslit do odpovídajícího layoutu

To se mě líbí, to budu používat :)