Parsedown::inlineImage() - Code Metrics - Inspection of "Apply fixes from StyleCI (#1205)" - Cecilapp/Cecil - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — markdown-responsive ( 66d0bb...dea121 )

by Arnaud

created 2021-11-18 23:46 UTC

Parsedown::inlineImage() B

↳ Parent: Parsedown

Complexity

Conditions	11
Paths	25

Size

Total Lines	55
Code Lines	33

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	11
eloc	33
nc	25
nop	1
dl	0
loc	55
rs	7.3166
c	1
b	0
f	0

How to fix Long Method Complexity

<?php
/**
 * This file is part of the Cecil/Cecil package.
 *
 * Copyright (c) Arnaud Ligny <[email protected]>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */

namespace Cecil\Converter;

use Cecil\Assets\Asset;
use Cecil\Builder;

class Parsedown extends \ParsedownToC
{
    /** @var Builder */
    protected $builder;

    /** {@inheritdoc} */
    protected $regexAttribute = '(?:[#.][-\w:\\\]+[ ]*|[-\w:\\\]+(?:=(?:["\'][^\n]*?["\']|[^\s]+)?)?[ ]*)';

    public function __construct(Builder $builder)
    {
        $this->builder = $builder;
        parent::__construct(['selectors' => $this->builder->getConfig()->get('body.toc')]);
    }

    /**
     * {@inheritdoc}
     */
    protected function inlineImage($excerpt)
    {
        $image = parent::inlineImage($excerpt);
        if (!isset($image)) {
            return null;
        }
        $image['element']['attributes']['src'] = trim($this->removeQuery($image['element']['attributes']['src']));
        $asset = new Asset($this->builder, $image['element']['attributes']['src']);
        $width = $asset->getWidth();
        /**
         * Should be lazy loaded?
         */
        if ($this->builder->getConfig()->get('content.images.lazy.enabled')) {
            $image['element']['attributes']['loading'] = 'lazy';
        }
        /**
         * Should be responsive?
         */
        if ($this->builder->getConfig()->get('content.images.responsive.enabled')) {
            $steps = $this->builder->getConfig()->get('content.images.responsive.width.steps');
            $wMin = $this->builder->getConfig()->get('content.images.responsive.width.min');
            $wMax = $this->builder->getConfig()->get('content.images.responsive.width.max');
            if ($width < $wMax) {
                $wMax = $width;
            }
            $srcset = '';
            for ($i = 1; $i <= $steps; $i++) {
                $w = (int) ceil($wMin + ($wMax - $wMin) / $steps * $i);
                $a = new Asset($this->builder, $image['element']['attributes']['src']);
                $img = $a->resize($w);
                $srcset .= sprintf('%s %sw', $img, $w);
                if ($i < $steps) {
                    $srcset .= ', ';
                }
            }
            // ie: srcset="/img-480.jpg 480w, /img-800.jpg 800w"
            $image['element']['attributes']['srcset'] = $srcset;
            $image['element']['attributes']['sizes'] = $this->builder->getConfig()->get('content.images.responsive.sizes.default');
        }
        /**
         * Should be resized?
         */
        if ($image['element']['attributes']['width'] !== null
            && (int) $image['element']['attributes']['width'] < $width
            && $this->builder->getConfig()->get('content.images.resize.enabled')
        ) {
            $width = (int) $image['element']['attributes']['width'];
            $imageResized = $asset->resize($width);
            $image['element']['attributes']['src'] = $imageResized;
        }
        if ($image['element']['attributes']['width'] === null) {
            $image['element']['attributes']['width'] = $width;
        }

        return $image;
    }

    /**
     * {@inheritdoc}
     */
    protected function parseAttributeData($attributeString)
    {
        $Data = [];

        $attributes = preg_split('/[ ]+/', $attributeString, -1, PREG_SPLIT_NO_EMPTY);

        $HtmlAtt = [];

        foreach ($attributes as $attribute) {
            if ($attribute[0] === '#') {
                $Data['id'] = substr($attribute, 1);
            } elseif ($attribute[0] === '.') {
                $classes[] = substr($attribute, 1);
            } else {
                parse_str($attribute, $parsed);
                $HtmlAtt = array_merge($HtmlAtt, $parsed);
            }
        }

        if (isset($classes)) {
            $Data['class'] = implode(' ', $classes);
        }
        if (isset($HtmlAtt)) {
            foreach ($HtmlAtt as $a => $v) {
                $Data[$a] = trim($v, '"');
            }
        }

        return $Data;
    }

    /**
     * Removes query string from URL.
     */
    private function removeQuery(string $path): string
    {
        return strtok($path, '?');
    }
}


