ByTIC\GoogleAnalytics\Tracking\Renderer\Renderer

Complexity

Total Complexity

2

Size/Duplication

Total Lines	29
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

0%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	render()	0	5	1
A	newScript()	0	8	1

<?php

namespace ByTIC\GoogleAnalytics\Tracking\Renderer;

use ByTIC\GoogleAnalytics\Tracking\GoogleAnalytics;
use ByTIC\GoogleAnalytics\Tracking\Renderer\Script\AnalyticsJs;
use ByTIC\GoogleAnalytics\Tracking\Renderer\Script\AbstractScript;
use ByTIC\GoogleAnalytics\Tracking\Traits\HasRenderer;

/**
 * Class Renderer
 * @package ByTIC\GoogleAnalytics\Tracking\Renderer
 */
class Renderer
{
    const DEFAULT_RENDERER = 'AnalyticsJs';

    /**
     * @param GoogleAnalytics|HasRenderer $analytics
     * @param string $name
     * @return string
     */
    public static function render($analytics, $name = SELF::DEFAULT_RENDERER)
    {
        $script = self::newScript($analytics, $name);

It seems like $analytics defined by parameter $analytics on line 23 can also be of type object<ByTIC\GoogleAnalytics\Tracking\Traits\HasRenderer>; however, ByTIC\GoogleAnalytics\Tracking\Renderer\Renderer::newScript() does only seem to accept object<ByTIC\GoogleAnalytics\Tracking\GoogleAnalytics>, maybe add an additional type check?

        return $script->render();
    }

    /**
     * @param string $name
     * @param GoogleAnalytics $analytics
     * @return AbstractScript
     */
    protected static function newScript($analytics, $name)
    {
        $class = '\ByTIC\GoogleAnalytics\Tracking\Renderer\Script\\' . $name;
        /** @var AbstractScript $script */
        $script = new $class();
        $script->setGoogleAnalytics($analytics);
        return $script;
    }
}