ControllerServlet::initResultInstance() - Code Metrics - Inspection of "Refactoring for usage with new DI implementation" - appserver-io/routlt - Measure and Improve Code Quality continuously with Scrutinizer

Completed
Pull Request — master (#48)

by Tim
created 2018-04-11 20:43 UTC
ControllerServlet::initResultInstance() B

↳ Parent: ControllerServlet
Complexity

Conditions	4
Paths	8
Size

Total Lines	27
Code Lines	10
Duplication

Lines	0
Ratio	0 %
Code Coverage

Tests	12
CRAP Score	4
Importance

Changes
Metric	Value
dl	0
loc	27
ccs	12
cts	12
cp	1
rs	8.5806
c	0
b	0
f	0
cc	4
eloc	10
nc	8
nop	2
crap	4
<?php

/**
 * AppserverIo\Routlt\ControllerServlet
 *
 * NOTICE OF LICENSE
 *
 * This source file is subject to the Open Software License (OSL 3.0)
 * that is available through the world-wide-web at this URL:
 * http://opensource.org/licenses/osl-3.0.php
 *
 * PHP version 5
 *
 * @author    Tim Wagner <[email protected]>
 * @copyright 2015 TechDivision GmbH <[email protected]>
 * @license   http://opensource.org/licenses/osl-3.0.php Open Software License (OSL 3.0)
 * @link      http://github.com/appserver-io/routlt
 * @link      http://www.appserver.io
 */

namespace AppserverIo\Routlt;

use AppserverIo\Http\HttpProtocol;
use AppserverIo\Properties\Properties;
use AppserverIo\Psr\Di\ProviderInterface;
use AppserverIo\Psr\Di\ObjectManagerInterface;
use AppserverIo\Psr\Servlet\Http\HttpServlet;
use AppserverIo\Psr\Servlet\ServletException;
use AppserverIo\Psr\Servlet\ServletConfigInterface;
use AppserverIo\Psr\Servlet\ServletRequestInterface;
use AppserverIo\Psr\Servlet\ServletResponseInterface;
use AppserverIo\Routlt\Util\ContextKeys;
use AppserverIo\Routlt\Util\ActionAware;
use AppserverIo\Routlt\Util\ServletContextAware;
use AppserverIo\Routlt\Results\ResultInterface;
use AppserverIo\Routlt\Description\PathDescriptorInterface;
use AppserverIo\Routlt\Description\ResultDescriptorInterface;
use AppserverIo\Routlt\Util\DescriptorAware;

/**
 * Abstract example implementation that provides some kind of basic MVC functionality
 * to handle requests by subclasses action methods.
 *
 * @author    Tim Wagner <[email protected]>
 * @copyright 2015 TechDivision GmbH <[email protected]>
 * @license   http://opensource.org/licenses/osl-3.0.php Open Software License (OSL 3.0)
 * @link      http://github.com/appserver-io/routlt
 * @link      http://www.appserver.io
 */
class ControllerServlet extends HttpServlet implements ControllerInterface
{

    /**
     * The key for the init parameter with the action namespace.
     *
     * @var string
     */
    const INIT_PARAMETER_ACTION_NAMESPACE = 'action.namespace';

    /**
     * The key for the init parameter with the path to the configuration file.
     *
     * @var string
     */
    const INIT_PARAMETER_ROUTLT_CONFIGURATION_FILE = 'routlt.configuration.file';

    /**
     * The default action if no valid action name was found in the path info.
     *
     * @var string
     */
    const DEFAULT_ROUTE = '/index';

    /**
     * The array with the initialized routes.
     *
     * @var array
     */
    protected $routes = array();

    /**
     * The array with the path descriptors.
     *
     * @var array
     */
    protected $paths = array();

    /**
     * The array with request method action -> route mappings.
     *
     * @var array
     */
    protected $actionMappings = array();

    /**
     * Initializes the servlet with the passed configuration.
     *
     * @param \AppserverIo\Psr\Servlet\ServletConfigInterface $config The configuration to initialize the servlet with
     *
     * @return void
     */
    public function init(ServletConfigInterface $config)
    {

        // call parent method
        parent::init($config);

        // load the values from the configuration file
        $this->initConfiguration();

        // initialize the routing
        $this->initRoutes();
    }

    /**
     * Returns the available routes.
     *
     * @return array The array with the available routes
     */
    public function getRoutes()
    {
        return $this->routes;
    }

    /**
     * Returns the array with request method action -> route mappings.
     *
     * @return array The request method action -> route mappings
     */
    public function getActionMappings()
    {
        return $this->actionMappings;
    }

    /**
     * Returns the naming directoy instance (the application).
     *
     * @return \AppserverIo\Psr\Naming\NamingDirectoryInterface The naming directory instance
     */
    public function getNamingDirectory()
    {
        return $this->getServletContext()->getApplication();

    }

    /**
     * Returns the object manager instance
     *
     * @return \AppserverIo\Psr\Di\ObjectManagerInterface The object manager instance
     */
    public function getObjectManager()
    {
        return $this->getNamingDirectory()->search(ObjectManagerInterface::IDENTIFIER);
    }

    /**
     * Returns the DI provider instance.
     *
     * @return \AppserverIo\Psr\Di\ProviderInterface The DI provider instance
     */
    public function getProvider()
    {
        return $this->getNamingDirectory()->search(ProviderInterface::IDENTIFIER);
    }

    /**
     * This method returns the default route we'll invoke if the path info doesn't contain one.
     *
     * @return string The default route
     */
    public function getDefaultRoute()
    {
        return ControllerServlet::DEFAULT_ROUTE;
    }

    /**
     * Returns the array with the path descriptors.
     *
     * @return array The array with the path descriptors
     */
    public function getPathDescriptors()
    {
        return $this->paths;
    }

    /**
     * Adds a path descriptor to the controller.
     *
     * @param \AppserverIo\Routlt\Description\PathDescriptorInterface $pathDescriptor The path descriptor to add
     *
     * @return void
     */
    public function addPathDescriptor(PathDescriptorInterface $pathDescriptor)
    {
        $this->paths[$pathDescriptor->getName()] = $pathDescriptor;
    }

    /**
     * Returns the path descriptor with the passed name.
     *
     * @param string $name The name of the path descriptor to return
     *
     * @return \AppserverIo\Routlt\Description\PathDescriptorInterface The path descriptor instance
     * @throws \Exception
     */
    public function getPathDescriptor($name)
    {

        // query whether or not the path descriptor exists
        if (isset($this->paths[$name])) {
            return $this->paths[$name];
        }

        // throw an exception if the requested path descriptor ist NOT available
        throw new \Exception(sprintf('Can\'t find path descriptor with name "%s"', $name));
    }

    /**
     * Loads the values found in the configuration file and merges
     * them with the servlet context initialization parameters.
     *
     * @return void
     */
    protected function initConfiguration()
    {

        // load the relative path to the Routlt configuration file
        $configurationFileName = $this->getInitParameter(ControllerServlet::INIT_PARAMETER_ROUTLT_CONFIGURATION_FILE);

        // load the path to the configuration file
        $configurationFile = $this->getServletConfig()->getWebappPath() . DIRECTORY_SEPARATOR . ltrim($configurationFileName, '/');

        // if the file is readable
        if (is_file($configurationFile) && is_readable($configurationFile)) {
            // load the  properties from the file
            $properties = new Properties();
            $properties->load($configurationFile);

            // append the properties to the servlet context
            foreach ($properties as $paramName => $paramValue) {
                $this->getServletContext()->addInitParameter($paramName, $paramValue);
            }
        }
    }

    /**
     * Initializes the available routes.
     *
     * @return void
     */
    protected function initRoutes()
    {

        // load the action namespace
        $actionNamespace = strtolower($this->getInitParameter(ControllerServlet::INIT_PARAMETER_ACTION_NAMESPACE));

        // register the beans located by annotations and the XML configuration
        foreach ($this->getObjectManager()->getObjectDescriptors() as $descriptor) {
            // check if we've found a servlet descriptor
            if ($descriptor instanceof PathDescriptorInterface) {
                // register the action's references
                $this->getServletContext()->registerReferences($descriptor);

                // initialize a new action instance
                $action = $this->initActionInstance($descriptor);

                // prepare the action's the result descriptors
                /** @var \AppserverIo\Routlt\Description\ResultDescriptorInterface $resultDescriptor */
                foreach ($descriptor->getResults() as $resultDescriptor) {
                    // register the result's references
                    $this->getServletContext()->registerReferences($resultDescriptor);

                    // initialize a new result instance and add it to the action
                    $action->addResult($this->initResultInstance($resultDescriptor, $action));
                }

                // prepare the route, e. g. /index/index
                $controllerName = str_replace($actionNamespace, '', $descriptor->getName());

                // initialize the action mappings
                foreach ($descriptor->getActions() as $actionDescriptors) {
                    // iterate over all request methods
                    /** @var \AppserverIo\Routlt\Description\ActionDescriptorInterface $actionDescriptor */
                    foreach ($actionDescriptors as $requestMethod => $actionDescriptor) {
                        // prepare the real action method name
                        $methodName = $actionDescriptor->getMethodName();
                        // prepare the action path -> concatenate route + action name
                        $actionPath = sprintf('%s%s', $controllerName, $actionDescriptor->getName());

                        // initialize the action mapping for the actual route
                        $actionMapping = new ActionMapping();
                        $actionMapping->setControllerName($controllerName);
                        $actionMapping->setMethodName($methodName);
                        $actionMapping->compile(
                            $actionPath,
                            $actionDescriptor->getRestrictions(),
                            $actionDescriptor->getDefaults()
                        );

                        // add the action path -> route mapping for the request method
                        $this->actionMappings[$requestMethod][$actionPath] = $actionMapping;

                        // add an alias for the route for the action's default method
                        if ($actionDescriptor->getMethodName() === $action->getDefaultMethod()) {
                            // initialize the action mapping for the default route
                            $actionMapping = new ActionMapping();
                            $actionMapping->setControllerName($controllerName);
                            $actionMapping->setMethodName($methodName);
                            $actionMapping->compile(
                                $controllerName,
                                $actionDescriptor->getRestrictions(),
                                $actionDescriptor->getDefaults()
                            );
                            // add the action mapping for the default route
                            $this->actionMappings[$requestMethod][$controllerName] = $actionMapping;
                        }
                    }
                }

                // add the initialized action
                $this->routes[$controllerName] = $action;
            }
        }
    }

    /**
     * Creates a new instance of the action from the passed path descriptor instance.
     *
     * @param \AppserverIo\Routlt\Description\PathDescriptorInterface $pathDescriptor The path descriptor to create the action from
     *
     * @return \AppserverIo\Routlt\ActionInterface The action instance
     */
    protected function initActionInstance(PathDescriptorInterface $pathDescriptor)
    {

        // create a new action instance
        $actionInstance = $this->getProvider()->get($pathDescriptor->getName());

        // if the action is servlet context aware
        if ($actionInstance instanceof ServletContextAware) {
            $actionInstance->setServletContext($this->getServletContext());
        }

        // if the action is descriptor aware
        if ($actionInstance instanceof DescriptorAware) {
            $actionInstance->setDescriptor($pathDescriptor);
        }

        // return the action instance
        return $actionInstance;
    }

    /**
     * Creates a new instance of the action result the passed descriptor.
     *
     * @param \AppserverIo\Routlt\Description\ResultDescriptorInterface $resultDescriptor The action result descriptor
     * @param \AppserverIo\Routlt\ActionInterface                       $action           The action instance the result is bound to
     *
     * @return \AppserverIo\Routlt\Results\ResultInterface The result instance
     */
    protected function initResultInstance(ResultDescriptorInterface $resultDescriptor, ActionInterface $action)
    {

        // create the result instance
        $resultInstance = $this->getProvider()->get($resultDescriptor->getType());

        // if the result is action aware
        if ($resultInstance instanceof ActionAware) {
            $resultInstance->setAction($action);
        }

        // if the result is descriptor aware
        if ($resultInstance instanceof DescriptorAware) {
            $resultInstance->setDescriptor($resultDescriptor);
        }

        // if the result is servlet context aware
        if ($resultInstance instanceof ServletContextAware) {
            $resultInstance->setServletContext($this->getServletContext());
        }

        // initialize the instance from the result descriptor
        $resultInstance->init($resultDescriptor);

        // return the result instance
        return $resultInstance;
    }

    /**
     * Checks whether or not an action is generally available for any request method.
     * Will return TRUE if so, FALSE otherwise.
     * This method replicates a lot of the checks generally necessary but omits the request method check.
     * Still best called in exception- or edge-cases
     *
     * @param string $pathInfo The action path which has been requested
     *
     * @return boolean
     */
    public function checkGeneralActionAvailability($pathInfo)
    {

        // iterate the request methods we have mappings for and check if we can find the requested action
        foreach ($this->getActionMappings() as $actionMapping) {
            $run = true;
            $requestedAction = $pathInfo;
            do {
                if (isset($actionMapping[$requestedAction])) {
                    return true;
                }
                // strip the last directory
                $requestedAction = dirname($requestedAction);

                // query whether we've to stop dispatching
                if ($requestedAction === '/' || $requestedAction === false) {
                    $run = false;
                }
            } while ($run === true);
        }

        // nothing found? Return false then
        return false;
    }

    /**
     * Returns the array with request method action -> route mappings
     * for the passed servlet request.
     *
     * @param \AppserverIo\Psr\Servlet\ServletRequestInterface $servletRequest The request instance
     *
     * @return array The request method action -> route mappings for the passed request method
     */
    public function getActionMappingsForServletRequest(ServletRequestInterface $servletRequest)
    {
        // load the servlet request method
        $requestMethod = $servletRequest->getMethod();
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}

        // load the action mappings
        $actionMappings = $this->getActionMappings();

        // query whether we've action mappings for the request method or not
        if (isset($actionMappings[$requestMethod])) {
            return $actionMappings[$requestMethod];
        }

        // nothing found? Method must not be allowed then
        throw new DispatchException(sprintf('Method %s not allowed', $requestMethod), 405);
    }

    /**
     * Delegates to HTTP method specific functions like doPost() for POST e.g.
     *
     * @param \AppserverIo\Psr\Servlet\ServletRequestInterface  $servletRequest  The request instance
     * @param \AppserverIo\Psr\Servlet\ServletResponseInterface $servletResponse The response sent back to the client
     *
     * @return void
     *
     * @throws \AppserverIo\Psr\Servlet\ServletException If no action has been found for the requested path
     */
    public function service(ServletRequestInterface $servletRequest, ServletResponseInterface $servletResponse)
    {

        try {
            // pre-initialize response
            $servletResponse->addHeader(HttpProtocol::HEADER_X_POWERED_BY, get_class($this));
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}

            // 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(
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
                        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) {
                            $provider->injectDependencies($instance->getDescriptor(), $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);
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
            }

            // 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);


        } 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);
        }
    }
}

Pull Request — master (#48)

ControllerServlet::initResultInstance() B

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Available Fixes

Available Fixes

Available Fixes

appserver-io / routlt

Pull Request — master (#48)

ControllerServlet::initResultInstance() B

Complexity

Size

Duplication

Code Coverage

Importance

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like