1			<?php
2			/**
3			* This file is part of the Cecil/Cecil package.
4			*
5			* Copyright (c) Arnaud Ligny <[email protected]>
6			*
7			* For the full copyright and license information, please view the LICENSE
8			* file that was distributed with this source code.
9			*/
10
11			namespace Cecil\Converter;
12
13			use Cecil\Assets\Asset;
14			use Cecil\Builder;
15
16			class Parsedown extends \ParsedownToC
17			{
18			/** @var Builder */
19			protected $builder;
20
21			/** {@inheritdoc} */
22			protected $regexAttribute = '(?:[#.][-\w:\\\]+[ ]\|[-\w:\\\]+(?:=(?:["\'][^\n]?["\']\|[^\s]+)?)?[ ]*)';
23
24			public function __construct(Builder $builder)
25			{
26			$this->builder = $builder;
27			parent::__construct(['selectors' => $this->builder->getConfig()->get('body.toc')]);
28			}
29
30			/**
31			* {@inheritdoc}
32			*/
33			protected function inlineImage($excerpt)
34			{
35			$image = parent::inlineImage($excerpt);
36			if (!isset($image)) {
37			return null;
38			}
39			$image['element']['attributes']['src'] = trim($this->removeQuery($image['element']['attributes']['src']));
40			$asset = new Asset($this->builder, $image['element']['attributes']['src']);
41			$width = $asset->getWidth();
42			/**
43			* Should be lazy loaded?
44			*/
45			if ($this->builder->getConfig()->get('content.images.lazy.enabled')) {
46			$image['element']['attributes']['loading'] = 'lazy';
47			}
48			/**
49			* Should be responsive?
50			*/
51			if ($this->builder->getConfig()->get('content.images.responsive.enabled')) {
52			$steps = $this->builder->getConfig()->get('content.images.responsive.width.steps');
53			$wMin = $this->builder->getConfig()->get('content.images.responsive.width.min');
54			$wMax = $this->builder->getConfig()->get('content.images.responsive.width.max');
55			if ($width < $wMax) {
56			$wMax = $width;
57			}
58			$srcset = '';
59			for ($i = 1; $i <= $steps; $i++) {
60			$w = (int) ceil($wMin + ($wMax - $wMin) / $steps * $i);
61			$a = new Asset($this->builder, $image['element']['attributes']['src']);
62			$img = $a->resize($w);
63			$srcset .= sprintf('%s %sw', $img, $w);
64			if ($i < $steps) {
65			$srcset .= ', ';
66			}
67			}
68			// ie: srcset="/img-480.jpg 480w, /img-800.jpg 800w"
69			$image['element']['attributes']['srcset'] = $srcset;
70			$image['element']['attributes']['sizes'] = $this->builder->getConfig()->get('content.images.responsive.sizes.default');
71			}
72			/**
73			* Should be resized?
74			*/
75			if ($image['element']['attributes']['width'] !== null
76			&& (int) $image['element']['attributes']['width'] < $width
77			&& $this->builder->getConfig()->get('content.images.resize.enabled')
78			) {
79			$width = (int) $image['element']['attributes']['width'];
80			$imageResized = $asset->resize($width);
81			$image['element']['attributes']['src'] = $imageResized;
82			}
83			if ($image['element']['attributes']['width'] === null) {
84			$image['element']['attributes']['width'] = $width;
85			}
86
87			return $image;
88			}
89
90			/**
91			* {@inheritdoc}
92			*/
93			protected function parseAttributeData($attributeString)
94			{
95			$Data = [];
96
97			$attributes = preg_split('/[ ]+/', $attributeString, -1, PREG_SPLIT_NO_EMPTY);
98
99			$HtmlAtt = [];
100
101			foreach ($attributes as $attribute) {
102			if ($attribute[0] === '#') {
103			$Data['id'] = substr($attribute, 1);
104			} elseif ($attribute[0] === '.') {
105			$classes[] = substr($attribute, 1);
106			} else {
107			parse_str($attribute, $parsed);
108			$HtmlAtt = array_merge($HtmlAtt, $parsed);
109			}
110			}
111
112			if (isset($classes)) {
113			$Data['class'] = implode(' ', $classes);
114			}
115			if (isset($HtmlAtt)) {
116			foreach ($HtmlAtt as $a => $v) {
117			$Data[$a] = trim($v, '"');
118			}
119			}
120
121			return $Data;
122			}
123
124			/**
125			* Removes query string from URL.
126			*/
127			private function removeQuery(string $path): string
128			{
129			return strtok($path, '?');
130			}
131			}
132

Cecilapp / Cecil

Push — markdown-responsive ( 66d0bb...dea121 )

Parsedown::inlineImage() B

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like