Saltwater\Root\Provider\Context

Complexity

Total Complexity

4

Size/Duplication

Total Lines	37
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	get()	0	16	3
A	newContext()	0	4	1

<?php

namespace Saltwater\Root\Provider;

use Saltwater\Server as S;
use Saltwater\Utils as U;
use Saltwater\Salt\Provider;
use Saltwater\Salt\Context as SwContext;
use Saltwater\Salt\Module;

class Context extends Provider
{
    /**
     * @param string         $name
     * @param SwContext|null $parent
     *
     * @return SwContext|null
     */
    public function get($name, $parent = null)
    {
        $module = S::$n->getContextModule($name);

        if (empty($module)) {
            return null;
        }

        $class = U::className($module::getNamespace(), 'context', $name);

        if (!class_exists($class)) {
            return null;
        }

        return $this->newContext($class, $parent, $module);

It seems like $module defined by \Saltwater\Server::$n->getContextModule($name) on line 21 can also be of type string; however, Saltwater\Root\Provider\Context::newContext() does only seem to accept object<Saltwater\Salt\Module>, maybe add an additional type check?

    }

    /**
     * @param string         $class
     * @param SwContext|null $parent
     * @param Module         $module
     *
     * @return SwContext
     */
    private function newContext($class, $parent, $module)
    {
        return new $class($parent, $module);
    }
}