ErrorController::errorAction()

Complexity

Conditions	8
Paths	17

Size

Total Lines	41
Code Lines	23

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	12
CRAP Score	12.096

Importance

Changes	2
Bugs	0	Features	0

<?php
    public function errorAction(): void
    {
        $errors = $this->_getParam('error_handler');

        if (!$errors || !$errors instanceof ArrayObject) {
            $this->view->message = 'You have reached the error page';

            return;
        }

        switch ($errors->type) {

The property type does not seem to exist on ArrayObject.

            case Zend_Controller_Plugin_ErrorHandler::EXCEPTION_NO_ROUTE:
            case Zend_Controller_Plugin_ErrorHandler::EXCEPTION_NO_CONTROLLER:
            case Zend_Controller_Plugin_ErrorHandler::EXCEPTION_NO_ACTION:
                // 404 error -- controller or action not found
                $this->getResponse()->setHttpResponseCode(404);
                $priority = Zend_Log::NOTICE;
                $this->view->message = 'Page not found';

                break;
            default:
                // application error
                $this->getResponse()->setHttpResponseCode(500);
                $priority = Zend_Log::CRIT;
                $this->view->message = 'Application error';

                break;
        }

        // Log exception, if logger available
        if ($log = $this->getLog()) {
            $log->log($this->view->message, $priority, $errors->exception);

Accessing message on the interface Zend_View_Interface suggest that you code against a concrete implementation. How about adding an instanceof check?

The property exception does not seem to exist on ArrayObject.

            $log->log('Request Parameters', $priority, $errors->request->getParams());

The property request does not seem to exist on ArrayObject.

        }

        // conditionally display exceptions
        if ($this->getInvokeArg('displayExceptions') == true) {
            $this->view->exception = $errors->exception;
        }

        $this->view->request = $errors->request;
    }