Temat: Kohana i obsługa błędu 404

Patrzę na powyższe przykłady i się zastanawiam: nie uważacie wyświetlania błędu 404 przy każdym wyjątku za niestosowne? Wyjątki w Kohanie mogą przecież pochodzić z różnych klas, nie tylko ReflectionException (gdzie słusznie można by wyświetlić 404), ale np. ErrorException, gdzie userowi wyświetliłbym np. 500, zamiast 404. Czemu? Żeby user miał świadomość, że strona jednak istnieje, ale chwilowo coś nie działa oraz przede wszystkim, żeby nie zwiększać prawdopodobieństwa, że Googlebot zacznie rzadziej odwiedzać stronę i ją szybciej wyindeksuje z uwagi na status HTTP.

Skłaniałbym się raczej ku takiemu rozwiązaniu:

$request = Request::instance($_SERVER['PATH_INFO']);



try

{

    // Attempt to execute the response

    $request->execute();

}

catch (Exception $e)

{

    if (Kohana::$environment === Kohana::DEVELOPMENT)

    {

        // Just re-throw the exception

        throw $e;

    }



    // Set the error code

    if ($e instanceof ReflectionException OR ! $e->getCode())

    {

        $error_code = 404;

    }

    elseif ($e instanceof ErrorException)

    {

        $error_code = 500;

    }

    elseif ($e->getCode())

    {

        $error_code = $e->getCode();

    }



    // Log the error

    Kohana::$log->add(Kohana::ERROR, Kohana::exception_text($e));



    // Check if the error template exists

    if (Kohana::find_file('views/errors/', $error_code))

    {

        $content = View::factory('errors/'.$error_code);

    }

    else

    {

        $content = View::factory('errors/default')

                    ->set('message', $e->getMessage());

    }



    // Create an error response

    $request->status = $error_code;

    $request->response = View::factory('errors/template')

                ->set('content', $content);

}

Paweł Dymek edytował(a) ten post dnia 28.07.10 o godzinie 16:31