ImageRenderer::renderOld() - Code Metrics - Inspection of "implemented image renderer and figcaption renderer" - jolicht/markdown-cms - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 98cc09...f90658 )

by Johannes

created 2017-04-05 04:46 UTC

ImageRenderer::renderOld() C

↳ Parent: ImageRenderer

Complexity

Conditions	11
Paths	144

Size

Total Lines	51
Code Lines	34

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
c	0
b	0
f	0
dl	0
loc	51
rs	5.3599
cc	11
eloc	34
nc	144
nop	2

How to fix Long Method Complexity

<?php
/**
 * Lichtenwallner  (https://lichtenwallner.at)
 *
 * @see https://github.com/jolicht/markdown-cms for the canonical source repository
 * @license https://github.com/jolicht/markdown-cms/blob/master/LICENSE MIT
 * @copyright Copyright (c) Johannes Lichtenwallner
 */
declare(strict_types = 1);
namespace Jolicht\MarkdownCms\Markdown\Renderer;

use League\CommonMark\ElementRendererInterface;
use League\CommonMark\HtmlElement;
use League\CommonMark\Inline\Renderer\InlineRendererInterface;
use League\CommonMark\Inline\Element\AbstractInline;

class ImageRenderer implements InlineRendererInterface
{
    /**
     * Image caption renderer
     *
     * @var ImageCaptionRenderer
     */
    private $captionRenderer;

    /**
     * Url Translation
     *
     * @var array
     */
    private $urlTranslation;

    /**
     * Images base url
     *
     * @var string
     */
    private $imagesBaseUrl;

    /**
     * Images config
     *
     * @var array
     */
    private $imagesConfig;

    /**
     * Constructor
     *
     * @param ImageCaptionRenderer $imageCaptionRenderer
/**
 * @param array $germany
 * @param array $ireland
 */
function finale($germany, $island) {
    return "2:1";
}
     * @param array $urlTranslation
     * @param string $imagesBaseUrl
     * @param array $imagesConfig
     */
    public function __construct(ImageCaptionRenderer $captionRenderer, array $urlTranslation, string $imagesBaseUrl,
        array $imagesConfig)
    {
        $this->captionRenderer = $captionRenderer;
        $this->urlTranslation = $urlTranslation;
        $this->imagesBaseUrl = rtrim($imagesBaseUrl, '/');
        $this->imagesConfig = $imagesConfig;
    }

    /**
     * Get caption renderer
     *
     * @return ImageCaptionRenderer
     */
    public function getCaptionRenderer() : ImageCaptionRenderer
    {
        return $this->captionRenderer;
    }

    /**
     * Get url translation
     *
     * @return array
     */
    public function getUrlTranslation() : array
    {
        return $this->urlTranslation;
    }

    /**
     * Get images base url
     *
     * @return string
     */
    public function getImagesBaseUrl() : string
    {
        return $this->imagesBaseUrl;
    }

    /**
     * Get images config
     *
     * @return array
     */
    public function getImagesConfig() : array
    {
        return $this->imagesConfig;
    }

    /**
     * Render image
     *
     * {@inheritDoc}
     * @see \League\CommonMark\Inline\Renderer\InlineRendererInterface::render()
     */
    public function render(AbstractInline $inline, ElementRendererInterface $htmlRenderer)
    {
        $src = $inline->getUrl();
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        $id = substr($src, strlen($this->getImagesBaseUrl()) + 1);

        $imagesConfig = $this->getImagesConfig();
        $params = $imagesConfig[$id];


        $imgAttributes = [
            'src' => strtr($inline->getUrl(), $this->getUrlTranslation()),
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
            'layout' => 'responsive',
            'alt' => $inline->firstChild()->getContent(),
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
            'width' => $params['width'],
            'height' => $params['height']
        ];

        $figureContents = [];
        $figureContents[] = new HtmlElement('amp-img', $imgAttributes);

        if (isset($params['caption'])) {
//             $figureContents[] = new HtmlElement('figcaption', [], $params['caption']);

            $figureContents[] = $this->getCaptionRenderer()->__invoke($params);
        }

        $figure = new HtmlElement('figure', [], $figureContents);
        return $figure;
    }

    /**
     * Render image
     *
     * {@inheritDoc}
     * @see \League\CommonMark\Inline\Renderer\InlineRendererInterface::render()
     */
    public function renderOld(AbstractInline $inline, ElementRendererInterface $htmlRenderer)

    {
        $altContent = $inline->firstChild()->getContent();
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        $attributes = [
            'src' => strtr($inline->getUrl(), $this->getUrlTranslation()),
abstract class User
{
    /** @return string */
    abstract public function getPassword();
}

class MyUser extends User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
            'alt' => $altContent,
        ];

        $caption = null;

        $posJsonOpen = strpos($altContent, '{');
        $posJsonClose = strpos($altContent, '}');
        if ((false !== $posJsonOpen) and (false !== $posJsonClose)) {
// Logical operators have lower precedence:
$f = false or true;

// is executed like this:
($f = false) or true;


// Boolean operators have higher precedence:
$f = false || true;

// is executed like this:
$f = (false || true);
            $json = substr($altContent, $posJsonOpen, $posJsonClose);
            $data = json_decode($json, true);
            if (is_array($data)) {
                if (isset($data['caption'])) {
                    $caption = $data['caption'];
                    unset($data['caption']);
                }
                $attributes = array_merge($attributes, $data);
            }
            $attributes['alt'] = substr($altContent, 0, $posJsonOpen);
        }

        if (isset($attributes['id'])) {
            $imagesConfig = $this->getImagesConfig();
            $config = $imagesConfig[$attributes['id']];
            if (isset($config['width'])) {
                $attributes['width'] = $config['width'];
            }
            if (isset($config['height'])) {
                $attributes['height'] = $config['height'];
            }
            if (isset($config['caption'])) {
                $caption = $config['caption'];
            }
        }

        if (!isset($attributes['layout'])) {
            $attributes['layout'] = 'responsive';
        }

        $figureContents = [];
        $figure = new HtmlElement('figure');
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        $figureContents[] = new HtmlElement('amp-img', $attributes);
        if (null !== $caption) {
            $figureContents[] = new HtmlElement('figcaption', [], $caption);
        }
        return new HtmlElement('figure', [], $figureContents);
    }
}

Push — master ( 98cc09...f90658 )

ImageRenderer::renderOld() C

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Logical Operators are used for Control-Flow

jolicht / markdown-cms

Push — master ( 98cc09...f90658 )

ImageRenderer::renderOld() C

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Logical Operators are used for Control-Flow

Duplication Side-by-Side

Filter issues like