DiffListItem - Code Metrics - Inspection of "Merge pull request #24 from caxy/feature/list-diff..." - caxy/php-htmldiff - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 24da9e...18bd75 )

by Josh

created 2016-01-12 19:35 UTC

DiffListItem A

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	120
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	1
Bugs	1	Features	0

Metric	Value
wmc	15
c	1
b	1
f	0
lcom	1
cbo	0
dl	0
loc	120
rs	10

13 Methods

Rating	Name	Size	Complexity
A	__construct()	7	1
A	getAttributes()	4	1
A	setAttributes()	6	1
A	getText()	4	1
A	setText()	6	1
A	getStartTag()	4	1
A	getStartTagWithDiffClass()	4	1
A	setStartTag()	6	1
A	getEndTag()	4	1
A	setEndTag()	6	1
A	getHtml()	6	3
A	getInnerHtml()	4	1
A	__toString()	4	1

<?php

namespace Caxy\HtmlDiff\ListDiff;

class DiffListItem
{
    protected $attributes = array();

    protected $text;

    protected $startTag;

    protected $endTag;

    public function __construct($text, $attributes = array(), $startTag, $endTag)
// $a must always be passed; it's default value is never used.
function someFunction($a = 5, $b) { }
    {
        $this->text = $text;
$a = "a";
$ab = "ab";
$abc = "abc";
        $this->attributes = $attributes;
        $this->startTag = $startTag;
$a = "a";
$ab = "ab";
$abc = "abc";
        $this->endTag = $endTag;
$a = "a";
$ab = "ab";
$abc = "abc";
    }

    /**
     * @return array
     */
    public function getAttributes()
    {
        return $this->attributes;
    }

    /**
     * @param array $attributes
     *
     * @return DiffListItem
     */
    public function setAttributes($attributes)
    {
        $this->attributes = $attributes;

        return $this;
    }

    /**
     * @return mixed
     */
    public function getText()
    {
        return $this->text;
    }

    /**
     * @param mixed $text
     *
     * @return DiffListItem
     */
    public function setText($text)
    {
        $this->text = $text;

        return $this;
    }

    /**
     * @return mixed
     */
    public function getStartTag()
    {
        return $this->startTag;
    }

    public function getStartTagWithDiffClass($class = 'normal')
    {
        return str_replace('>', ' class="'.$class.'">', $this->startTag);
    }

    /**
     * @param mixed $startTag
     *
     * @return DiffListItem
     */
    public function setStartTag($startTag)
    {
        $this->startTag = $startTag;

        return $this;
    }

    /**
     * @return mixed
     */
    public function getEndTag()
    {
        return $this->endTag;
    }

    /**
     * @param mixed $endTag
     *
     * @return DiffListItem
     */
    public function setEndTag($endTag)
    {
        $this->endTag = $endTag;

        return $this;
    }

    public function getHtml($class = 'normal', $wrapTag = null)
    {
        $startWrap = $wrapTag ? sprintf('<%s>', $wrapTag) : '';
        $endWrap = $wrapTag ? sprintf('</%s>', $wrapTag) : '';
$a = "a";
$ab = "ab";
$abc = "abc";
        return sprintf('%s%s%s%s%s', $this->getStartTagWithDiffClass($class), $startWrap, $this->getInnerHtml(), $endWrap, $this->endTag);
    }

    public function getInnerHtml()
    {
        return implode('', $this->text);
    }

    public function __toString()
    {
        return $this->getHtml();
    }
}

1			<?php
2
3			namespace Caxy\HtmlDiff\ListDiff;
4
5			class DiffListItem
6			{
7			protected $attributes = array();
8
9			protected $text;
10
11			protected $startTag;
12
13			protected $endTag;
14
15			public function __construct($text, $attributes = array(), $startTag, $endTag)
			0 ignored issues – show Coding Style introduced 2016-01-12 15:21 UTC by Report Bug Copy Issue Report Parameters which have default values should be placed at the end. If you place a parameter with a default value before a parameter with a default value, the default value of the first parameter will never be used as it will always need to be passed anyway: // $a must always be passed; it's default value is never used. function someFunction($a = 5, $b) { } Loading history...
16			{
17			$this->text = $text;
			0 ignored issues – show Coding Style introduced 2016-01-12 15:21 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 7 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
18			$this->attributes = $attributes;
19			$this->startTag = $startTag;
			0 ignored issues – show Coding Style introduced 2016-01-12 15:21 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 3 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
20			$this->endTag = $endTag;
			0 ignored issues – show Coding Style introduced 2016-01-12 15:21 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 5 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
21			}
22
23			/**
24			* @return array
25			*/
26			public function getAttributes()
27			{
28			return $this->attributes;
29			}
30
31			/**
32			* @param array $attributes
33			*
34			* @return DiffListItem
35			*/
36			public function setAttributes($attributes)
37			{
38			$this->attributes = $attributes;
39
40			return $this;
41			}
42
43			/**
44			* @return mixed
45			*/
46			public function getText()
47			{
48			return $this->text;
49			}
50
51			/**
52			* @param mixed $text
53			*
54			* @return DiffListItem
55			*/
56			public function setText($text)
57			{
58			$this->text = $text;
59
60			return $this;
61			}
62
63			/**
64			* @return mixed
65			*/
66			public function getStartTag()
67			{
68			return $this->startTag;
69			}
70
71			public function getStartTagWithDiffClass($class = 'normal')
72			{
73			return str_replace('>', ' class="'.$class.'">', $this->startTag);
74			}
75
76			/**
77			* @param mixed $startTag
78			*
79			* @return DiffListItem
80			*/
81			public function setStartTag($startTag)
82			{
83			$this->startTag = $startTag;
84
85			return $this;
86			}
87
88			/**
89			* @return mixed
90			*/
91			public function getEndTag()
92			{
93			return $this->endTag;
94			}
95
96			/**
97			* @param mixed $endTag
98			*
99			* @return DiffListItem
100			*/
101			public function setEndTag($endTag)
102			{
103			$this->endTag = $endTag;
104
105			return $this;
106			}
107
108			public function getHtml($class = 'normal', $wrapTag = null)
109			{
110			$startWrap = $wrapTag ? sprintf('<%s>', $wrapTag) : '';
111			$endWrap = $wrapTag ? sprintf('</%s>', $wrapTag) : '';
			0 ignored issues – show Coding Style introduced 2016-01-12 15:21 UTC by Report Bug Copy Issue Report Equals sign not aligned with surrounding assignments; expected 3 spaces but found 1 space This check looks for multiple assignments in successive lines of code. It will report an issue if the operators are not in a straight line. To visualize $a = "a"; $ab = "ab"; $abc = "abc"; will produce issues in the first and second line, while this second example $a = "a"; $ab = "ab"; $abc = "abc"; will produce no issues. Loading history...
112			return sprintf('%s%s%s%s%s', $this->getStartTagWithDiffClass($class), $startWrap, $this->getInnerHtml(), $endWrap, $this->endTag);
113			}
114
115			public function getInnerHtml()
116			{
117			return implode('', $this->text);
118			}
119
120			public function __toString()
121			{
122			return $this->getHtml();
123			}
124			}

caxy / php-htmldiff

Push — master ( 24da9e...18bd75 )

DiffListItem A

Complexity

Size/Duplication

Coupling/Cohesion

Importance

13 Methods

Duplication Side-by-Side

Filter issues like