Ffcms\Core\Network\Request\MvcFeatures

Complexity

Total Complexity

23

Size/Duplication

Total Lines	140
Duplicated Lines	0 %

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	setAdd()	0	3	1
A	getAction()	0	3	1
C	setPathdata()	0	26	8
A	getAdd()	0	3	1
A	setController()	0	3	1
B	searchRedirect()	0	21	7
A	getController()	0	3	1
A	getID()	0	3	1
A	setAction()	0	3	1
A	setId()	0	3	1

<?php

namespace Ffcms\Core\Network\Request;

use Ffcms\Core\App;
use Ffcms\Core\Helper\Type\Any;
use Ffcms\Core\Helper\Type\Str;
use Symfony\Component\HttpFoundation\RedirectResponse;

/**
 * Trait MvcFeatures. Mvc features for Request networking
 * @package Ffcms\Core\Network\Request
 */
trait MvcFeatures
{
    protected $controller;
    protected $action;
    protected $argumentId;
    protected $argumentAdd;


    /**
     * Check if current url in redirect map
     * @return void
     */
    private function searchRedirect(): void
    {
        // calculated depend of language
        $pathway = $this->getPathInfo();

It seems like getPathInfo() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        /** @var array $routing */
        $routing = App::$Properties->getAll('Routing');

        if (!Any::isArray($routing) || !isset($routing['Redirect']) || !Any::isArray($routing['Redirect'])) {
            return;
        }

        // check if source uri is key in redirect target map
        if (array_key_exists($pathway, $routing['Redirect'])) {
            $target = $this->getSchemeAndHttpHost(); // . $this->getBasePath() . '/' . rtrim($routing['Redirect'][$pathway], '/');

It seems like getSchemeAndHttpHost() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            if ($this->getBasePath() !== null && !Str::likeEmpty($this->getBasePath())) {

It seems like getBasePath() must be provided by classes using this trait. How about adding it as abstract method to this trait?

                $target .= '/' . $this->getBasePath();
            }
            $target .= rtrim($routing['Redirect'][$pathway], '/');
            $redirect = new RedirectResponse($target);
            $redirect->send();
            exit();

Using exit here is not recommended.

        }
    }

    /**
     * Working with path array data
     * @param array|null $pathArray
     * @return void
     */
    private function setPathdata(?array $pathArray = null): void
    {
        if (!Any::isArray($pathArray) || count($pathArray) < 1) {
            return;
        }

        // check if array length is more then 4 basic elements and slice it recursive
        if (count($pathArray) > 4) {
            $this->setPathdata(array_slice($pathArray, 0, 4));

It seems like $pathArray can also be of type null; however, parameter $array of array_slice() does only seem to accept array, maybe add an additional type check?

            return;
        }

        // Switch path array as reverse without break point! Caution: drugs inside!
        switch (count($pathArray)) {
            case 4:
                $this->argumentAdd = $pathArray[3];
            // no break
            case 3:
                $this->argumentId = $pathArray[2];
            // no break
            case 2:
                $this->action = ucfirst(Str::lowerCase($pathArray[1]));
            // no break
            case 1:
                $this->controller = ucfirst(Str::lowerCase($pathArray[0]));
                break;
        }
    }

    /**
     * Get current controller name
     * @return string
     */
    public function getController(): ?string
    {
        return $this->controller;
    }

    /**
     * Get current controller action() name
     * @return string
     */
    public function getAction(): ?string
    {
        return $this->action;
    }

    /**
     * Get current $id argument for controller action
     * @return string|null
     */
    public function getID(): ?string
    {
        return urldecode($this->argumentId);
    }

    /**
     * Set current controller name
     * @param string $name
     */
    public function setController($name): void
    {
        $this->controller = $name;
    }

    /**
     * Set current action value
     * @param string $name
     */
    public function setAction($name): void
    {
        $this->action = $name;
    }

    /**
     * Set current id argument value
     * @param mixed $name
     */
    public function setId($name): void
    {
        $this->argumentId = $name;
    }

    /**
     * Set current add argument value
     * @param mixed $name
     */
    public function setAdd($name): void
    {
        $this->argumentAdd = $name;
    }

    /**
     * Get current $add argument for controller action
     * @return string|null
     */
    public function getAdd(): ?string
    {
        return urldecode($this->argumentAdd);
    }
}