Comodojo\Dispatcher\Dispatcher

Complexity

Total Complexity

17

Size/Duplication

Total Lines	223
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	16

Test Coverage

Coverage

68.82%

Importance

Changes	7
Bugs	1	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
B	__construct()	0	44	5
B	dispatch()	0	91	5
A	processConfigurationParameters()	0	10	4
A	emitServiceSpecializedEvents()	0	14	1
A	processDispatcherException()	0	15	1
B	shutdown()	0	24	1

<?php namespace Comodojo\Dispatcher;

use \Comodojo\Dispatcher\Components\AbstractModel;
use \Comodojo\Dispatcher\Components\DefaultConfiguration;
use \Comodojo\Dispatcher\Cache\ServerCache;
use \Comodojo\Dispatcher\Request\Model as Request;
use \Comodojo\Dispatcher\Router\Model as Router;
use \Comodojo\Dispatcher\Response\Model as Response;
use \Comodojo\Dispatcher\Extra\Model as Extra;
use \Comodojo\Dispatcher\Output\Processor;
use \Comodojo\Dispatcher\Events\DispatcherEvent;
use \Comodojo\Dispatcher\Events\ServiceEvent;
use \Comodojo\Foundation\Base\Configuration;
use \Comodojo\Foundation\Timing\TimingTrait;
use \Comodojo\Foundation\Events\Manager as EventsManager;
use \Comodojo\Foundation\Logging\Manager as LogManager;
use \Comodojo\SimpleCache\Manager as SimpleCacheManager;
use \Psr\Log\LoggerInterface;
use \Comodojo\Exception\DispatcherException;
use \Exception;

/**
 * @package     Comodojo Dispatcher
 * @author      Marco Giovinazzi <marco.giovinazzi@comodojo.org>
 * @author      Marco Castiello <marco.castiello@gmail.com>
 * @license     GPL-3.0+
 *
 * LICENSE:
 *
 * This program is free software: you can redistribute it and/or modify
 * it under the terms of the GNU Affero General Public License as
 * published by the Free Software Foundation, either version 3 of the
 * License, or (at your option) any later version.
 *
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
 * GNU Affero General Public License for more details.
 *
 * You should have received a copy of the GNU Affero General Public License
 * along with this program. If not, see <http://www.gnu.org/licenses/>.
 */

class Dispatcher extends AbstractModel {

    use TimingTrait;

    // protected $mode = self::PROTECTDATA;

46% of this comment could be valid code. Did you maybe forget this after debugging?

    /**
     * The main dispatcher constructor.
     *
     * @property Configuration $configuration
     * @property LoggerInterface $logger
     * @property EventsManager $events
     * @property Cache $cache
     * @property Extra $extra
     * @property Request $request
     * @property Router $router
     * @property Response $response
     */
    public function __construct(
        array $configuration = [],
        EventsManager $events = null,
        SimpleCacheManager $cache = null,
        LoggerInterface $logger = null
    ) {

        // starting output buffer
        ob_start();

        // fix current timestamp
        $this->setTiming();

        // init core components
        // create new configuration object and merge configuration
        $configuration_object = new Configuration( DefaultConfiguration::get() );
        $configuration_object->merge($configuration);
        $logger = is_null($logger) ? LogManager::createFromConfiguration($configuration_object)->getLogger() : $logger;

        parent::__construct($configuration_object, $logger);

        $this->logger->debug("--------------------------------------------------------");

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->logger->debug("Dispatcher run-cycle starts at ".$this->getTime()->format('c'));

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        try {
            
            $this->setRaw('events', is_null($events) ? EventsManager::create($this->logger) : $events);

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

            $this->setRaw('cache', is_null($cache) ? SimpleCacheManager::createFromConfiguration($this->configuration, $this->logger) : $cache);

The property configuration does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

            // init models
            $this->setRaw('extra', new Extra($this->logger));

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

The call to Model::__construct() has too many arguments starting with $this->logger.

            $this->setRaw('request', new Request($this->configuration, $this->logger));

The property configuration does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

            $this->setRaw('router', new Router($this->configuration, $this->logger, $this->cache, $this->extra));

The property cache does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

The property extra does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

The property configuration does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

            $this->setRaw('response', new Response($this->configuration, $this->logger));

The property configuration does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        } catch (Exception $e) {
            $this->logger->critical($e->getMessage(),$e->getTrace());

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

            throw $e;
        }

        // we're ready!
        $this->logger->debug("Dispatcher ready");

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

    }

