wmde / hamcrest-html-matchers

src/TagMatcher.php

Indentation +3 added lines, -3 removed lines

@@ -6,8 +6,8 @@
 
 abstract class TagMatcher extends TypeSafeDiagnosingMatcher {
 
-	public function __construct() {
-		parent::__construct( self::TYPE_OBJECT, \DOMElement::class );
-	}
+    public function __construct() {
+        parent::__construct( self::TYPE_OBJECT, \DOMElement::class );
+    }
 
 }

Spacing +1 added lines, -1 removed lines

@@ -7,7 +7,7 @@
 abstract class TagMatcher extends TypeSafeDiagnosingMatcher {
 
 	public function __construct() {
-		parent::__construct( self::TYPE_OBJECT, \DOMElement::class );
+		parent::__construct(self::TYPE_OBJECT, \DOMElement::class);
 	}
 
 }

src/RootElementMatcher.php

Indentation +50 added lines, -50 removed lines

@@ -8,62 +8,62 @@
 
 class RootElementMatcher extends TypeSafeDiagnosingMatcher {
 
-	/**
-	 * @var Matcher
-	 */
-	private $tagMatcher;
+    /**
+     * @var Matcher
+     */
+    private $tagMatcher;
 
-	/**
-	 * @param Matcher|null $tagMatcher
-	 *
-	 * @return static
-	 */
-	public static function havingRootElement( Matcher $tagMatcher = null ) {
-		return new static( $tagMatcher );
-	}
+    /**
+     * @param Matcher|null $tagMatcher
+     *
+     * @return static
+     */
+    public static function havingRootElement( Matcher $tagMatcher = null ) {
+        return new static( $tagMatcher );
+    }
 
-	public function __construct( Matcher $tagMatcher = null ) {
-		parent::__construct( self::TYPE_OBJECT, \DOMDocument::class );
-		$this->tagMatcher = $tagMatcher;
-	}
+    public function __construct( Matcher $tagMatcher = null ) {
+        parent::__construct( self::TYPE_OBJECT, \DOMDocument::class );
+        $this->tagMatcher = $tagMatcher;
+    }
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'having root element ' );
-		if ( $this->tagMatcher ) {
-			$description->appendDescriptionOf( $this->tagMatcher );
-		}
-	}
+    public function describeTo( Description $description ) {
+        $description->appendText( 'having root element ' );
+        if ( $this->tagMatcher ) {
+            $description->appendDescriptionOf( $this->tagMatcher );
+        }
+    }
 
-	/**
-	 * @param \DOMDocument $item
-	 * @param Description $mismatchDescription
-	 *
-	 * @return bool
-	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		$DOMNodeList = iterator_to_array( $item->documentElement->childNodes );
+    /**
+     * @param \DOMDocument $item
+     * @param Description $mismatchDescription
+     *
+     * @return bool
+     */
+    protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
+        $DOMNodeList = iterator_to_array( $item->documentElement->childNodes );
 
-		$body = array_shift( $DOMNodeList );
-		$DOMNodeList = iterator_to_array( $body->childNodes );
-		if ( count( $DOMNodeList ) > 1 ) {
-			// TODO Test this description
-			$mismatchDescription->appendText( 'having ' . count( $DOMNodeList ) . ' root elements ' );
-			return false;
-		}
+        $body = array_shift( $DOMNodeList );
+        $DOMNodeList = iterator_to_array( $body->childNodes );
+        if ( count( $DOMNodeList ) > 1 ) {
+            // TODO Test this description
+            $mismatchDescription->appendText( 'having ' . count( $DOMNodeList ) . ' root elements ' );
+            return false;
+        }
 
-		$target = array_shift( $DOMNodeList );
-		if ( !$target ) {
-			// TODO Reproduce?
-			$mismatchDescription->appendText( 'having no root elements ' );
-			return false;
-		}
-		if ( $this->tagMatcher ) {
-			$mismatchDescription->appendText( 'root element ' );
-			$this->tagMatcher->describeMismatch( $target, $mismatchDescription );
-			return $this->tagMatcher->matches( $target );
-		}
+        $target = array_shift( $DOMNodeList );
+        if ( !$target ) {
+            // TODO Reproduce?
+            $mismatchDescription->appendText( 'having no root elements ' );
+            return false;
+        }
+        if ( $this->tagMatcher ) {
+            $mismatchDescription->appendText( 'root element ' );
+            $this->tagMatcher->describeMismatch( $target, $mismatchDescription );
+            return $this->tagMatcher->matches( $target );
+        }
 
-		return (bool)$target;
-	}
+        return (bool)$target;
+    }
 
 }

Spacing +21 added lines, -21 removed lines

@@ -18,19 +18,19 @@ 
 	 *
 	 * @return static
 	 */
