From a9a7da440d023574963432b9b7951204de875794 Mon Sep 17 00:00:00 2001
From: Sebastian Bergmann <sb@sebastian-bergmann.de>
Date: Fri, 28 Sep 2018 11:51:09 +0800
Subject: [PATCH] Cleanup

---
 src/Framework/Assert.php | 14 ++++++--------
 src/Util/Xml.php         |  7 +++++++
 2 files changed, 13 insertions(+), 8 deletions(-)

diff --git a/src/Framework/Assert.php b/src/Framework/Assert.php
index a97fc95678b..0f0cfa155fa 100644
--- a/src/Framework/Assert.php
+++ b/src/Framework/Assert.php
@@ -1774,13 +1774,8 @@ public static function assertXmlStringNotEqualsXmlString($expectedXml, $actualXm
      */
     public static function assertEqualXMLStructure(DOMElement $expectedElement, DOMElement $actualElement, bool $checkAttributes = false, string $message = ''): void
     {
-        $tmp             = new DOMDocument;
-        $expectedElement = $tmp->importNode($expectedElement, true);
-
-        $tmp           = new DOMDocument;
-        $actualElement = $tmp->importNode($actualElement, true);
-
-        unset($tmp);
+        $expectedElement = Xml::import($expectedElement);
+        $actualElement   = Xml::import($actualElement);
 
         static::assertSame(
             $expectedElement->tagName,
@@ -1801,8 +1796,11 @@ public static function assertEqualXMLStructure(DOMElement $expectedElement, DOME
             );
 
             for ($i = 0; $i < $expectedElement->attributes->length; $i++) {
+                /** @var \DOMAttr $expectedAttribute */
                 $expectedAttribute = $expectedElement->attributes->item($i);
-                $actualAttribute   = $actualElement->attributes->getNamedItem(
+
+                /** @var \DOMAttr $actualAttribute */
+                $actualAttribute = $actualElement->attributes->getNamedItem(
                     $expectedAttribute->name
                 );
 
diff --git a/src/Util/Xml.php b/src/Util/Xml.php
index f807b0ca237..4aa8d0787fc 100644
--- a/src/Util/Xml.php
+++ b/src/Util/Xml.php
@@ -19,6 +19,13 @@
 
 final class Xml
 {
+    public static function import(DOMElement $element): DOMElement
+    {
+        $document = new DOMDocument;
+
+        return $document->importNode($element, true);
+    }
+
     /**
      * Load an $actual document into a DOMDocument.  This is called
      * from the selector assertions.