    public function dispatch() {

        $this->logger->debug("Starting to dispatch.");

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->logger->debug("Emitting global dispatcher event.");

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->events->emit( new DispatcherEvent($this) );

The property events does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        if ( $this->configuration->get('enabled') === false ) {

The property configuration does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

            $this->logger->debug("Dispatcher disabled, shutting down gracefully.");

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

            $status = $this->configuration->get('disabled-status');

The property configuration does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

            $content = $this->configuration->get('disabled-message');

The property configuration does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

            $this->response->status->set($status);

The property response does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

            $this->response->content->set($content);

The property response does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

            return $this->shutdown();

        }

        $cache = new ServerCache($this->cache);

The property cache does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->events->emit( $this->emitServiceSpecializedEvents('dispatcher.request') );

The property events does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->events->emit( $this->emitServiceSpecializedEvents('dispatcher.request.'.$this->request->method->get()) );

The property events does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

The property request does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->events->emit( $this->emitServiceSpecializedEvents('dispatcher.request.#') );

The property events does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        if ( $cache->read($this->request, $this->response) ) {

The property request does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

The property response does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

            return $this->shutdown();

        }

        $this->logger->debug("Starting router");

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        try {

            $this->route = $this->router->route($this->request);

The property route does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __set, maybe consider adding a @property annotation.

The property router does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

The property request does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        } catch (DispatcherException $de) {

            $this->logger->debug("Route error (".$de->getStatus()."), shutting down dispatcher");

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

            $this->processDispatcherException($de);

            return $this->shutdown();

        }

        $route_type = $this->route->getType();

The property route does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        $route_service = $this->route->getServiceName();

The property route does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->logger->debug("Route acquired, type $route_type directed to $route_service");

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->events->emit( $this->emitServiceSpecializedEvents('dispatcher.route') );

The property events does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->events->emit( $this->emitServiceSpecializedEvents('dispatcher.route.'.$route_type) );

The property events does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->events->emit( $this->emitServiceSpecializedEvents('dispatcher.route.'.$route_service) );

The property events does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->events->emit( $this->emitServiceSpecializedEvents('dispatcher.route.#') );

The property events does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        // translate route to service

        $this->logger->debug("Running $route_service service");

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        try {

            $this->router->compose($this->response);

The property router does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

The property response does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        } catch (DispatcherException $de) {

            $this->logger->debug("Service exception (".$de->getStatus()."), shutting down dispatcher");

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

            $this->processDispatcherException($de);

        }

        $this->processConfigurationParameters($this->route);

The property route does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        $cache->dump($this->request, $this->response, $this->route);

The property request does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

The property response does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

The property route does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        return $this->shutdown();

    }

    private function processConfigurationParameters($route) {

        $params = $route->getParameter('headers');

        if ( !empty($params) && is_array($params) ) {

            foreach($params as $name=>$value) $this->response->headers->set($name, $value);

The property response does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        }

    }

    private function emitServiceSpecializedEvents($name) {

        $this->logger->debug("Emitting $name service-event");

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        return new ServiceEvent(
            $name,
            $this->logger,

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

            $this->request,

The property request does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

            $this->router,

The property router does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

            $this->response,

The property response does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

            $this->extra

The property extra does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        );

    }

    private function processDispatcherException(DispatcherException $de) {

        $status = $de->getStatus();

        $message = $de->getMessage();

        $headers = $de->getHeaders();

        $this->response->status->set($status);

The property response does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->response->content->set($message);

The property response does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->response->headers->merge($headers);

The property response does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

    }

    private function shutdown() {

        $this->response->consolidate($this->request, $this->route);

The property response does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

The property request does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

The property route does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->events->emit( $this->emitServiceSpecializedEvents('dispatcher.response') );

The property events does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->events->emit( $this->emitServiceSpecializedEvents('dispatcher.response.'.$this->response->status->get()) );

The property events does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

The property response does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->events->emit( $this->emitServiceSpecializedEvents('dispatcher.response.#') );

The property events does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->logger->debug("Composing return value");

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        $return = Processor::parse($this->configuration, $this->logger, $this->request, $this->response);

The property configuration does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

The property request does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

The property response does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        $this->logger->debug("Dispatcher run-cycle ends");

The property logger does not exist on object<Comodojo\Dispatcher\Dispatcher>. Since you implemented __get, maybe consider adding a @property annotation.

        // This could cause WSOD with some PHP-FPM configurations
        // if ( function_exists('fastcgi_finish_request') ) fastcgi_finish_request();

57% of this comment could be valid code. Did you maybe forget this after debugging?

        // else ob_end_clean();

58% of this comment could be valid code. Did you maybe forget this after debugging?

        ob_end_clean();

        return $return;

    }

}