core\src\FrontController

Complexity

Total Complexity

26

Size/Duplication

Total Lines	170
Duplicated Lines	3.53 %

Coupling/Cohesion

Components	1
Dependencies	8

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
C	display()	0	39	8
B	main()	3	23	6
A	page()	0	4	1
A	category()	0	3	1
A	shopCategory()	0	3	1
A	product()	3	9	3
B	module()	0	31	5

<?php namespace core\src;

use CI;
use CMSFactory\Events;
use core\models\Route;
use core\src\Controller\CategoryController;
use core\src\Controller\PageController;
use core\src\Exception\PageNotFoundException;

class FrontController
{

Opening brace of a class must be on the same line as the definition

    /**
     * @var CI
     */
    private $ci;

    /**
     * @var CoreModel
     */
    private $frontModel;

    /**
     * @var Router
     */
    private $router;

    /**
     * FrontController constructor.
     * @param CI $ci
     * @param CoreModel $frontModel
     * @param Router $router
     */
    public function __construct(CI $ci, CoreModel $frontModel, Router $router) {
        $this->ci = $ci;
        $this->frontModel = $frontModel;
        $this->router = $router;
    }

    /**
     * @param string $url
     * @throws PageNotFoundException

Comment missing or not on the next line for @throws tag in function comment

     */
    public function display($url) {

        $route = $this->router->findRoute($url);

        if (!$route) {
            throw new PageNotFoundException();
        }

        $this->ci->core->core_data['data_type'] = $route->getType();
        $id = $route->getEntityId();
        $this->ci->core->core_data['id'] = $id;

        CoreFactory::getConfiguration()->setCurrentEntityType($route->getType());
        CoreFactory::getConfiguration()->setCurrentEntityId($id);

        switch ($route->getType()) {
            case Route::TYPE_MAIN:
                $this->main();
                break;

Case breaking statements must be followed by a single blank line

            case Route::TYPE_PRODUCT:
                $this->product($id);
                break;

Case breaking statements must be followed by a single blank line

            case Route::TYPE_SHOP_CATEGORY:
                $this->shopCategory($id);
                break;

Case breaking statements must be followed by a single blank line

            case Route::TYPE_CATEGORY:
                $this->category($id);
                break;

Case breaking statements must be followed by a single blank line

            case Route::TYPE_PAGE:
                $this->page($id);
                break;

Case breaking statements must be followed by a single blank line

            case Route::TYPE_MODULE:
                $this->module($route->getUrl());
                break;

Case breaking statements must be followed by a single blank line

            default:
                throw new PageNotFoundException();

        }
    }

    private function main() {

        if ($this->ci->input->get() || strstr($this->ci->input->server('REQUEST_URI'), '?')) {
            $this->ci->template->registerCanonical(site_url());
        }

        Events::create()->registerEvent(NULL, 'Core:_mainPage')->runFactory();

        $settings = CoreFactory::getConfiguration()->getSettings();

        switch ($settings['main_type']) {
            case 'page':
                Events::create()->registerEvent(NULL, 'Core:_mainPage')->runFactory();
                $this->page($settings['main_page_id']);
                break;

Case breaking statements must be followed by a single blank line

            case 'category':
                $this->category($settings['main_page_cat']);
                break;

Case breaking statements must be followed by a single blank line

            case 'module':
                $this->module($settings['main_page_module']);
                break;
        }
    }

    /**
     * @param int $id
     */
    private function page($id) {

        (new PageController($this->ci, $this->frontModel))->index($id);
    }

    /**
     * @param int $id
     */
    private function category($id) {
        (new CategoryController($this->ci, $this->frontModel))->index($id);
    }

    /**
     * @param int $id
     */
    private function shopCategory($id) {
        $this->module('shop/category/index/' . $id);
    }

    /**
     * @param int $id
     */
    private function product($id) {

        if ($this->ci->input->get() || strstr($this->ci->input->server('REQUEST_URI'), '?')) {
            $this->ci->template->registerCanonical(site_url($this->ci->uri->uri_string()));
        }

        $this->module('shop/product/index/' . $id);

    }

    /**
     * @param string $url
     * @throws PageNotFoundException

Comment missing or not on the next line for @throws tag in function comment

     */
    private function module($url) {

        $moduleSegments = explode('/', $url);

        $moduleName = array_shift($moduleSegments);
        $moduleMethod = array_shift($moduleSegments);
        if ($moduleMethod == FALSE) {
            $moduleMethod = 'index';
        }

        $file = getModulePath($moduleName) . $moduleMethod . EXT;
        $this->core_data['module'] = $moduleName;

The property core_data does not exist. Did you maybe forget to declare it?

        if (file_exists($file)) {
            $moduleAction = array_shift($moduleSegments) ?: 'index';
            $moduleFullPath = $moduleName . '/' . $moduleMethod;
            $module = $this->ci->load->module($moduleName . '/' . $moduleMethod);

        } else {
            $moduleAction = $moduleMethod;
            $moduleFullPath = $moduleName . '/' . $moduleName;
            $module = $this->ci->load->module($moduleName);
        }

        $args = $moduleSegments;
        if (method_exists($module, $moduleAction)) {
            echo \modules::run($moduleFullPath . '/' . $moduleAction, $args);
        } else {
            throw new PageNotFoundException();
        }
    }

}