RicardoFiorani\Renderer\DefaultRenderer

Complexity

Total Complexity

1

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	1

1 Method

Rating	Name	Duplication	Size	Complexity
A	renderVideoEmbedCode()	0	9	1

<?php
/**
 * @author Ricardo Fiorani
 */

namespace RicardoFiorani\Renderer;

class DefaultRenderer implements EmbedRendererInterface
{
    /**
     * A friend once told me that html inside strings
     * isn't a good thing, but god knows I trying
     * to make it not dependant of some html element generator
     * and I'm too lazy/unnecessary to make one of these on my own.
     * Edit: The best I could do is do an sprintf, so...
     *
     * @param string $embedUrl
     * @param int $width
     * @param int $height
     *
     * @return string
     */
    public function renderVideoEmbedCode($embedUrl, $width, $height)
    {
        return sprintf(
            '<iframe width="%s" height="%s" src="%s" frameborder="0" webkitAllowFullScreen mozallowfullscreen allowFullScreen></iframe>',

This line exceeds maximum limit of 120 characters; contains 137 characters

            $width,
            $height,
            addslashes($embedUrl)
        );
    }
}

As per coding style, files should not end with a newline character.