AppserverIo\Routlt\ControllerServlet::service()

Complexity

Conditions	15
Paths	714

Size

Total Lines	114
Code Lines	43

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	48
CRAP Score	15.0144

Importance

Changes	2
Bugs	0	Features	0

<?php
    public function service(ServletRequestInterface $servletRequest, ServletResponseInterface $servletResponse)
    {

        try {
            // pre-initialize response
            $servletResponse->addHeader(HttpProtocol::HEADER_X_POWERED_BY, get_class($this));

It seems like you code against a concrete implementation and not the interface AppserverIo\Psr\Servlet\ServletResponseInterface as the method addHeader() does only exist in the following implementations of said interface: AppserverIo\Psr\Servlet\Http\HttpServletResponseWrapper.

            // load the path info from the servlet request
            $pathInfo = $servletRequest->getPathInfo();

            // if the requested action has been found in the path info
            if ($pathInfo == null) {
                $pathInfo = $this->getDefaultRoute();
            }

            // prepare the path of the requested action
            $requestedAction = $pathInfo;

            // load the routes
            $routes = $this->getRoutes();

            // load the DI provider
            $provider = $this->getProvider();

            // load the action mappings for the actual servlet request
            $actionMappings = $this->getActionMappingsForServletRequest($servletRequest);

            // initialize the parameter map with the values from the request
            if ($servletRequest->getParameterMap()) {
                $parameterMap = $servletRequest->getParameterMap();
            } else {
                $parameterMap = array();
            }

            // iterate over the action mappings and try to find a mapping
            foreach ($actionMappings as $actionMapping) {
                // try to match actual request by the tokenizer
                if ($actionMapping->match($requestedAction)) {
                    // initialize the request attributes with the values from the action mapping
                    $servletRequest->setParameterMap(

It seems like you code against a concrete implementation and not the interface AppserverIo\Psr\Servlet\ServletRequestInterface as the method setParameterMap() does only exist in the following implementations of said interface: AppserverIo\Psr\Servlet\Http\HttpServletRequestWrapper.

                        array_merge($parameterMap, $actionMapping->getRequestParameters())
                    );

                    // resolve the action with the found mapping
                    $action = $routes[$actionMapping->getControllerName()];

                    // query whether or not the action has a descriptor
                    if ($action instanceof DescriptorAware) {
                        $provider->injectDependencies($action->getDescriptor(), $action);
                    }

                    // set the method that has to be invoked in the action context
                    $action->setAttribute(ContextKeys::METHOD_NAME, $actionMapping->getMethodName());

                    // pre-dispatch the action
                    $action->preDispatch($servletRequest, $servletResponse);

                    // if the action has been dispatched, we're done
                    if ($servletRequest->isDispatched()) {
                        return;
                    }

                    // initialize the result with the default value
                    $result = ActionInterface::INPUT;

                    // if not dispatch the action
                    if ($newResult = $action->perform($servletRequest, $servletResponse)) {
                        $result = $newResult;
                    }

                    // process the result if available
                    if (($instance = $action->findResult($result)) instanceof ResultInterface) {
                        // query whether or not the result has a descriptor
                        if ($instance instanceof DescriptorAware && $descriptor = $instance->getDescriptor()) {
                            $provider->injectDependencies($descriptor, $instance);
                        }

                        // query whether or not the result is action aware
                        if ($instance instanceof ActionAware) {
                            $instance->setAction($action);
                        }

                        // process the result
                        $instance->process($servletRequest, $servletResponse);
                    }

                    // post-dispatch the action instance
                    $action->postDispatch($servletRequest, $servletResponse);

                    // stop processing
                    return;
                }
            }

            // We did not find anything for this method/URI connection. We have to evaluate if there simply
            // is a method restriction. This replicates a lot of the checks we did before but omits extra
            // iterations in a positive dispatch event, 4xx's should be the exception and can handle that
            // penalty therefore
            if ($this->checkGeneralActionAvailability($pathInfo)) {
                // nothing found? Method must not be allowed then
                throw new DispatchException(sprintf('Method %s not allowed', $servletRequest->getMethod()), 405);

It seems like you code against a concrete implementation and not the interface AppserverIo\Psr\Servlet\ServletRequestInterface as the method getMethod() does only exist in the following implementations of said interface: AppserverIo\Psr\Servlet\Http\HttpServletRequestWrapper.

            }

            // throw an action, because we can't find an action mapping
            throw new DispatchException(sprintf('Can\'t find action to dispatch path info %s', $pathInfo), 404);

Blank line found at end of control structure

        } catch (DispatchException $de) {
            // results in a 4xx error
            throw new ServletException($de->__toString(), $de->getCode());
        } catch (\Exception $e) {
            // results in a 500 error page
            throw new ServletException($e->__toString(), 500);
        }
    }
}