joesama / pintu

src/Routings/Manager.php

<?php

namespace Joesama\Pintu\Routings;

use Illuminate\Support\Arr;
use Illuminate\Routing\Router;
use Joesama\Pintu\Routings\Builder;
use Joesama\Pintu\Components\Manager as ComponentManager;

class Manager
{
    /**
     * Collection of API definition.
     *
     * @var ComponentManager
     */
    private $component;

    /**
     * Default controller namespaces.
     *
     * @var string
     */
    private $namespace;

    /**
     * Initialize routes manager.
     *
     * @param ComponentManager $component
     */
    public function __construct(ComponentManager $component)
    {
        $this->component = $component;
    }

    /**
     * Build the routing.
     *
     * @return void
     */
    public function routingRegistration(Router $router)
    {
        $builder = new Builder($router);

        $builder->componentRouting($this->component->getComponent(), $this->getComponentNameSpace());

        if ($this->component->getLanding()) {
            $builder->landingRouting($this->getComponentNameSpace());
        }

        $builder->apiRouting($this->component->getApi(), $this->getApiNameSpace());
    }

    /**
     * Set default namespace.
     *
     * @param string|null $namespace
     *
     * @return void
     */
    public function setNameSpace(string $namespace = null)
    {
        $this->namespace = $namespace;
    }

    /**
     * Get component controller namespace.
     *
     * @return string
     */
    public function getComponentNameSpace(): string
    {
        if ($this->namespace === null) {
            return Arr::get($this->component->getComponentNameSpace(), 'component');

The expression return Illuminate\Support\Arr::get($this->component->getComponentNameSpace(), 'component') could return the type null which is incompatible with the type-hinted return string. Consider adding an additional type-check to rule them out.

        }

        return $this->namespace;
    }

    /**
     * Get API controller namespace.
     *
     * @return string
     */
    public function getApiNameSpace(): string
    {
        if ($this->namespace === null) {
            return Arr::get($this->component->getComponentNameSpace(), 'api');

The expression return Illuminate\Support\Arr::get($this->component->getComponentNameSpace(), 'api') could return the type null which is incompatible with the type-hinted return string. Consider adding an additional type-check to rule them out.

        }

        return $this->namespace;
    }
}