Graze\CiffRenderer\Field\Builder\StaticGraphicBuilder

Complexity

Total Complexity

4

Size/Duplication

Total Lines	40
Duplicated Lines	100 %

Coupling/Cohesion

Components	0
Dependencies	4

Test Coverage

Coverage

85.71%

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	buildRenderer()	13	13	2
A	instantiateParser()	4	4	1
A	instantiateRenderer()	4	4	1

<?php

namespace Graze\CiffRenderer\Field\Builder;

use Graze\CiffRenderer\Field\Parser\StaticGraphicParser;
use Graze\CiffRenderer\Field\Renderer\StaticGraphicRenderer;
use Graze\CiffRenderer\Field\Parser\ParserInterface;
use Graze\CiffRenderer\Field\Renderer\RendererInterface;
use Graze\CiffRenderer\Exception\RuntimeException;

class StaticGraphicBuilder extends AbstractBuilder

This class seems to be duplicated in your project.

{
    /**
     * @return StaticGraphicParser
     */
    protected function instantiateParser()
    {
        return new StaticGraphicParser($xmlField);

The variable $xmlField does not exist. Did you forget to declare it?

The call to StaticGraphicParser::__construct() has too many arguments starting with $xmlField.

    }

    /**
     * @param ParserInterface $parser
     *
     * @return StaticGraphicRenderer
     */
    protected function instantiateRenderer(ParserInterface $parser)
    {
        return new StaticGraphicRenderer();
    }

    /**
     * @param RendererInterface $renderer
     * @param ParserInterface $parser
     *
     * @return StaticGraphicRenderer
     */
    public function buildRenderer(RendererInterface $renderer, ParserInterface $parser)
    {
        $renderer = parent::buildRenderer($renderer, $parser);

        $graphicResolver = $this->getGraphicResolver();
        if (!$graphicResolver) {
            throw new RuntimeException('Graphic resolver not set');
        }

        $renderer->setGraphicResolver($graphicResolver);

        return $renderer;
    }
}