Refactor

Author: sebastianbergmann

src/Util/Xml/Loader.php

@@ -14,6 +14,7 @@
 use function libxml_get_errors;
 use function libxml_use_internal_errors;
 use function sprintf;
+use function trim;
 use DOMDocument;
 
 /**
@@ -42,25 +43,25 @@ public function loadFile(string $filename): DOMDocument
             );
         }
 
-        return $this->load($contents, $filename);
+        if (trim($contents) === '') {
+            throw new XmlException(
+                sprintf(
+                    'Could not parse XML from empty file "%s"',
+                    $filename,
+                ),
+            );
+        }
+
+        return $this->load($contents);
     }
 
     /**
      * @throws XmlException
      */
-    public function load(string $actual, ?string $filename = null): DOMDocument
+    public function load(string $actual): DOMDocument
     {
         if ($actual === '') {
-            if ($filename === null) {
-                throw new XmlException('Could not parse XML from empty string');
-            }
-
-            throw new XmlException(
-                sprintf(
-                    'Could not parse XML from empty file "%s"',
-                    $filename,
-                ),
-            );
+            throw new XmlException('Could not parse XML from empty string');
         }
 
         $document                     = new DOMDocument;
@@ -78,17 +79,7 @@ public function load(string $actual, ?string $filename = null): DOMDocument
         libxml_use_internal_errors($internal);
         error_reporting($reporting);
 
-        if ($loaded === false || $message !== '') {
-            if ($filename !== null) {
-                throw new XmlException(
-                    sprintf(
-                        'Could not load "%s"%s',
-                        $filename,
-                        $message !== '' ? ":\n" . $message : '',
-                    ),
-                );
-            }
-
+        if ($loaded === false) {
             if ($message === '') {
                 $message = 'Could not load XML for unknown reason';
             }