GitHub Access Token became invalid

It seems like the GitHub access token used for retrieving details about this repository from GitHub became invalid. This might prevent certain types of inspections from being run (in particular, everything related to pull requests).
Please ask an admin of your repository to re-new the access token on this website.
Completed
Pull Request — master (#8)
by no
08:18
created
src/XmlNodeRecursiveIterator.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -4,9 +4,9 @@  discard block
 block discarded – undo
4 4
 
5 5
 class XmlNodeRecursiveIterator extends \ArrayIterator {
6 6
 
7
-    public function __construct( \DOMNodeList $nodeList ) {
8
-        $queue = $this->addElementsToQueue( [], $nodeList );
9
-        parent::__construct( $queue );
7
+    public function __construct(\DOMNodeList $nodeList) {
8
+        $queue = $this->addElementsToQueue([], $nodeList);
9
+        parent::__construct($queue);
10 10
     }
11 11
 
12 12
     /**
@@ -15,12 +15,12 @@  discard block
 block discarded – undo
15 15
      *
16 16
      * @return array New queue
17 17
      */
18
-    private function addElementsToQueue( array $queue, \DOMNodeList $nodeList ) {
18
+    private function addElementsToQueue(array $queue, \DOMNodeList $nodeList) {
19 19
         /** @var \DOMElement $node */
20
-        foreach ( $nodeList as $node ) {
20
+        foreach ($nodeList as $node) {
21 21
             $queue[] = $node;
22
-            if ( $node->childNodes !== null ) {
23
-                $queue = $this->addElementsToQueue( $queue, $node->childNodes );
22
+            if ($node->childNodes !== null) {
23
+                $queue = $this->addElementsToQueue($queue, $node->childNodes);
24 24
             }
25 25
         }
26 26
 
Please login to merge, or discard this patch.
src/TagNameMatcher.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -13,18 +13,18 @@  discard block
 block discarded – undo
13 13
      */
14 14
     private $tagNameMatcher;
15 15
 
16
-    public static function withTagName( $tagName ) {
17
-        return new static( Util::wrapValueWithIsEqual( $tagName ) );
16
+    public static function withTagName($tagName) {
17
+        return new static(Util::wrapValueWithIsEqual($tagName));
18 18
     }
19 19
 
20
-    public function __construct( Matcher $tagNameMatcher ) {
20
+    public function __construct(Matcher $tagNameMatcher) {
21 21
         parent::__construct();
22 22
         $this->tagNameMatcher = $tagNameMatcher;
23 23
     }
24 24
 
25
-    public function describeTo( Description $description ) {
26
-        $description->appendText( 'with tag name ' )
27
-            ->appendDescriptionOf( $this->tagNameMatcher );
25
+    public function describeTo(Description $description) {
26
+        $description->appendText('with tag name ')
27
+            ->appendDescriptionOf($this->tagNameMatcher);
28 28
     }
29 29
 
30 30
     /**
@@ -33,10 +33,10 @@  discard block
 block discarded – undo
33 33
      *
34 34
      * @return bool
35 35
      */
36
-    protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
37
-        $mismatchDescription->appendText( 'tag name ' );
38
-        $this->tagNameMatcher->describeMismatch( $item->tagName, $mismatchDescription );
39
-        return $this->tagNameMatcher->matches( $item->tagName );
36
+    protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
37
+        $mismatchDescription->appendText('tag name ');
38
+        $this->tagNameMatcher->describeMismatch($item->tagName, $mismatchDescription);
39
+        return $this->tagNameMatcher->matches($item->tagName);
40 40
     }
41 41
 
42 42
 }
Please login to merge, or discard this patch.
src/ChildElementMatcher.php 1 patch
Spacing   +18 added lines, -18 removed lines patch added patch discarded remove patch
@@ -13,19 +13,19 @@  discard block
 block discarded – undo
13 13
      */
14 14
     private $matcher;
15 15
 
16
-    public static function havingChild( Matcher $elementMatcher = null ) {
17
-        return new static( $elementMatcher );
16
+    public static function havingChild(Matcher $elementMatcher = null) {
17
+        return new static($elementMatcher);
18 18
     }
19 19
 
20
-    public function __construct( Matcher $matcher = null ) {
21
-        parent::__construct( \DOMNode::class );
20
+    public function __construct(Matcher $matcher = null) {
21
+        parent::__construct(\DOMNode::class);
22 22
         $this->matcher = $matcher;
23 23
     }
24 24
 
25
-    public function describeTo( Description $description ) {
26
-        $description->appendText( 'having child ' );
27
-        if ( $this->matcher ) {
28
-            $description->appendDescriptionOf( $this->matcher );
25
+    public function describeTo(Description $description) {
26
+        $description->appendText('having child ');
27
+        if ($this->matcher) {
28
+            $description->appendDescriptionOf($this->matcher);
29 29
         }
30 30
     }
31 31
 
@@ -35,32 +35,32 @@  discard block
 block discarded – undo
35 35
      *
36 36
      * @return bool
37 37
      */
38
-    protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
39
-        if ( $item instanceof \DOMDocument ) {
40
-            $directChildren = iterator_to_array( $item->documentElement->childNodes );
38
+    protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
39
+        if ($item instanceof \DOMDocument) {
40
+            $directChildren = iterator_to_array($item->documentElement->childNodes);
41 41
 
42
-            $body = array_shift( $directChildren );
42
+            $body = array_shift($directChildren);
43 43
             $directChildren = $body->childNodes;
44 44
         } else {
45 45
             $directChildren = $item->childNodes;
46 46
         }
47 47
 
48
-        if ( $directChildren->length === 0 ) {
49
-            $mismatchDescription->appendText( 'having no children' );
48
+        if ($directChildren->length === 0) {
49
+            $mismatchDescription->appendText('having no children');
50 50
             return false;
51 51
         }
52 52
 
53
-        if ( !$this->matcher ) {
53
+        if (!$this->matcher) {
54 54
             return $directChildren->length > 0;
55 55
         }
56 56
 
57
-        foreach ( new XmlNodeRecursiveIterator( $directChildren ) as $child ) {
58
-            if ( $this->matcher && $this->matcher->matches( $child ) ) {
57
+        foreach (new XmlNodeRecursiveIterator($directChildren) as $child) {
58
+            if ($this->matcher && $this->matcher->matches($child)) {
59 59
                 return true;
60 60
             }
61 61
         }
62 62
 
63
-        $mismatchDescription->appendText( 'having no children ' )->appendDescriptionOf( $this->matcher );
63
+        $mismatchDescription->appendText('having no children ')->appendDescriptionOf($this->matcher);
64 64
         return false;
65 65
     }
66 66
 
Please login to merge, or discard this patch.
src/functions.php 1 patch
Spacing   +27 added lines, -27 removed lines patch added patch discarded remove patch
@@ -2,88 +2,88 @@
 block discarded – undo
2 2
 
3 3
 use Hamcrest\Matcher;
4 4
 
5
-if ( !function_exists( 'htmlPiece' ) ) {
5
+if (!function_exists('htmlPiece')) {
6 6
     /**
7 7
      * @param mixed $elementMatcher
8 8
      *
9 9
      * @return \WMDE\HamcrestHtml\HtmlMatcher
10 10
      */
11
-    function htmlPiece( Matcher $elementMatcher = null ) {
12
-        return \WMDE\HamcrestHtml\HtmlMatcher::htmlPiece( $elementMatcher );
11
+    function htmlPiece(Matcher $elementMatcher = null) {
12
+        return \WMDE\HamcrestHtml\HtmlMatcher::htmlPiece($elementMatcher);
13 13
     }
14 14
 }
15 15
 
16
-if ( !function_exists( 'havingRootElement' ) ) {
17
-    function havingRootElement( Matcher $matcher = null ) {
18
-        return \WMDE\HamcrestHtml\RootElementMatcher::havingRootElement( $matcher );
16
+if (!function_exists('havingRootElement')) {
17
+    function havingRootElement(Matcher $matcher = null) {
18
+        return \WMDE\HamcrestHtml\RootElementMatcher::havingRootElement($matcher);
19 19
     }
20 20
 }
21 21
 
22
-if ( !function_exists( 'havingDirectChild' ) ) {
23
-    function havingDirectChild( Matcher $elementMatcher = null ) {
24
-        return \WMDE\HamcrestHtml\DirectChildElementMatcher::havingDirectChild( $elementMatcher );
22
+if (!function_exists('havingDirectChild')) {
23
+    function havingDirectChild(Matcher $elementMatcher = null) {
24
+        return \WMDE\HamcrestHtml\DirectChildElementMatcher::havingDirectChild($elementMatcher);
25 25
     }
26 26
 }
27 27
 
28
-if ( !function_exists( 'havingChild' ) ) {
29
-    function havingChild( Matcher $elementMatcher = null ) {
30
-        return \WMDE\HamcrestHtml\ChildElementMatcher::havingChild( $elementMatcher );
28
+if (!function_exists('havingChild')) {
29
+    function havingChild(Matcher $elementMatcher = null) {
30
+        return \WMDE\HamcrestHtml\ChildElementMatcher::havingChild($elementMatcher);
31 31
     }
32 32
 }
33 33
 
34
-if ( !function_exists( 'withTagName' ) ) {
34
+if (!function_exists('withTagName')) {
35 35
     /**
36 36
      * @param Matcher|string $tagName
37 37
      *
38 38
      * @return \WMDE\HamcrestHtml\TagNameMatcher
39 39
      */
40
-    function withTagName( $tagName ) {
41
-        return \WMDE\HamcrestHtml\TagNameMatcher::withTagName( $tagName );
40
+    function withTagName($tagName) {
41
+        return \WMDE\HamcrestHtml\TagNameMatcher::withTagName($tagName);
42 42
     }
43 43
 }
44 44
 
45
-if ( !function_exists( 'withAttribute' ) ) {
45
+if (!function_exists('withAttribute')) {
46 46
     /**
47 47
      * @param Matcher|string $attributeName
48 48
      *
49 49
      * @return \WMDE\HamcrestHtml\AttributeMatcher
50 50
      */
51
-    function withAttribute( $attributeName ) {
52
-        return \WMDE\HamcrestHtml\AttributeMatcher::withAttribute( $attributeName );
51
+    function withAttribute($attributeName) {
52
+        return \WMDE\HamcrestHtml\AttributeMatcher::withAttribute($attributeName);
53 53
     }
54 54
 }
55 55
 
56
-if ( !function_exists( 'withClass' ) ) {
56
+if (!function_exists('withClass')) {
57 57
     /**
58 58
      * @param Matcher|string $class
59 59
      *
60 60
      * @return \WMDE\HamcrestHtml\ClassMatcher
61 61
      */
62
-    function withClass( $class ) {
62
+    function withClass($class) {
63 63
         //TODO don't allow to call with empty string
64 64
 
65
-        return \WMDE\HamcrestHtml\ClassMatcher::withClass( $class );
65
+        return \WMDE\HamcrestHtml\ClassMatcher::withClass($class);
66 66
     }
67 67
 }
68 68
 
69
-if ( !function_exists( 'havingTextContents' ) ) {
69
+if (!function_exists('havingTextContents')) {
70 70
     /**
71 71
      * @param Matcher|string $text
72 72
      *
73 73
      * @return \WMDE\HamcrestHtml\TextContentsMatcher
74 74
      */
75
-    function havingTextContents( $text ) {
76
-        return \WMDE\HamcrestHtml\TextContentsMatcher::havingTextContents( $text );
75
+    function havingTextContents($text) {
76
+        return \WMDE\HamcrestHtml\TextContentsMatcher::havingTextContents($text);
77 77
     }
78 78
 }
79 79
 
80
-if ( !function_exists( 'tagMatchingOutline' ) ) {
80
+if (!function_exists('tagMatchingOutline')) {
81 81
     /**
82 82
      * @param string $htmlOutline
83 83
      *
84 84
      * @return \WMDE\HamcrestHtml\ComplexTagMatcher
85 85
      */
86
-    function tagMatchingOutline( $htmlOutline ) {
87
-        return \WMDE\HamcrestHtml\ComplexTagMatcher::tagMatchingOutline( $htmlOutline );
86
+    function tagMatchingOutline($htmlOutline) {
87
+        return \WMDE\HamcrestHtml\ComplexTagMatcher::tagMatchingOutline($htmlOutline);
88 88
     }
89 89
 }
Please login to merge, or discard this patch.
src/ClassMatcher.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -13,17 +13,17 @@  discard block
 block discarded – undo
13 13
      */
14 14
     private $classMatcher;
15 15
 
16
-    public static function withClass( $class ) {
17
-        return new static( Util::wrapValueWithIsEqual( $class ) );
16
+    public static function withClass($class) {
17
+        return new static(Util::wrapValueWithIsEqual($class));
18 18
     }
19 19
 
20
-    public function __construct( Matcher $class ) {
20
+    public function __construct(Matcher $class) {
21 21
         parent::__construct();
22 22
         $this->classMatcher = $class;
23 23
     }
24 24
 
25
-    public function describeTo( Description $description ) {
26
-        $description->appendText( 'with class ' )->appendDescriptionOf( $this->classMatcher );
25
+    public function describeTo(Description $description) {
26
+        $description->appendText('with class ')->appendDescriptionOf($this->classMatcher);
27 27
     }
28 28
 
29 29
     /**
@@ -32,12 +32,12 @@  discard block
 block discarded – undo
32 32
      *
33 33
      * @return bool
34 34
      */
35
-    protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
36
-        $classAttribute = $item->getAttribute( 'class' );
35
+    protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
36
+        $classAttribute = $item->getAttribute('class');
37 37
 
38
-        $classes = preg_split( '/\s+/u', $classAttribute );
39
-        foreach ( $classes as $class ) {
40
-            if ( $this->classMatcher->matches( $class ) ) {
38
+        $classes = preg_split('/\s+/u', $classAttribute);
39
+        foreach ($classes as $class) {
40
+            if ($this->classMatcher->matches($class)) {
41 41
                 return true;
42 42
             }
43 43
         }
Please login to merge, or discard this patch.
src/TextContentsMatcher.php 1 patch
Spacing   +7 added lines, -7 removed lines patch added patch discarded remove patch
@@ -13,17 +13,17 @@  discard block
 block discarded – undo
13 13
      */
14 14
     private $matcher;
15 15
 
16
-    public static function havingTextContents( $text ) {
17
-        return new static( Util::wrapValueWithIsEqual( $text ) );
16
+    public static function havingTextContents($text) {
17
+        return new static(Util::wrapValueWithIsEqual($text));
18 18
     }
19 19
 
20
-    public function __construct( Matcher $matcher ) {
20
+    public function __construct(Matcher $matcher) {
21 21
         parent::__construct();
22 22
         $this->matcher = $matcher;
23 23
     }
24 24
 
25
-    public function describeTo( Description $description ) {
26
-        $description->appendText( 'having text contents ' )->appendDescriptionOf( $this->matcher );
25
+    public function describeTo(Description $description) {
26
+        $description->appendText('having text contents ')->appendDescriptionOf($this->matcher);
27 27
     }
28 28
 
29 29
     /**
@@ -32,8 +32,8 @@  discard block
 block discarded – undo
32 32
      *
33 33
      * @return bool
34 34
      */
35
-    protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
36
-        return $this->matcher->matches( $item->textContent );
35
+    protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
36
+        return $this->matcher->matches($item->textContent);
37 37
     }
38 38
 
39 39
 }
Please login to merge, or discard this patch.
src/DirectChildElementMatcher.php 1 patch
Spacing   +20 added lines, -20 removed lines patch added patch discarded remove patch
@@ -13,19 +13,19 @@  discard block
 block discarded – undo
13 13
      */
14 14
     private $matcher;
15 15
 
16
-    public static function havingDirectChild( Matcher $elementMatcher = null ) {
17
-        return new static( $elementMatcher );
16
+    public static function havingDirectChild(Matcher $elementMatcher = null) {
17
+        return new static($elementMatcher);
18 18
     }
19 19
 
20
-    public function __construct( Matcher $matcher = null ) {
21
-        parent::__construct( \DOMNode::class );
20
+    public function __construct(Matcher $matcher = null) {
21
+        parent::__construct(\DOMNode::class);
22 22
         $this->matcher = $matcher;
23 23
     }
24 24
 
25
-    public function describeTo( Description $description ) {
26
-        $description->appendText( 'having direct child ' );
27
-        if ( $this->matcher ) {
28
-            $description->appendDescriptionOf( $this->matcher );
25
+    public function describeTo(Description $description) {
26
+        $description->appendText('having direct child ');
27
+        if ($this->matcher) {
28
+            $description->appendDescriptionOf($this->matcher);
29 29
         }
30 30
     }
31 31
 
@@ -35,36 +35,36 @@  discard block
 block discarded – undo
35 35
      *
36 36
      * @return bool
37 37
      */
38
-    protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
39
-        if ( $item instanceof \DOMDocument ) {
40
-            $directChildren = iterator_to_array( $item->documentElement->childNodes );
38
+    protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
39
+        if ($item instanceof \DOMDocument) {
40
+            $directChildren = iterator_to_array($item->documentElement->childNodes);
41 41
 
42
-            $body = array_shift( $directChildren );
42
+            $body = array_shift($directChildren);
43 43
             $directChildren = $body->childNodes;
44 44
         } else {
45 45
             $directChildren = $item->childNodes;
46 46
         }
47 47
 
48
-        if ( $directChildren->length === 0 ) {
49
-            $mismatchDescription->appendText( 'with no direct children' );
48
+        if ($directChildren->length === 0) {
49
+            $mismatchDescription->appendText('with no direct children');
50 50
             return false;
51 51
         }
52 52
 
53 53
         $childWord = $directChildren->length === 1 ? 'child' : 'children';
54 54
 
55
-        $mismatchDescription->appendText( "with direct {$childWord} " );
55
+        $mismatchDescription->appendText("with direct {$childWord} ");
56 56
 
57
-        if ( !$this->matcher ) {
58
-            return count( $directChildren ) !== 0;
57
+        if (!$this->matcher) {
58
+            return count($directChildren) !== 0;
59 59
         }
60 60
 
61
-        foreach ( $directChildren as $child ) {
62
-            if ( $this->matcher && $this->matcher->matches( $child ) ) {
61
+        foreach ($directChildren as $child) {
62
+            if ($this->matcher && $this->matcher->matches($child)) {
63 63
                 return true;
64 64
             }
65 65
         }
66 66
 
67
-        $this->matcher->describeMismatch( $child, $mismatchDescription );
67
+        $this->matcher->describeMismatch($child, $mismatchDescription);
68 68
 
69 69
         return false;
70 70
     }
Please login to merge, or discard this patch.
src/ComplexTagMatcher.php 1 patch
Spacing   +59 added lines, -59 removed lines patch added patch discarded remove patch
@@ -30,21 +30,21 @@  discard block
 block discarded – undo
30 30
      * @param string $htmlOutline
31 31
      * @return self
32 32
      */
33
-    public static function tagMatchingOutline( $htmlOutline ) {
34
-        return new self( $htmlOutline );
33
+    public static function tagMatchingOutline($htmlOutline) {
34
+        return new self($htmlOutline);
35 35
     }
36 36
 
37
-    public function __construct( $tagHtmlRepresentation ) {
37
+    public function __construct($tagHtmlRepresentation) {
38 38
         parent::__construct();
39 39
 
40 40
         $this->tagHtmlOutline = $tagHtmlRepresentation;
41
-        $this->matcher = $this->createMatcherFromHtml( $tagHtmlRepresentation );
41
+        $this->matcher = $this->createMatcherFromHtml($tagHtmlRepresentation);
42 42
     }
43 43
 
44
-    public function describeTo( Description $description ) {
45
-        $description->appendText( 'tag matching outline `' )
46
-            ->appendText( $this->tagHtmlOutline )
47
-            ->appendText( '` ' );
44
+    public function describeTo(Description $description) {
45
+        $description->appendText('tag matching outline `')
46
+            ->appendText($this->tagHtmlOutline)
47
+            ->appendText('` ');
48 48
     }
49 49
 
50 50
     /**
@@ -53,40 +53,40 @@  discard block
 block discarded – undo
53 53
      *
54 54
      * @return bool
55 55
      */
56
-    protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
57
-        $result = $this->matcher->matches( $item );
58
-        if ( !$result ) {
59
-            $mismatchDescription->appendText( 'was `' )
60
-                ->appendText( $this->elementToString( $item ) )
61
-                ->appendText( '`' );
56
+    protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
57
+        $result = $this->matcher->matches($item);
58
+        if (!$result) {
59
+            $mismatchDescription->appendText('was `')
60
+                ->appendText($this->elementToString($item))
61
+                ->appendText('`');
62 62
         }
63 63
         return $result;
64 64
     }
65 65
 
66
-    private function createMatcherFromHtml( $htmlOutline ) {
67
-        $document = $this->parseHtml( $htmlOutline );
68
-        $targetTag = $this->getSingleTagFromThe( $document );
66
+    private function createMatcherFromHtml($htmlOutline) {
67
+        $document = $this->parseHtml($htmlOutline);
68
+        $targetTag = $this->getSingleTagFromThe($document);
69 69
 
70
-        $this->assertTagDoesNotContainChildren( $targetTag );
70
+        $this->assertTagDoesNotContainChildren($targetTag);
71 71
 
72
-        $attributeMatchers = $this->createAttributeMatchers( $htmlOutline, $targetTag );
73
-        $classMatchers = $this->createClassMatchers( $targetTag );
72
+        $attributeMatchers = $this->createAttributeMatchers($htmlOutline, $targetTag);
73
+        $classMatchers = $this->createClassMatchers($targetTag);
74 74
 
75 75
         return AllOf::allOf(
76
-            new TagNameMatcher( IsEqual::equalTo( $targetTag->tagName ) ),
77
-            call_user_func_array( [ AllOf::class, 'allOf' ], $attributeMatchers ),
78
-            call_user_func_array( [ AllOf::class, 'allOf' ], $classMatchers )
76
+            new TagNameMatcher(IsEqual::equalTo($targetTag->tagName)),
77
+            call_user_func_array([AllOf::class, 'allOf'], $attributeMatchers),
78
+            call_user_func_array([AllOf::class, 'allOf'], $classMatchers)
79 79
         );
80 80
     }
81 81
 
82
-    private function isUnknownTagError( \LibXMLError $error ) {
82
+    private function isUnknownTagError(\LibXMLError $error) {
83 83
         return $error->code === self::XML_UNKNOWN_TAG_ERROR_CODE;
84 84
     }
85 85
 
86
-    private function isBooleanAttribute( $inputHtml, $attributeName ) {
87
-        $quotedName = preg_quote( $attributeName, '/' );
86
+    private function isBooleanAttribute($inputHtml, $attributeName) {
87
+        $quotedName = preg_quote($attributeName, '/');
88 88
 
89
-        $attributeHasValueAssigned = preg_match( "/\b{$quotedName}\s*=/ui", $inputHtml );
89
+        $attributeHasValueAssigned = preg_match("/\b{$quotedName}\s*=/ui", $inputHtml);
90 90
         return !$attributeHasValueAssigned;
91 91
     }
92 92
 
@@ -96,26 +96,26 @@  discard block
 block discarded – undo
96 96
      * @return \DOMDocument
97 97
      * @throws \InvalidArgumentException
98 98
      */
99
-    private function parseHtml( $html ) {
100
-        $internalErrors = libxml_use_internal_errors( true );
99
+    private function parseHtml($html) {
100
+        $internalErrors = libxml_use_internal_errors(true);
101 101
         $document = new \DOMDocument();
102 102
 
103
-        if ( !@$document->loadHTML( $html ) ) {
104
-            throw new \InvalidArgumentException( "There was some parsing error of `$html`" );
103
+        if (!@$document->loadHTML($html)) {
104
+            throw new \InvalidArgumentException("There was some parsing error of `$html`");
105 105
         }
106 106
 
107 107
         $errors = libxml_get_errors();
108 108
         libxml_clear_errors();
109
-        libxml_use_internal_errors( $internalErrors );
109
+        libxml_use_internal_errors($internalErrors);
110 110
 
111 111
         /** @var \LibXMLError $error */
112
-        foreach ( $errors as $error ) {
113
-            if ( $this->isUnknownTagError( $error ) ) {
112
+        foreach ($errors as $error) {
113
+            if ($this->isUnknownTagError($error)) {
114 114
                 continue;
115 115
             }
116 116
 
117 117
             throw new \InvalidArgumentException(
118
-                'There was parsing error: ' . trim( $error->message ) . ' on line ' . $error->line
118
+                'There was parsing error: ' . trim($error->message) . ' on line ' . $error->line
119 119
             );
120 120
         }
121 121
 
@@ -128,22 +128,22 @@  discard block
 block discarded – undo
128 128
      * @return \DOMElement
129 129
      * @throws \InvalidArgumentException
130 130
      */
131
-    private function getSingleTagFromThe( \DOMDocument $document ) {
132
-        $directChildren = iterator_to_array( $document->documentElement->childNodes );
131
+    private function getSingleTagFromThe(\DOMDocument $document) {
132
+        $directChildren = iterator_to_array($document->documentElement->childNodes);
133 133
 
134
-        $body = array_shift( $directChildren );
135
-        $directChildren = iterator_to_array( $body->childNodes );
134
+        $body = array_shift($directChildren);
135
+        $directChildren = iterator_to_array($body->childNodes);
136 136
 
137
-        if ( count( $directChildren ) !== 1 ) {
138
-            throw new InvalidArgumentException( 'Expected exactly 1 tag description, got ' . count( $directChildren ) );
137
+        if (count($directChildren) !== 1) {
138
+            throw new InvalidArgumentException('Expected exactly 1 tag description, got ' . count($directChildren));
139 139
         }
140 140
 
141 141
         return $directChildren[0];
142 142
     }
143 143
 
144
-    private function assertTagDoesNotContainChildren( \DOMElement $targetTag ) {
145
-        if ( $targetTag->childNodes->length > 0 ) {
146
-            throw new InvalidArgumentException( 'Nested elements are not allowed' );
144
+    private function assertTagDoesNotContainChildren(\DOMElement $targetTag) {
145
+        if ($targetTag->childNodes->length > 0) {
146
+            throw new InvalidArgumentException('Nested elements are not allowed');
147 147
         }
148 148
     }
149 149
 
@@ -152,17 +152,17 @@  discard block
 block discarded – undo
152 152
      * @param $targetTag
153 153
      * @return AttributeMatcher[]
154 154
      */
155
-    private function createAttributeMatchers( $inputHtml, \DOMElement $targetTag ) {
155
+    private function createAttributeMatchers($inputHtml, \DOMElement $targetTag) {
156 156
         $attributeMatchers = [];
157 157
         /** @var \DOMAttr $attribute */
158
-        foreach ( $targetTag->attributes as $attribute ) {
159
-            if ( $attribute->name === 'class' ) {
158
+        foreach ($targetTag->attributes as $attribute) {
159
+            if ($attribute->name === 'class') {
160 160
                 continue;
161 161
             }
162 162
 
163
-            $attributeMatcher = new AttributeMatcher( IsEqual::equalTo( $attribute->name ) );
164
-            if ( !$this->isBooleanAttribute( $inputHtml, $attribute->name ) ) {
165
-                $attributeMatcher = $attributeMatcher->havingValue( IsEqual::equalTo( $attribute->value ) );
163
+            $attributeMatcher = new AttributeMatcher(IsEqual::equalTo($attribute->name));
164
+            if (!$this->isBooleanAttribute($inputHtml, $attribute->name)) {
165
+                $attributeMatcher = $attributeMatcher->havingValue(IsEqual::equalTo($attribute->value));
166 166
             }
167 167
 
168 168
             $attributeMatchers[] = $attributeMatcher;
@@ -174,23 +174,23 @@  discard block
 block discarded – undo
174 174
      * @param \DOMElement $targetTag
175 175
      * @return ClassMatcher[]
176 176
      */
177
-    private function createClassMatchers( $targetTag ) {
177
+    private function createClassMatchers($targetTag) {
178 178
         $classMatchers = [];
179
-        $classValue = $targetTag->getAttribute( 'class' );
180
-        foreach ( explode( ' ', $classValue ) as $expectedClass ) {
181
-            if ( $expectedClass === '' ) {
179
+        $classValue = $targetTag->getAttribute('class');
180
+        foreach (explode(' ', $classValue) as $expectedClass) {
181
+            if ($expectedClass === '') {
182 182
                 continue;
183 183
             }
184
-            $classMatchers[] = new ClassMatcher( IsEqual::equalTo( $expectedClass ) );
184
+            $classMatchers[] = new ClassMatcher(IsEqual::equalTo($expectedClass));
185 185
         }
186 186
         return $classMatchers;
187 187
     }
188 188
 
189
-    private function elementToString( \DOMElement $element ) {
189
+    private function elementToString(\DOMElement $element) {
190 190
         $newDocument = new \DOMDocument();
191
-        $cloned = $element->cloneNode( true );
192
-        $newDocument->appendChild( $newDocument->importNode( $cloned, true ) );
193
-        return trim( $newDocument->saveHTML() );
191
+        $cloned = $element->cloneNode(true);
192
+        $newDocument->appendChild($newDocument->importNode($cloned, true));
193
+        return trim($newDocument->saveHTML());
194 194
     }
195 195
 
196 196
 }
Please login to merge, or discard this patch.
src/RootElementMatcher.php 1 patch
Spacing   +21 added lines, -21 removed lines patch added patch discarded remove patch
@@ -18,19 +18,19 @@  discard block
 block discarded – undo
18 18
      *
19 19
      * @return static
20 20
      */
21
-    public static function havingRootElement( Matcher $tagMatcher = null ) {
22
-        return new static( $tagMatcher );
21
+    public static function havingRootElement(Matcher $tagMatcher = null) {
22
+        return new static($tagMatcher);
23 23
     }
24 24
 
25
-    public function __construct( Matcher $tagMatcher = null ) {
26
-        parent::__construct( self::TYPE_OBJECT, \DOMDocument::class );
25
+    public function __construct(Matcher $tagMatcher = null) {
26
+        parent::__construct(self::TYPE_OBJECT, \DOMDocument::class);
27 27
         $this->tagMatcher = $tagMatcher;
28 28
     }
29 29
 
30
-    public function describeTo( Description $description ) {
31
-        $description->appendText( 'having root element ' );
32
-        if ( $this->tagMatcher ) {
33
-            $description->appendDescriptionOf( $this->tagMatcher );
30
+    public function describeTo(Description $description) {
31
+        $description->appendText('having root element ');
32
+        if ($this->tagMatcher) {
33
+            $description->appendDescriptionOf($this->tagMatcher);
34 34
         }
35 35
     }
36 36
 
@@ -40,27 +40,27 @@  discard block
 block discarded – undo
40 40
      *
41 41
      * @return bool
42 42
      */
43
-    protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
44
-        $DOMNodeList = iterator_to_array( $item->documentElement->childNodes );
43
+    protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
44
+        $DOMNodeList = iterator_to_array($item->documentElement->childNodes);
45 45
 
46
-        $body = array_shift( $DOMNodeList );
47
-        $DOMNodeList = iterator_to_array( $body->childNodes );
48
-        if ( count( $DOMNodeList ) > 1 ) {
46
+        $body = array_shift($DOMNodeList);
47
+        $DOMNodeList = iterator_to_array($body->childNodes);
48
+        if (count($DOMNodeList) > 1) {
49 49
             //TODO Test this description
50
-            $mismatchDescription->appendText( 'having ' . count( $DOMNodeList ) . ' root elements ' );
50
+            $mismatchDescription->appendText('having ' . count($DOMNodeList) . ' root elements ');
51 51
             return false;
52 52
         }
53 53
 
54
-        $target = array_shift( $DOMNodeList );
55
-        if ( !$target ) {
54
+        $target = array_shift($DOMNodeList);
55
+        if (!$target) {
56 56
             //TODO Reproduce?
57
-            $mismatchDescription->appendText( 'having no root elements ' );
57
+            $mismatchDescription->appendText('having no root elements ');
58 58
             return false;
59 59
         }
60
-        if ( $this->tagMatcher ) {
61
-            $mismatchDescription->appendText( 'root element ' );
62
-            $this->tagMatcher->describeMismatch( $target, $mismatchDescription );
63
-            return $this->tagMatcher->matches( $target );
60
+        if ($this->tagMatcher) {
61
+            $mismatchDescription->appendText('root element ');
62
+            $this->tagMatcher->describeMismatch($target, $mismatchDescription);
63
+            return $this->tagMatcher->matches($target);
64 64
         }
65 65
 
66 66
         return (bool)$target;
Please login to merge, or discard this patch.