Core\Controller\Controller

Complexity

Total Complexity

10

Size/Duplication

Total Lines	105
Duplicated Lines	0 %

Importance

Changes	5
Bugs	0	Features	2

9 Methods

Rating	Name	Duplication	Size	Complexity
A	forbidden()	0	4	1
A	accessGlobal()	0	5	1
A	notFound()	0	4	1
A	tableName()	0	4	1
A	formatDate()	0	8	1
A	__construct()	0	6	1
A	render()	0	7	1
A	antiXss()	0	3	1
A	previousPage()	0	6	2

<?php

namespace Core\Controller;

use Core\Services\Globals\Globals;

class Controller
{
    protected string $viewPath;
    protected $template;

    public function __construct()
    {
        $this->GET = $this->accessGlobal('GET');

The property GET does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->POST = $this->accessGlobal('POST');

The property POST does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->SERVER = $this->accessGlobal('SERVER');

The property SERVER does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        $this->SESSION = $this->accessGlobal('SESSION');

The property SESSION does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

    }

    /**
     * Sends to the targeted View and sends the $variables
     * @param string $view ex: table.methode.attributs - 'post.show.1'
     * @param array $variables
     * @return void
     */
    protected function render(string $view, array $variables = [])
    {
        \ob_start();
        \extract($variables);
        require($this->viewPath . \str_replace('.', '/', $view) . '.php');
        $content = \ob_get_clean();
        require($this->viewPath . 'templates/' . $this->template . '.php');
    }

    /**
     * Get the name of the table associated with the controller
     * @return string ex : 'Post'
     */
    protected function tableName(): string
    {
        \preg_match('/[\w\-\_]+(?=Controller)/im', \get_called_class(), $tableName);
        return $tableName[0];
    }

    /**
     * Return to the previous page
     * @return void
     */
    protected function previousPage()
    {
        if (isset($this->GET['return'])) {
            return \header('Location: index.php?p=' . $this->GET['return']);

Are you sure the usage of header('Location: index.php?p=' . $this->GET['return']) is correct as it seems to always return null.

        }
        return \header('Location: index.php');

Are you sure the usage of header('Location: index.php') is correct as it seems to always return null.

    }

    /**
     * Format a date at the Local stardard
     * @param string $date
     * @return string
     */
    protected function formatDate(string $date): string
    {
        setlocale(LC_TIME, 'fr', 'fr_FR', 'fr_FR.ISO8859-1');

        $date = \strtotime($date);
        $day = strftime("%d ", $date);
        $day .= ucfirst(strftime("%B ", $date));
        return $day .= strftime("%G", $date);
    }

    /**
     * Management in the event of an error HTTP 403
     * @return void
     */
    protected function forbidden()
    {
        header("HTTP/1.0 403 Forbidden");
        return \header('Location: index.php?p=user.login');

Are you sure the usage of header('Location: index.php?p=user.login') is correct as it seems to always return null.

    }

    /**
     * Management in the event of an error 404
     * @return void
     */
    protected function notFound()
    {
        header("HTTP/1.0 404 Not Found");
        return \header('Location: index.php');

Are you sure the usage of header('Location: index.php') is correct as it seems to always return null.

    }

    /**
     * Add an XSS filter on the displayed datas
     * @param string $input
     * @return string
     */
    protected function antiXss(string $input): string
    {
        return \nl2br(\htmlspecialchars($input), ENT_QUOTES);

Core\Controller\ENT_QUOTES of type integer is incompatible with the type boolean expected by parameter $use_xhtml of nl2br().

    }

    /**
     * Secure get access to superglobals
     * @param string $global name of the superglobal targeted
     * @return mixed
     */
    protected function accessGlobal(string $global)
    {
        $globals = new Globals;
        $global = 'get' . $global;
        return $globals->$global();
    }
}