-	public static function havingRootElement( Matcher $tagMatcher = null ) {
-		return new static( $tagMatcher );
+	public static function havingRootElement(Matcher $tagMatcher = null) {
+		return new static($tagMatcher);
 	}
 
-	public function __construct( Matcher $tagMatcher = null ) {
-		parent::__construct( self::TYPE_OBJECT, \DOMDocument::class );
+	public function __construct(Matcher $tagMatcher = null) {
+		parent::__construct(self::TYPE_OBJECT, \DOMDocument::class);
 		$this->tagMatcher = $tagMatcher;
 	}
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'having root element ' );
-		if ( $this->tagMatcher ) {
-			$description->appendDescriptionOf( $this->tagMatcher );
+	public function describeTo(Description $description) {
+		$description->appendText('having root element ');
+		if ($this->tagMatcher) {
+			$description->appendDescriptionOf($this->tagMatcher);
 		}
 	}
 
@@ -40,27 +40,27 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		$DOMNodeList = iterator_to_array( $item->documentElement->childNodes );
+	protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
+		$DOMNodeList = iterator_to_array($item->documentElement->childNodes);
 
-		$body = array_shift( $DOMNodeList );
-		$DOMNodeList = iterator_to_array( $body->childNodes );
-		if ( count( $DOMNodeList ) > 1 ) {
+		$body = array_shift($DOMNodeList);
+		$DOMNodeList = iterator_to_array($body->childNodes);
+		if (count($DOMNodeList) > 1) {
 			// TODO Test this description
-			$mismatchDescription->appendText( 'having ' . count( $DOMNodeList ) . ' root elements ' );
+			$mismatchDescription->appendText('having ' . count($DOMNodeList) . ' root elements ');
 			return false;
 		}
 
-		$target = array_shift( $DOMNodeList );
-		if ( !$target ) {
+		$target = array_shift($DOMNodeList);
+		if (!$target) {
 			// TODO Reproduce?
-			$mismatchDescription->appendText( 'having no root elements ' );
+			$mismatchDescription->appendText('having no root elements ');
 			return false;
 		}
-		if ( $this->tagMatcher ) {
-			$mismatchDescription->appendText( 'root element ' );
-			$this->tagMatcher->describeMismatch( $target, $mismatchDescription );
-			return $this->tagMatcher->matches( $target );
+		if ($this->tagMatcher) {
+			$mismatchDescription->appendText('root element ');
+			$this->tagMatcher->describeMismatch($target, $mismatchDescription);
+			return $this->tagMatcher->matches($target);
 		}
 
 		return (bool)$target;

src/DirectChildElementMatcher.php

Indentation +60 added lines, -60 removed lines

@@ -8,65 +8,65 @@
 
 class DirectChildElementMatcher extends TypeSafeDiagnosingMatcher {
 
-	/**
-	 * @var Matcher
-	 */
-	private $matcher;
-
-	public static function havingDirectChild( Matcher $elementMatcher = null ) {
-		return new static( $elementMatcher );
-	}
-
-	public function __construct( Matcher $matcher = null ) {
-		parent::__construct( \DOMNode::class );
-		$this->matcher = $matcher;
-	}
-
-	public function describeTo( Description $description ) {
-		$description->appendText( 'having direct child ' );
-		if ( $this->matcher ) {
-			$description->appendDescriptionOf( $this->matcher );
-		}
-	}
-
-	/**
-	 * @param \DOMDocument|\DOMNode $item
-	 * @param Description $mismatchDescription
-	 *
-	 * @return bool
-	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		if ( $item instanceof \DOMDocument ) {
-			$directChildren = iterator_to_array( $item->documentElement->childNodes );
-
-			$body = array_shift( $directChildren );
-			$directChildren = $body->childNodes;
-		} else {
-			$directChildren = $item->childNodes;
-		}
-
-		if ( $directChildren->length === 0 ) {
-			$mismatchDescription->appendText( 'with no direct children' );
-			return false;
-		}
-
-		$childWord = $directChildren->length === 1 ? 'child' : 'children';
-
-		$mismatchDescription->appendText( "with direct {$childWord} " );
-
-		if ( !$this->matcher ) {
-			return count( $directChildren ) !== 0;
-		}
-
-		foreach ( $directChildren as $child ) {
-			if ( $this->matcher && $this->matcher->matches( $child ) ) {
-				return true;
-			}
-		}
-
-		$this->matcher->describeMismatch( $child, $mismatchDescription );
-
-		return false;
-	}
+    /**
+     * @var Matcher
+     */
+    private $matcher;
+
+    public static function havingDirectChild( Matcher $elementMatcher = null ) {
+        return new static( $elementMatcher );
+    }
+
+    public function __construct( Matcher $matcher = null ) {
+        parent::__construct( \DOMNode::class );
+        $this->matcher = $matcher;
+    }
+
+    public function describeTo( Description $description ) {
+        $description->appendText( 'having direct child ' );
+        if ( $this->matcher ) {
+            $description->appendDescriptionOf( $this->matcher );
+        }
+    }
+
+    /**
+     * @param \DOMDocument|\DOMNode $item
+     * @param Description $mismatchDescription
+     *
+     * @return bool
+     */
+    protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
+        if ( $item instanceof \DOMDocument ) {
+            $directChildren = iterator_to_array( $item->documentElement->childNodes );
+
+            $body = array_shift( $directChildren );
+            $directChildren = $body->childNodes;
+        } else {
+            $directChildren = $item->childNodes;
+        }
+
+        if ( $directChildren->length === 0 ) {
+            $mismatchDescription->appendText( 'with no direct children' );
+            return false;
+        }
+
+        $childWord = $directChildren->length === 1 ? 'child' : 'children';
+
+        $mismatchDescription->appendText( "with direct {$childWord} " );
+
+        if ( !$this->matcher ) {
+            return count( $directChildren ) !== 0;
+        }
+
+        foreach ( $directChildren as $child ) {
+            if ( $this->matcher && $this->matcher->matches( $child ) ) {
+                return true;
+            }
+        }
+
+        $this->matcher->describeMismatch( $child, $mismatchDescription );
+
+        return false;
+    }
 
 }

Spacing +20 added lines, -20 removed lines

@@ -13,19 +13,19 @@ 
 	 */
 	private $matcher;
 
-	public static function havingDirectChild( Matcher $elementMatcher = null ) {
-		return new static( $elementMatcher );
+	public static function havingDirectChild(Matcher $elementMatcher = null) {
+		return new static($elementMatcher);
 	}
 
-	public function __construct( Matcher $matcher = null ) {
-		parent::__construct( \DOMNode::class );
+	public function __construct(Matcher $matcher = null) {
+		parent::__construct(\DOMNode::class);
 		$this->matcher = $matcher;
 	}
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'having direct child ' );
-		if ( $this->matcher ) {
-			$description->appendDescriptionOf( $this->matcher );
+	public function describeTo(Description $description) {
+		$description->appendText('having direct child ');
+		if ($this->matcher) {
+			$description->appendDescriptionOf($this->matcher);
 		}
 	}
 
@@ -35,36 +35,36 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		if ( $item instanceof \DOMDocument ) {
-			$directChildren = iterator_to_array( $item->documentElement->childNodes );
+	protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
+		if ($item instanceof \DOMDocument) {
+			$directChildren = iterator_to_array($item->documentElement->childNodes);
 
-			$body = array_shift( $directChildren );
+			$body = array_shift($directChildren);
 			$directChildren = $body->childNodes;
 		} else {
 			$directChildren = $item->childNodes;
 		}
 
-		if ( $directChildren->length === 0 ) {
-			$mismatchDescription->appendText( 'with no direct children' );
+		if ($directChildren->length === 0) {
+			$mismatchDescription->appendText('with no direct children');
 			return false;
 		}
 
 		$childWord = $directChildren->length === 1 ? 'child' : 'children';
 
-		$mismatchDescription->appendText( "with direct {$childWord} " );
+		$mismatchDescription->appendText("with direct {$childWord} ");
 
-		if ( !$this->matcher ) {
-			return count( $directChildren ) !== 0;
+		if (!$this->matcher) {
+			return count($directChildren) !== 0;
 		}
 
-		foreach ( $directChildren as $child ) {
-			if ( $this->matcher && $this->matcher->matches( $child ) ) {
+		foreach ($directChildren as $child) {
+			if ($this->matcher && $this->matcher->matches($child)) {
 				return true;
 			}
 		}
 
-		$this->matcher->describeMismatch( $child, $mismatchDescription );
+		$this->matcher->describeMismatch($child, $mismatchDescription);
 
 		return false;
 	}

src/XmlNodeRecursiveIterator.php

Indentation +20 added lines, -20 removed lines

@@ -4,27 +4,27 @@
 
 class XmlNodeRecursiveIterator extends \ArrayIterator {
 
-	public function __construct( \DOMNodeList $nodeList ) {
-		$queue = $this->addElementsToQueue( [], $nodeList );
-		parent::__construct( $queue );
-	}
+    public function __construct( \DOMNodeList $nodeList ) {
+        $queue = $this->addElementsToQueue( [], $nodeList );
+        parent::__construct( $queue );
+    }
 
-	/**
-	 * @param \DOMNode[] $queue
-	 * @param \DOMNodeList $nodeList
-	 *
-	 * @return \DOMNode[] New queue
-	 */
-	private function addElementsToQueue( array $queue, \DOMNodeList $nodeList ) {
-		/** @var \DOMNode $node */
-		foreach ( $nodeList as $node ) {
-			$queue[] = $node;
-			if ( $node->childNodes !== null ) {
-				$queue = $this->addElementsToQueue( $queue, $node->childNodes );
-			}
-		}
+    /**
+     * @param \DOMNode[] $queue
+     * @param \DOMNodeList $nodeList
+     *
+     * @return \DOMNode[] New queue
+     */
+    private function addElementsToQueue( array $queue, \DOMNodeList $nodeList ) {
+        /** @var \DOMNode $node */
+        foreach ( $nodeList as $node ) {
+            $queue[] = $node;
+            if ( $node->childNodes !== null ) {
+                $queue = $this->addElementsToQueue( $queue, $node->childNodes );
+            }
+        }
 
-		return $queue;
-	}
+        return $queue;
+    }
 
 }

Spacing +7 added lines, -7 removed lines

@@ -4,9 +4,9 @@ 
 
 class XmlNodeRecursiveIterator extends \ArrayIterator {
 
-	public function __construct( \DOMNodeList $nodeList ) {
-		$queue = $this->addElementsToQueue( [], $nodeList );
-		parent::__construct( $queue );
+	public function __construct(\DOMNodeList $nodeList) {
+		$queue = $this->addElementsToQueue([], $nodeList);
+		parent::__construct($queue);
 	}
 
 	/**
@@ -15,12 +15,12 @@ 
 	 *
 	 * @return \DOMNode[] New queue
 	 */
-	private function addElementsToQueue( array $queue, \DOMNodeList $nodeList ) {
+	private function addElementsToQueue(array $queue, \DOMNodeList $nodeList) {
 		/** @var \DOMNode $node */
-		foreach ( $nodeList as $node ) {
+		foreach ($nodeList as $node) {
 			$queue[] = $node;
-			if ( $node->childNodes !== null ) {
-				$queue = $this->addElementsToQueue( $queue, $node->childNodes );
+			if ($node->childNodes !== null) {
+				$queue = $this->addElementsToQueue($queue, $node->childNodes);
 			}
 		}
 

src/HtmlMatcher.php

Indentation +94 added lines, -94 removed lines

@@ -8,99 +8,99 @@
 
 class HtmlMatcher extends DiagnosingMatcher {
 
-	/**
-	 * @link http://www.xmlsoft.org/html/libxml-xmlerror.html#xmlParserErrors
-	 * @link https://github.com/Chronic-Dev/libxml2/blob/683f296a905710ff285c28b8644ef3a3d8be9486/include/libxml/xmlerror.h#L257
-	 */
-	const XML_UNKNOWN_TAG_ERROR_CODE = 801;
-
-	/**
-	 * @var Matcher
-	 */
-	private $elementMatcher;
-
-	/**
-	 * @param Matcher $elementMatcher
-	 *
-	 * @return self
-	 */
-	public static function htmlPiece( Matcher $elementMatcher = null ) {
-		return new static( $elementMatcher );
-	}
-
-	private function __construct( Matcher $elementMatcher = null ) {
-		$this->elementMatcher = $elementMatcher;
-	}
-
-	public function describeTo( Description $description ) {
-		$description->appendText( 'valid html piece ' );
-		if ( $this->elementMatcher ) {
-			$description->appendDescriptionOf( $this->elementMatcher );
-		}
-	}
-
-	protected function matchesWithDiagnosticDescription( $html, Description $mismatchDescription ) {
-		$internalErrors = libxml_use_internal_errors( true );
-		$document = new \DOMDocument();
-
-		$html = $this->escapeScriptTagContents( $html );
-
-		if ( !@$document->loadHTML( mb_convert_encoding( $html, 'HTML-ENTITIES', 'UTF-8' ) ) ) {
-			$mismatchDescription->appendText( 'there was some parsing error' );
-			return false;
-		}
-
-		$errors = libxml_get_errors();
-		libxml_clear_errors();
-		libxml_use_internal_errors( $internalErrors );
-
-		$result = true;
-		/** @var \LibXMLError $error */
-		foreach ( $errors as $error ) {
-			if ( $this->isUnknownTagError( $error ) ) {
-				continue;
-			}
-
-			$mismatchDescription->appendText( 'there was parsing error: ' )
-				->appendText( trim( $error->message ) )
-				->appendText( ' on line ' )
-				->appendText( $error->line );
-			$result = false;
-		}
-
-		if ( $result === false ) {
-			return $result;
-		}
-		$mismatchDescription->appendText( 'valid html piece ' );
-
-		if ( $this->elementMatcher ) {
-			$result = $this->elementMatcher->matches( $document );
-			$this->elementMatcher->describeMismatch( $document, $mismatchDescription );
-		}
-
-		$mismatchDescription->appendText( "\nActual html:\n" )->appendText( $html );
-
-		return $result;
-	}
-
-	/**
-	 * @param \LibXMLError $error
-	 *
-	 * @return bool
-	 */
-	private function isUnknownTagError( \LibXMLError $error ) {
-		return $error->code === self::XML_UNKNOWN_TAG_ERROR_CODE;
-	}
-
-	/**
-	 * @param string $html
-	 *
-	 * @return string HTML
-	 */
-	private function escapeScriptTagContents( $html ) {
-		return preg_replace_callback( '#(<script.*>)(.*)(</script>)#isU', function ( $matches ) {
-			return $matches[1] . str_replace( '</', '<\/', $matches[2] ) . $matches[3];
-		}, $html );
-	}
+    /**
+     * @link http://www.xmlsoft.org/html/libxml-xmlerror.html#xmlParserErrors
+     * @link https://github.com/Chronic-Dev/libxml2/blob/683f296a905710ff285c28b8644ef3a3d8be9486/include/libxml/xmlerror.h#L257
+     */
+    const XML_UNKNOWN_TAG_ERROR_CODE = 801;
+
+    /**
+     * @var Matcher
+     */
+    private $elementMatcher;
+
+    /**
+     * @param Matcher $elementMatcher
+     *
+     * @return self
+     */
+    public static function htmlPiece( Matcher $elementMatcher = null ) {
+        return new static( $elementMatcher );
+    }
+
+    private function __construct( Matcher $elementMatcher = null ) {
+        $this->elementMatcher = $elementMatcher;
+    }
+
+    public function describeTo( Description $description ) {
+        $description->appendText( 'valid html piece ' );
+        if ( $this->elementMatcher ) {
+            $description->appendDescriptionOf( $this->elementMatcher );
+        }
+    }
+
+    protected function matchesWithDiagnosticDescription( $html, Description $mismatchDescription ) {
+        $internalErrors = libxml_use_internal_errors( true );
+        $document = new \DOMDocument();
+
+        $html = $this->escapeScriptTagContents( $html );
+
+        if ( !@$document->loadHTML( mb_convert_encoding( $html, 'HTML-ENTITIES', 'UTF-8' ) ) ) {
+            $mismatchDescription->appendText( 'there was some parsing error' );
+            return false;
+        }
+
+        $errors = libxml_get_errors();
+        libxml_clear_errors();
+        libxml_use_internal_errors( $internalErrors );
+
+        $result = true;
+        /** @var \LibXMLError $error */
+        foreach ( $errors as $error ) {
+            if ( $this->isUnknownTagError( $error ) ) {
+                continue;
+            }
+
+            $mismatchDescription->appendText( 'there was parsing error: ' )
+                ->appendText( trim( $error->message ) )
+                ->appendText( ' on line ' )
+                ->appendText( $error->line );
+            $result = false;
+        }
+
+        if ( $result === false ) {
+            return $result;
+        }
+        $mismatchDescription->appendText( 'valid html piece ' );
+
+        if ( $this->elementMatcher ) {
+            $result = $this->elementMatcher->matches( $document );
+            $this->elementMatcher->describeMismatch( $document, $mismatchDescription );
+        }
+
+        $mismatchDescription->appendText( "\nActual html:\n" )->appendText( $html );
+
+        return $result;
+    }
+
+    /**
+     * @param \LibXMLError $error
+     *
+     * @return bool
+     */
+    private function isUnknownTagError( \LibXMLError $error ) {
+        return $error->code === self::XML_UNKNOWN_TAG_ERROR_CODE;
+    }
+
+    /**
+     * @param string $html
+     *
+     * @return string HTML
+     */
+    private function escapeScriptTagContents( $html ) {
+        return preg_replace_callback( '#(<script.*>)(.*)(</script>)#isU', function ( $matches ) {
+            return $matches[1] . str_replace( '</', '<\/', $matches[2] ) . $matches[3];
+        }, $html );
+    }
 
 }

Spacing +30 added lines, -30 removed lines

@@ -24,61 +24,61 @@ 
 	 *
 	 * @return self
 	 */
-	public static function htmlPiece( Matcher $elementMatcher = null ) {
-		return new static( $elementMatcher );
+	public static function htmlPiece(Matcher $elementMatcher = null) {
+		return new static($elementMatcher);
 	}
 
-	private function __construct( Matcher $elementMatcher = null ) {
+	private function __construct(Matcher $elementMatcher = null) {
 		$this->elementMatcher = $elementMatcher;
 	}
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'valid html piece ' );
-		if ( $this->elementMatcher ) {
-			$description->appendDescriptionOf( $this->elementMatcher );
+	public function describeTo(Description $description) {
+		$description->appendText('valid html piece ');
+		if ($this->elementMatcher) {
+			$description->appendDescriptionOf($this->elementMatcher);
 		}
 	}
 
-	protected function matchesWithDiagnosticDescription( $html, Description $mismatchDescription ) {
-		$internalErrors = libxml_use_internal_errors( true );
+	protected function matchesWithDiagnosticDescription($html, Description $mismatchDescription) {
+		$internalErrors = libxml_use_internal_errors(true);
 		$document = new \DOMDocument();
 
-		$html = $this->escapeScriptTagContents( $html );
+		$html = $this->escapeScriptTagContents($html);
 
-		if ( !@$document->loadHTML( mb_convert_encoding( $html, 'HTML-ENTITIES', 'UTF-8' ) ) ) {
-			$mismatchDescription->appendText( 'there was some parsing error' );
+		if (!@$document->loadHTML(mb_convert_encoding($html, 'HTML-ENTITIES', 'UTF-8'))) {
+			$mismatchDescription->appendText('there was some parsing error');
 			return false;
 		}
 
 		$errors = libxml_get_errors();
 		libxml_clear_errors();
-		libxml_use_internal_errors( $internalErrors );
+		libxml_use_internal_errors($internalErrors);
 
 		$result = true;
 		/** @var \LibXMLError $error */
-		foreach ( $errors as $error ) {
-			if ( $this->isUnknownTagError( $error ) ) {
+		foreach ($errors as $error) {
+			if ($this->isUnknownTagError($error)) {
 				continue;
 			}
 
-			$mismatchDescription->appendText( 'there was parsing error: ' )
-				->appendText( trim( $error->message ) )
-				->appendText( ' on line ' )
-				->appendText( $error->line );
+			$mismatchDescription->appendText('there was parsing error: ')
+				->appendText(trim($error->message))
+				->appendText(' on line ')
+				->appendText($error->line);
 			$result = false;
 		}
 
-		if ( $result === false ) {
+		if ($result === false) {
 			return $result;
 		}
-		$mismatchDescription->appendText( 'valid html piece ' );
+		$mismatchDescription->appendText('valid html piece ');
 
-		if ( $this->elementMatcher ) {
-			$result = $this->elementMatcher->matches( $document );
-			$this->elementMatcher->describeMismatch( $document, $mismatchDescription );
+		if ($this->elementMatcher) {
+			$result = $this->elementMatcher->matches($document);
+			$this->elementMatcher->describeMismatch($document, $mismatchDescription);
 		}
 
-		$mismatchDescription->appendText( "\nActual html:\n" )->appendText( $html );
+		$mismatchDescription->appendText("\nActual html:\n")->appendText($html);
 
 		return $result;
 	}
@@ -88,7 +88,7 @@ 
 	 *
 	 * @return bool
 	 */
-	private function isUnknownTagError( \LibXMLError $error ) {
+	private function isUnknownTagError(\LibXMLError $error) {
 		return $error->code === self::XML_UNKNOWN_TAG_ERROR_CODE;
 	}
 
@@ -97,10 +97,10 @@ 
 	 *
 	 * @return string HTML
 	 */
-	private function escapeScriptTagContents( $html ) {
-		return preg_replace_callback( '#(<script.*>)(.*)(</script>)#isU', function ( $matches ) {
-			return $matches[1] . str_replace( '</', '<\/', $matches[2] ) . $matches[3];
-		}, $html );
+	private function escapeScriptTagContents($html) {
+		return preg_replace_callback('#(<script.*>)(.*)(</script>)#isU', function($matches) {
+			return $matches[1] . str_replace('</', '<\/', $matches[2]) . $matches[3];
+		}, $html);
 	}
 
 }

src/functions.php

Indentation +58 added lines, -58 removed lines

@@ -3,87 +3,87 @@
 use Hamcrest\Matcher;
 
 if ( !function_exists( 'htmlPiece' ) ) {
-	/**
-	 * @param Matcher|null $elementMatcher
-	 *
-	 * @return \WMDE\HamcrestHtml\HtmlMatcher
-	 */
-	function htmlPiece( Matcher $elementMatcher = null ) {
-		return \WMDE\HamcrestHtml\HtmlMatcher::htmlPiece( $elementMatcher );
-	}
+    /**
+     * @param Matcher|null $elementMatcher
+     *
+     * @return \WMDE\HamcrestHtml\HtmlMatcher
+     */
+    function htmlPiece( Matcher $elementMatcher = null ) {
+        return \WMDE\HamcrestHtml\HtmlMatcher::htmlPiece( $elementMatcher );
+    }
 }
 
 if ( !function_exists( 'havingRootElement' ) ) {
-	function havingRootElement( Matcher $matcher = null ) {
-		return \WMDE\HamcrestHtml\RootElementMatcher::havingRootElement( $matcher );
-	}
+    function havingRootElement( Matcher $matcher = null ) {
+        return \WMDE\HamcrestHtml\RootElementMatcher::havingRootElement( $matcher );
+    }
 }
 
 if ( !function_exists( 'havingDirectChild' ) ) {
-	function havingDirectChild( Matcher $elementMatcher = null ) {
-		return \WMDE\HamcrestHtml\DirectChildElementMatcher::havingDirectChild( $elementMatcher );
-	}
+    function havingDirectChild( Matcher $elementMatcher = null ) {
+        return \WMDE\HamcrestHtml\DirectChildElementMatcher::havingDirectChild( $elementMatcher );
+    }
 }
 
 if ( !function_exists( 'havingChild' ) ) {
-	function havingChild( Matcher $elementMatcher = null ) {
-		return \WMDE\HamcrestHtml\ChildElementMatcher::havingChild( $elementMatcher );
-	}
+    function havingChild( Matcher $elementMatcher = null ) {
+        return \WMDE\HamcrestHtml\ChildElementMatcher::havingChild( $elementMatcher );
+    }
 }
 
 if ( !function_exists( 'withTagName' ) ) {
-	/**
-	 * @param Matcher|string $tagName
-	 *
-	 * @return \WMDE\HamcrestHtml\TagNameMatcher
-	 */
-	function withTagName( $tagName ) {
-		return \WMDE\HamcrestHtml\TagNameMatcher::withTagName( $tagName );
-	}
+    /**
+     * @param Matcher|string $tagName
+     *
+     * @return \WMDE\HamcrestHtml\TagNameMatcher
+     */
+    function withTagName( $tagName ) {
+        return \WMDE\HamcrestHtml\TagNameMatcher::withTagName( $tagName );
+    }
 }
 
 if ( !function_exists( 'withAttribute' ) ) {
-	/**
-	 * @param Matcher|string $attributeName
-	 *
-	 * @return \WMDE\HamcrestHtml\AttributeMatcher
-	 */
-	function withAttribute( $attributeName ) {
-		return \WMDE\HamcrestHtml\AttributeMatcher::withAttribute( $attributeName );
-	}
+    /**
+     * @param Matcher|string $attributeName
+     *
+     * @return \WMDE\HamcrestHtml\AttributeMatcher
+     */
+    function withAttribute( $attributeName ) {
+        return \WMDE\HamcrestHtml\AttributeMatcher::withAttribute( $attributeName );
+    }
 }
 
 if ( !function_exists( 'withClass' ) ) {
-	/**
-	 * @param Matcher|string $class
-	 *
-	 * @return \WMDE\HamcrestHtml\ClassMatcher
-	 */
-	function withClass( $class ) {
-		// TODO don't allow to call with empty string
+    /**
+     * @param Matcher|string $class
+     *
+     * @return \WMDE\HamcrestHtml\ClassMatcher
+     */
+    function withClass( $class ) {
+        // TODO don't allow to call with empty string
 
-		return \WMDE\HamcrestHtml\ClassMatcher::withClass( $class );
-	}
+        return \WMDE\HamcrestHtml\ClassMatcher::withClass( $class );
+    }
 }
 
 if ( !function_exists( 'havingTextContents' ) ) {
-	/**
-	 * @param Matcher|string $text
-	 *
-	 * @return \WMDE\HamcrestHtml\TextContentsMatcher
-	 */
-	function havingTextContents( $text ) {
-		return \WMDE\HamcrestHtml\TextContentsMatcher::havingTextContents( $text );
-	}
+    /**
+     * @param Matcher|string $text
+     *
+     * @return \WMDE\HamcrestHtml\TextContentsMatcher
+     */
+    function havingTextContents( $text ) {
+        return \WMDE\HamcrestHtml\TextContentsMatcher::havingTextContents( $text );
+    }
 }
 
 if ( !function_exists( 'tagMatchingOutline' ) ) {
-	/**
-	 * @param string $htmlOutline
-	 *
-	 * @return \WMDE\HamcrestHtml\ComplexTagMatcher
-	 */
-	function tagMatchingOutline( $htmlOutline ) {
-		return \WMDE\HamcrestHtml\ComplexTagMatcher::tagMatchingOutline( $htmlOutline );
-	}
+    /**
+     * @param string $htmlOutline
+     *
+     * @return \WMDE\HamcrestHtml\ComplexTagMatcher
+     */
+    function tagMatchingOutline( $htmlOutline ) {
+        return \WMDE\HamcrestHtml\ComplexTagMatcher::tagMatchingOutline( $htmlOutline );
+    }
 }

Spacing +27 added lines, -27 removed lines

@@ -2,88 +2,88 @@
 
 use Hamcrest\Matcher;
 
-if ( !function_exists( 'htmlPiece' ) ) {
+if (!function_exists('htmlPiece')) {
 	/**
 	 * @param Matcher|null $elementMatcher
 	 *
 	 * @return \WMDE\HamcrestHtml\HtmlMatcher
 	 */
-	function htmlPiece( Matcher $elementMatcher = null ) {
-		return \WMDE\HamcrestHtml\HtmlMatcher::htmlPiece( $elementMatcher );
+	function htmlPiece(Matcher $elementMatcher = null) {
+		return \WMDE\HamcrestHtml\HtmlMatcher::htmlPiece($elementMatcher);
 	}
 }
 
-if ( !function_exists( 'havingRootElement' ) ) {
-	function havingRootElement( Matcher $matcher = null ) {
-		return \WMDE\HamcrestHtml\RootElementMatcher::havingRootElement( $matcher );
+if (!function_exists('havingRootElement')) {
+	function havingRootElement(Matcher $matcher = null) {
+		return \WMDE\HamcrestHtml\RootElementMatcher::havingRootElement($matcher);
 	}
 }
 
-if ( !function_exists( 'havingDirectChild' ) ) {
-	function havingDirectChild( Matcher $elementMatcher = null ) {
-		return \WMDE\HamcrestHtml\DirectChildElementMatcher::havingDirectChild( $elementMatcher );
+if (!function_exists('havingDirectChild')) {
+	function havingDirectChild(Matcher $elementMatcher = null) {
+		return \WMDE\HamcrestHtml\DirectChildElementMatcher::havingDirectChild($elementMatcher);
 	}
 }
 
-if ( !function_exists( 'havingChild' ) ) {
-	function havingChild( Matcher $elementMatcher = null ) {
-		return \WMDE\HamcrestHtml\ChildElementMatcher::havingChild( $elementMatcher );
+if (!function_exists('havingChild')) {
+	function havingChild(Matcher $elementMatcher = null) {
+		return \WMDE\HamcrestHtml\ChildElementMatcher::havingChild($elementMatcher);
 	}
 }
 
-if ( !function_exists( 'withTagName' ) ) {
+if (!function_exists('withTagName')) {
 	/**
 	 * @param Matcher|string $tagName
 	 *
 	 * @return \WMDE\HamcrestHtml\TagNameMatcher
 	 */
-	function withTagName( $tagName ) {
-		return \WMDE\HamcrestHtml\TagNameMatcher::withTagName( $tagName );
+	function withTagName($tagName) {
+		return \WMDE\HamcrestHtml\TagNameMatcher::withTagName($tagName);
 	}
 }
 
-if ( !function_exists( 'withAttribute' ) ) {
+if (!function_exists('withAttribute')) {
 	/**
 	 * @param Matcher|string $attributeName
 	 *
 	 * @return \WMDE\HamcrestHtml\AttributeMatcher
 	 */
-	function withAttribute( $attributeName ) {
-		return \WMDE\HamcrestHtml\AttributeMatcher::withAttribute( $attributeName );
+	function withAttribute($attributeName) {
+		return \WMDE\HamcrestHtml\AttributeMatcher::withAttribute($attributeName);
 	}
 }
 
-if ( !function_exists( 'withClass' ) ) {
+if (!function_exists('withClass')) {
 	/**
 	 * @param Matcher|string $class
 	 *
 	 * @return \WMDE\HamcrestHtml\ClassMatcher
 	 */
-	function withClass( $class ) {
+	function withClass($class) {
 		// TODO don't allow to call with empty string
 
-		return \WMDE\HamcrestHtml\ClassMatcher::withClass( $class );
+		return \WMDE\HamcrestHtml\ClassMatcher::withClass($class);
 	}
 }
 
-if ( !function_exists( 'havingTextContents' ) ) {
+if (!function_exists('havingTextContents')) {
 	/**
 	 * @param Matcher|string $text
 	 *
 	 * @return \WMDE\HamcrestHtml\TextContentsMatcher
 	 */
-	function havingTextContents( $text ) {
-		return \WMDE\HamcrestHtml\TextContentsMatcher::havingTextContents( $text );
+	function havingTextContents($text) {
+		return \WMDE\HamcrestHtml\TextContentsMatcher::havingTextContents($text);
 	}
 }
 
-if ( !function_exists( 'tagMatchingOutline' ) ) {
+if (!function_exists('tagMatchingOutline')) {
 	/**
 	 * @param string $htmlOutline
 	 *
 	 * @return \WMDE\HamcrestHtml\ComplexTagMatcher
 	 */
-	function tagMatchingOutline( $htmlOutline ) {
-		return \WMDE\HamcrestHtml\ComplexTagMatcher::tagMatchingOutline( $htmlOutline );
+	function tagMatchingOutline($htmlOutline) {
+		return \WMDE\HamcrestHtml\ComplexTagMatcher::tagMatchingOutline($htmlOutline);
 	}
 }

src/ChildElementMatcher.php

Doc Comments +1 added lines, -1 removed lines

@@ -14,7 +14,7 @@
 	private $matcher;
 
 	/**
-	 * @param Matcher|string|null $elementMatcher
+	 * @param null|Matcher $elementMatcher
 	 *
 	 * @return self
 	 */

Indentation +62 added lines, -62 removed lines

@@ -8,75 +8,75 @@
 
 class ChildElementMatcher extends TypeSafeDiagnosingMatcher {
 
-	/**
-	 * @var Matcher|string|null
-	 */
-	private $matcher;
+    /**
+     * @var Matcher|string|null
+     */
+    private $matcher;
 
-	/**
-	 * @param Matcher|string|null $elementMatcher
-	 *
-	 * @return self
-	 */
-	public static function havingChild( $elementMatcher = null ) {
-		return new static( $elementMatcher );
-	}
+    /**
+     * @param Matcher|string|null $elementMatcher
+     *
+     * @return self
+     */
+    public static function havingChild( $elementMatcher = null ) {
+        return new static( $elementMatcher );
+    }
 
-	/**
-	 * @param Matcher|string|null $matcher
-	 */
-	public function __construct( $matcher = null ) {
-		parent::__construct( \DOMNode::class );
-		$this->matcher = $matcher;
-	}
+    /**
+     * @param Matcher|string|null $matcher
+     */
+    public function __construct( $matcher = null ) {
+        parent::__construct( \DOMNode::class );
+        $this->matcher = $matcher;
+    }
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'having child ' );
-		if ( $this->matcher ) {
-			if ( $this->matcher instanceof Matcher ) {
-				$description->appendDescriptionOf( $this->matcher );
-			} else {
-				$description->appendValue( $this->matcher );
-			}
-		}
-	}
+    public function describeTo( Description $description ) {
+        $description->appendText( 'having child ' );
+        if ( $this->matcher ) {
+            if ( $this->matcher instanceof Matcher ) {
+                $description->appendDescriptionOf( $this->matcher );
+            } else {
+                $description->appendValue( $this->matcher );
+            }
+        }
+    }
 
-	/**
-	 * @param \DOMDocument|\DOMNode $item
-	 * @param Description $mismatchDescription
-	 *
-	 * @return bool
-	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		if ( $item instanceof \DOMDocument ) {
-			$item = $item->documentElement->childNodes->item( 0 );
-			/** @var \DOMElement $item */
-		}
-		$directChildren = $item->childNodes;
+    /**
+     * @param \DOMDocument|\DOMNode $item
+     * @param Description $mismatchDescription
+     *
+     * @return bool
+     */
+    protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
+        if ( $item instanceof \DOMDocument ) {
+            $item = $item->documentElement->childNodes->item( 0 );
+            /** @var \DOMElement $item */
+        }
+        $directChildren = $item->childNodes;
 
-		if ( $directChildren->length === 0 ) {
-			$mismatchDescription->appendText( 'having no children' );
-			return false;
-		}
+        if ( $directChildren->length === 0 ) {
+            $mismatchDescription->appendText( 'having no children' );
+            return false;
+        }
 
-		if ( !$this->matcher ) {
-			return true;
-		}
+        if ( !$this->matcher ) {
+            return true;
+        }
 
-		if ( $this->matcher instanceof Matcher ) {
-			foreach ( new XmlNodeRecursiveIterator( $directChildren ) as $child ) {
-				if ( $this->matcher->matches( $child ) ) {
-					return true;
-				}
-			}
-		} else {
-			if ( $item->getElementsByTagName( $this->matcher )->length !== 0 ) {
-				return true;
-			}
-		}
+        if ( $this->matcher instanceof Matcher ) {
+            foreach ( new XmlNodeRecursiveIterator( $directChildren ) as $child ) {
+                if ( $this->matcher->matches( $child ) ) {
+                    return true;
+                }
+            }
+        } else {
+            if ( $item->getElementsByTagName( $this->matcher )->length !== 0 ) {
+                return true;
+            }
+        }
 
-		$mismatchDescription->appendText( 'having no children ' )->appendDescriptionOf( $this->matcher );
-		return false;
-	}
+        $mismatchDescription->appendText( 'having no children ' )->appendDescriptionOf( $this->matcher );
+        return false;
+    }
 
 }

Spacing +21 added lines, -21 removed lines

@@ -18,25 +18,25 @@ 
 	 *
 	 * @return self
 	 */
-	public static function havingChild( $elementMatcher = null ) {
-		return new static( $elementMatcher );
+	public static function havingChild($elementMatcher = null) {
+		return new static($elementMatcher);
 	}
 
 	/**
 	 * @param Matcher|string|null $matcher
 	 */
-	public function __construct( $matcher = null ) {
-		parent::__construct( \DOMNode::class );
+	public function __construct($matcher = null) {
+		parent::__construct(\DOMNode::class);
 		$this->matcher = $matcher;
 	}
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'having child ' );
-		if ( $this->matcher ) {
-			if ( $this->matcher instanceof Matcher ) {
-				$description->appendDescriptionOf( $this->matcher );
+	public function describeTo(Description $description) {
+		$description->appendText('having child ');
+		if ($this->matcher) {
+			if ($this->matcher instanceof Matcher) {
+				$description->appendDescriptionOf($this->matcher);
 			} else {
-				$description->appendValue( $this->matcher );
+				$description->appendValue($this->matcher);
 			}
 		}
 	}
@@ -47,35 +47,35 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		if ( $item instanceof \DOMDocument ) {
-			$item = $item->documentElement->childNodes->item( 0 );
+	protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
+		if ($item instanceof \DOMDocument) {
+			$item = $item->documentElement->childNodes->item(0);
 			/** @var \DOMElement $item */
 		}
 		$directChildren = $item->childNodes;
 
-		if ( $directChildren->length === 0 ) {
-			$mismatchDescription->appendText( 'having no children' );
+		if ($directChildren->length === 0) {
+			$mismatchDescription->appendText('having no children');
 			return false;
 		}
 
-		if ( !$this->matcher ) {
+		if (!$this->matcher) {
 			return true;
 		}
 
-		if ( $this->matcher instanceof Matcher ) {
-			foreach ( new XmlNodeRecursiveIterator( $directChildren ) as $child ) {
-				if ( $this->matcher->matches( $child ) ) {
+		if ($this->matcher instanceof Matcher) {
+			foreach (new XmlNodeRecursiveIterator($directChildren) as $child) {
+				if ($this->matcher->matches($child)) {
 					return true;
 				}
 			}
 		} else {
-			if ( $item->getElementsByTagName( $this->matcher )->length !== 0 ) {
+			if ($item->getElementsByTagName($this->matcher)->length !== 0) {
 				return true;
 			}
 		}
 
-		$mismatchDescription->appendText( 'having no children ' )->appendDescriptionOf( $this->matcher );
+		$mismatchDescription->appendText('having no children ')->appendDescriptionOf($this->matcher);
 		return false;
 	}
 

src/ClassMatcher.php

Indentation +52 added lines, -52 removed lines

@@ -7,57 +7,57 @@
 
 class ClassMatcher extends TagMatcher {
 
-	/**
-	 * @var Matcher|string
-	 */
-	private $classMatcher;
-
-	/**
-	 * @param Matcher|string $class
-	 *
-	 * @return self
-	 */
-	public static function withClass( $class ) {
-		return new static( $class );
-	}
-
-	/**
-	 * @param Matcher|string $class
-	 */
-	public function __construct( $class ) {
-		parent::__construct();
-		$this->classMatcher = $class;
-	}
-
-	public function describeTo( Description $description ) {
-		$description->appendText( 'with class ' );
-		if ( $this->classMatcher instanceof Matcher ) {
-			$description->appendDescriptionOf( $this->classMatcher );
-		} else {
-			$description->appendValue( $this->classMatcher );
-		}
-	}
-
-	/**
-	 * @param \DOMElement $item
-	 * @param Description $mismatchDescription
-	 *
-	 * @return bool
-	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		$classAttribute = $item->getAttribute( 'class' );
-
-		if ( $this->classMatcher instanceof Matcher ) {
-			$classes = preg_split( '/\s+/u', $classAttribute );
-			foreach ( $classes as $class ) {
-				if ( $this->classMatcher->matches( $class ) ) {
-					return true;
-				}
-			}
-			return false;
-		} else {
-			return (bool)preg_match( '/(^|\s)' . preg_quote( $this->classMatcher, '/' ) . '(\s|$)/u', $classAttribute );
-		}
-	}
+    /**
+     * @var Matcher|string
+     */
+    private $classMatcher;
+
+    /**
+     * @param Matcher|string $class
+     *
+     * @return self
+     */
+    public static function withClass( $class ) {
+        return new static( $class );
+    }
+
+    /**
+     * @param Matcher|string $class
+     */
+    public function __construct( $class ) {
+        parent::__construct();
+        $this->classMatcher = $class;
+    }
+
+    public function describeTo( Description $description ) {
+        $description->appendText( 'with class ' );
+        if ( $this->classMatcher instanceof Matcher ) {
+            $description->appendDescriptionOf( $this->classMatcher );
+        } else {
+            $description->appendValue( $this->classMatcher );
+        }
+    }
+
+    /**
+     * @param \DOMElement $item
+     * @param Description $mismatchDescription
+     *
+     * @return bool
+     */
+    protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
+        $classAttribute = $item->getAttribute( 'class' );
+
+        if ( $this->classMatcher instanceof Matcher ) {
+            $classes = preg_split( '/\s+/u', $classAttribute );
+            foreach ( $classes as $class ) {
+                if ( $this->classMatcher->matches( $class ) ) {
+                    return true;
+                }
+            }
+            return false;
+        } else {
+            return (bool)preg_match( '/(^|\s)' . preg_quote( $this->classMatcher, '/' ) . '(\s|$)/u', $classAttribute );
+        }
+    }
 
 }

Spacing +15 added lines, -15 removed lines

@@ -17,24 +17,24 @@ 
 	 *
 	 * @return self
 	 */
-	public static function withClass( $class ) {
-		return new static( $class );
+	public static function withClass($class) {
+		return new static($class);
 	}
 
 	/**
 	 * @param Matcher|string $class
 	 */
-	public function __construct( $class ) {
+	public function __construct($class) {
 		parent::__construct();
 		$this->classMatcher = $class;
 	}
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'with class ' );
-		if ( $this->classMatcher instanceof Matcher ) {
-			$description->appendDescriptionOf( $this->classMatcher );
+	public function describeTo(Description $description) {
+		$description->appendText('with class ');
+		if ($this->classMatcher instanceof Matcher) {
+			$description->appendDescriptionOf($this->classMatcher);
 		} else {
-			$description->appendValue( $this->classMatcher );
+			$description->appendValue($this->classMatcher);
 		}
 	}
 
@@ -44,19 +44,19 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		$classAttribute = $item->getAttribute( 'class' );
+	protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
+		$classAttribute = $item->getAttribute('class');
 
-		if ( $this->classMatcher instanceof Matcher ) {
-			$classes = preg_split( '/\s+/u', $classAttribute );
-			foreach ( $classes as $class ) {
-				if ( $this->classMatcher->matches( $class ) ) {
+		if ($this->classMatcher instanceof Matcher) {
+			$classes = preg_split('/\s+/u', $classAttribute);
+			foreach ($classes as $class) {
+				if ($this->classMatcher->matches($class)) {
 					return true;
 				}
 			}
 			return false;
 		} else {
-			return (bool)preg_match( '/(^|\s)' . preg_quote( $this->classMatcher, '/' ) . '(\s|$)/u', $classAttribute );
+			return (bool)preg_match('/(^|\s)' . preg_quote($this->classMatcher, '/') . '(\s|$)/u', $classAttribute);
 		}
 	}
 

src/AttributeMatcher.php

Indentation +100 added lines, -100 removed lines

@@ -7,114 +7,114 @@
 
 class AttributeMatcher extends TagMatcher {
 
-	/**
-	 * @var Matcher|string
-	 */
-	private $attributeNameMatcher;
+    /**
+     * @var Matcher|string
+     */
+    private $attributeNameMatcher;
 
-	/**
-	 * @var Matcher|string|null
-	 */
-	private $valueMatcher;
+    /**
+     * @var Matcher|string|null
+     */
+    private $valueMatcher;
 
-	/**
-	 * @param Matcher|string $attributeName
-	 *
-	 * @return self
-	 */
-	public static function withAttribute( $attributeName ) {
-		return new static( $attributeName );
-	}
+    /**
+     * @param Matcher|string $attributeName
+     *
+     * @return self
+     */
+    public static function withAttribute( $attributeName ) {
+        return new static( $attributeName );
+    }
 
-	/**
-	 * @param Matcher|string $attributeNameMatcher
-	 */
-	public function __construct( $attributeNameMatcher ) {
-		parent::__construct();
+    /**
+     * @param Matcher|string $attributeNameMatcher
+     */
+    public function __construct( $attributeNameMatcher ) {
+        parent::__construct();
 
-		$this->attributeNameMatcher = $attributeNameMatcher;
-	}
+        $this->attributeNameMatcher = $attributeNameMatcher;
+    }
 
-	/**
-	 * @param Matcher|string $value
-	 *
-	 * @return AttributeMatcher
-	 */
-	public function havingValue( $value ) {
-		// TODO: Throw exception if value is set
-		$result = clone $this;
-		$result->valueMatcher = $value;
+    /**
+     * @param Matcher|string $value
+     *
+     * @return AttributeMatcher
+     */
+    public function havingValue( $value ) {
+        // TODO: Throw exception if value is set
+        $result = clone $this;
+        $result->valueMatcher = $value;
 
-		return $result;
-	}
+        return $result;
+    }
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'with attribute ' );
-		if ( $this->attributeNameMatcher instanceof Matcher ) {
-			$description->appendDescriptionOf( $this->attributeNameMatcher );
-		} else {
-			$description->appendValue( $this->attributeNameMatcher );
-		}
-		if ( $this->valueMatcher !== null ) {
-			$description->appendText( ' having value ' );
-			if ( $this->attributeNameMatcher instanceof Matcher ) {
-				$description->appendDescriptionOf( $this->valueMatcher );
-			} else {
-				$description->appendValue( $this->valueMatcher );
-			}
-		}
-	}
+    public function describeTo( Description $description ) {
+        $description->appendText( 'with attribute ' );
+        if ( $this->attributeNameMatcher instanceof Matcher ) {
+            $description->appendDescriptionOf( $this->attributeNameMatcher );
+        } else {
+            $description->appendValue( $this->attributeNameMatcher );
+        }
+        if ( $this->valueMatcher !== null ) {
+            $description->appendText( ' having value ' );
+            if ( $this->attributeNameMatcher instanceof Matcher ) {
+                $description->appendDescriptionOf( $this->valueMatcher );
+            } else {
+                $description->appendValue( $this->valueMatcher );
+            }
+        }
+    }
 
-	/**
-	 * @param \DOMElement $item
-	 * @param Description $mismatchDescription
-	 *
-	 * @return bool
-	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		if ( $this->valueMatcher === null ) {
-			if ( $this->attributeNameMatcher instanceof Matcher ) {
-				/** @var \DOMAttr $attribute */
-				foreach ( $item->attributes as $attribute ) {
-					if ( $this->attributeNameMatcher->matches( $attribute->name ) ) {
-						return true;
-					}
-				}
-			} else {
-				return $item->hasAttribute( $this->attributeNameMatcher );
-			}
-		} else {
-			if ( $this->attributeNameMatcher instanceof Matcher ) {
-				if ( $this->valueMatcher instanceof Matcher ) {
-					/** @var \DOMAttr $attribute */
-					foreach ( $item->attributes as $attribute ) {
-						if ( $this->attributeNameMatcher->matches( $attribute->name )
-							&& $this->valueMatcher->matches( $attribute->value )
-						) {
-							return true;
-						}
-					}
-				} else {
-					/** @var \DOMAttr $attribute */
-					foreach ( $item->attributes as $attribute ) {
-						if ( $this->attributeNameMatcher->matches( $attribute->name )
-							&& $attribute->value === $this->valueMatcher
-						) {
-							return true;
-						}
-					}
-				}
-			} else {
-				if ( $this->valueMatcher instanceof Matcher ) {
-					return $item->hasAttribute( $this->attributeNameMatcher )
-						&& $this->valueMatcher->matches( $item->getAttribute( $this->attributeNameMatcher ) );
-				} else {
-					return $item->getAttribute( $this->attributeNameMatcher ) === $this->valueMatcher;
-				}
-			}
-		}
+    /**
+     * @param \DOMElement $item
+     * @param Description $mismatchDescription
+     *
+     * @return bool
+     */
+    protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
+        if ( $this->valueMatcher === null ) {
+            if ( $this->attributeNameMatcher instanceof Matcher ) {
+                /** @var \DOMAttr $attribute */
+                foreach ( $item->attributes as $attribute ) {
+                    if ( $this->attributeNameMatcher->matches( $attribute->name ) ) {
+                        return true;
+                    }
+                }
+            } else {
+                return $item->hasAttribute( $this->attributeNameMatcher );
+            }
+        } else {
+            if ( $this->attributeNameMatcher instanceof Matcher ) {
+                if ( $this->valueMatcher instanceof Matcher ) {
+                    /** @var \DOMAttr $attribute */
+                    foreach ( $item->attributes as $attribute ) {
+                        if ( $this->attributeNameMatcher->matches( $attribute->name )
+                            && $this->valueMatcher->matches( $attribute->value )
+                        ) {
+                            return true;
+                        }
+                    }
+                } else {
+                    /** @var \DOMAttr $attribute */
+                    foreach ( $item->attributes as $attribute ) {
+                        if ( $this->attributeNameMatcher->matches( $attribute->name )
+                            && $attribute->value === $this->valueMatcher
+                        ) {
+                            return true;
+                        }
+                    }
+                }
+            } else {
+                if ( $this->valueMatcher instanceof Matcher ) {
+                    return $item->hasAttribute( $this->attributeNameMatcher )
+                        && $this->valueMatcher->matches( $item->getAttribute( $this->attributeNameMatcher ) );
+                } else {
+                    return $item->getAttribute( $this->attributeNameMatcher ) === $this->valueMatcher;
+                }
+            }
+        }
 
-		return false;
-	}
+        return false;
+    }
 
 }

Spacing +31 added lines, -31 removed lines

@@ -22,14 +22,14 @@ 
 	 *
 	 * @return self
 	 */
-	public static function withAttribute( $attributeName ) {
-		return new static( $attributeName );
+	public static function withAttribute($attributeName) {
+		return new static($attributeName);
 	}
 
 	/**
 	 * @param Matcher|string $attributeNameMatcher
 	 */
-	public function __construct( $attributeNameMatcher ) {
+	public function __construct($attributeNameMatcher) {
 		parent::__construct();
 
 		$this->attributeNameMatcher = $attributeNameMatcher;
@@ -40,7 +40,7 @@ 
 	 *
 	 * @return AttributeMatcher
 	 */
-	public function havingValue( $value ) {
+	public function havingValue($value) {
 		// TODO: Throw exception if value is set
 		$result = clone $this;
 		$result->valueMatcher = $value;
@@ -48,19 +48,19 @@ 
 		return $result;
 	}
 
-	public function describeTo( Description $description ) {
-		$description->appendText( 'with attribute ' );
-		if ( $this->attributeNameMatcher instanceof Matcher ) {
-			$description->appendDescriptionOf( $this->attributeNameMatcher );
+	public function describeTo(Description $description) {
+		$description->appendText('with attribute ');
+		if ($this->attributeNameMatcher instanceof Matcher) {
+			$description->appendDescriptionOf($this->attributeNameMatcher);
 		} else {
-			$description->appendValue( $this->attributeNameMatcher );
+			$description->appendValue($this->attributeNameMatcher);
 		}
-		if ( $this->valueMatcher !== null ) {
-			$description->appendText( ' having value ' );
-			if ( $this->attributeNameMatcher instanceof Matcher ) {
-				$description->appendDescriptionOf( $this->valueMatcher );
+		if ($this->valueMatcher !== null) {
+			$description->appendText(' having value ');
+			if ($this->attributeNameMatcher instanceof Matcher) {
+				$description->appendDescriptionOf($this->valueMatcher);
 			} else {
-				$description->appendValue( $this->valueMatcher );
+				$description->appendValue($this->valueMatcher);
 			}
 		}
 	}
@@ -71,33 +71,33 @@ 
 	 *
 	 * @return bool
 	 */
-	protected function matchesSafelyWithDiagnosticDescription( $item, Description $mismatchDescription ) {
-		if ( $this->valueMatcher === null ) {
-			if ( $this->attributeNameMatcher instanceof Matcher ) {
+	protected function matchesSafelyWithDiagnosticDescription($item, Description $mismatchDescription) {
+		if ($this->valueMatcher === null) {
+			if ($this->attributeNameMatcher instanceof Matcher) {
 				/** @var \DOMAttr $attribute */
-				foreach ( $item->attributes as $attribute ) {
-					if ( $this->attributeNameMatcher->matches( $attribute->name ) ) {
+				foreach ($item->attributes as $attribute) {
+					if ($this->attributeNameMatcher->matches($attribute->name)) {
 						return true;
 					}
 				}
 			} else {
-				return $item->hasAttribute( $this->attributeNameMatcher );
+				return $item->hasAttribute($this->attributeNameMatcher);
 			}
 		} else {
-			if ( $this->attributeNameMatcher instanceof Matcher ) {
-				if ( $this->valueMatcher instanceof Matcher ) {
+			if ($this->attributeNameMatcher instanceof Matcher) {
+				if ($this->valueMatcher instanceof Matcher) {
 					/** @var \DOMAttr $attribute */
-					foreach ( $item->attributes as $attribute ) {
-						if ( $this->attributeNameMatcher->matches( $attribute->name )
-							&& $this->valueMatcher->matches( $attribute->value )
+					foreach ($item->attributes as $attribute) {
+						if ($this->attributeNameMatcher->matches($attribute->name)
+							&& $this->valueMatcher->matches($attribute->value)
 						) {
 							return true;
 						}
 					}
 				} else {
 					/** @var \DOMAttr $attribute */
-					foreach ( $item->attributes as $attribute ) {
-						if ( $this->attributeNameMatcher->matches( $attribute->name )
+					foreach ($item->attributes as $attribute) {
+						if ($this->attributeNameMatcher->matches($attribute->name)
 							&& $attribute->value === $this->valueMatcher
 						) {
 							return true;
@@ -105,11 +105,11 @@ 
 					}
 				}
 			} else {
-				if ( $this->valueMatcher instanceof Matcher ) {
-					return $item->hasAttribute( $this->attributeNameMatcher )
-						&& $this->valueMatcher->matches( $item->getAttribute( $this->attributeNameMatcher ) );
+				if ($this->valueMatcher instanceof Matcher) {
+					return $item->hasAttribute($this->attributeNameMatcher)
+						&& $this->valueMatcher->matches($item->getAttribute($this->attributeNameMatcher));
 				} else {
-					return $item->getAttribute( $this->attributeNameMatcher ) === $this->valueMatcher;
+					return $item->getAttribute($this->attributeNameMatcher) === $this->valueMatcher;
 				}
 			}
 		}