Breadcrumbs::getHtml() - Code Metrics - Inspection of "SCSS. Consructor added." - floor12/breadcrumbs-microdata - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( d11c82...442210 )

by Evgenii

created 2021-05-04 11:52 UTC

Breadcrumbs::getHtml() A

↳ Parent: Breadcrumbs

Complexity

Conditions	4
Paths	4

Size

Total Lines	21
Code Lines	15

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	15
c	0
b	0
f	0
dl	0
loc	21
rs	9.7666
cc	4
nc	4
nop	0

<?php


namespace floor12\Breadcrumbs;


class Breadcrumbs
{
    const HTML_OL = '<ol itemscope itemtype="http://schema.org/BreadcrumbList" class="%s" id="%s">%s</ol>';
    const HTML_LI = '<li itemprop="itemListElement" itemscope itemtype="http://schema.org/ListItem">%s</li>';
    const HTML_A = '<a data-pjax="0" itemprop="item" href="%s">%s</a>';
    const HTML_SPAN = '<span itemprop="name">%s</span>';
    const HTML_META_POSITION = '<meta itemprop="position" content="%d">';

    protected $cssClass = 'f12-breadcrumbs';
    protected $mainId = 'f12-breadcrumbs';
    protected $elements = [];

    public function __construct(array $elements = [])
    {
        $this->elements = $elements;
    }

    public function addElement(string $name, string $url = null): self
    {
        $this->elements[$url] = $name;
        return $this;
    }

    public function getHtml(): ?string
    {
        if (\count($this->elements) === 0) {
            return null;
        }
        $renderedElements = [];
        $position = 0;
        foreach ($this->elements as $url => $name) {
            $position++;
            $element = $element = sprintf(self::HTML_SPAN, $name) . sprintf(self::HTML_META_POSITION, $position);

            if (!is_numeric($url)) {
                $element = sprintf(self::HTML_A, $url, $element);
            }
            $renderedElements[] = sprintf(self::HTML_LI, $element);
        }

        return sprintf(
            self::HTML_OL,
            $this->cssClass,
            $this->mainId,
            implode($renderedElements)
        );
    }

    public function setCssClass(string $cssClass): self
    {
        $this->cssClass = $cssClass;
        return $this;
    }

    public function setMainId(string $mainId): self
    {
        $this->mainId = $mainId;
        return $this;
    }
}


1			<?php
2
3
4			namespace floor12\Breadcrumbs;
5
6
7			class Breadcrumbs
8			{
9			const HTML_OL = '<ol itemscope itemtype="http://schema.org/BreadcrumbList" class="%s" id="%s">%s</ol>';
10			const HTML_LI = '<li itemprop="itemListElement" itemscope itemtype="http://schema.org/ListItem">%s</li>';
11			const HTML_A = '<a data-pjax="0" itemprop="item" href="%s">%s</a>';
12			const HTML_SPAN = '<span itemprop="name">%s</span>';
13			const HTML_META_POSITION = '<meta itemprop="position" content="%d">';
14
15			protected $cssClass = 'f12-breadcrumbs';
16			protected $mainId = 'f12-breadcrumbs';
17			protected $elements = [];
18
19			public function __construct(array $elements = [])
20			{
21			$this->elements = $elements;
22			}
23
24			public function addElement(string $name, string $url = null): self
25			{
26			$this->elements[$url] = $name;
27			return $this;
28			}
29
30			public function getHtml(): ?string
31			{
32			if (\count($this->elements) === 0) {
33			return null;
34			}
35			$renderedElements = [];
36			$position = 0;
37			foreach ($this->elements as $url => $name) {
38			$position++;
39			$element = $element = sprintf(self::HTML_SPAN, $name) . sprintf(self::HTML_META_POSITION, $position);
			0 ignored issues – show Unused Code introduced 2021-05-03 16:14 UTC by Report Bug Copy Issue Report The assignment to `$element` is dead and can be removed. Loading history...
40			if (!is_numeric($url)) {
41			$element = sprintf(self::HTML_A, $url, $element);
42			}
43			$renderedElements[] = sprintf(self::HTML_LI, $element);
44			}
45
46			return sprintf(
47			self::HTML_OL,
48			$this->cssClass,
49			$this->mainId,
50			implode($renderedElements)
51			);
52			}
53
54			public function setCssClass(string $cssClass): self
55			{
56			$this->cssClass = $cssClass;
57			return $this;
58			}
59
60			public function setMainId(string $mainId): self
61			{
62			$this->mainId = $mainId;
63			return $this;
64			}
65			}
66

floor12 / breadcrumbs-microdata

Push — master ( d11c82...442210 )

Breadcrumbs::getHtml() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like