Ensure that library re-sets original xml entity loading value after doing what is needed

seamuslee001 · seamuslee001 · commit 8d457468d206 · 2019-02-24T14:07:36.000+11:00
diff --git a/src/Common/XMLReader.php b/src/Common/XMLReader.php
@@ -71,10 +71,11 @@ public function getDomFromZip($zipFile, $xmlFile)
      */
     public function getDomFromString($content)
     {
+        $originalLibXMLEntityValue = libxml_disable_entity_loader();
         libxml_disable_entity_loader(true);
         $this->dom = new \DOMDocument();
         $this->dom->loadXML($content);
-
+        libxml_disable_entity_loader($originalLibXMLEntityValue);
         return $this->dom;
     }
 

Original file line number	Diff line number	Diff line change
`@@ -71,10 +71,11 @@ public function getDomFromZip($zipFile, $xmlFile)`
`71`	`71`	`*/`
`72`	`72`	`public function getDomFromString($content)`
`73`	`73`	`{`
	`74`	`+ $originalLibXMLEntityValue = libxml_disable_entity_loader();`
`74`	`75`	`libxml_disable_entity_loader(true);`
`75`	`76`	`$this->dom = new \DOMDocument();`
`76`	`77`	`$this->dom->loadXML($content);`
`77`		`-`
	`78`	`+ libxml_disable_entity_loader($originalLibXMLEntityValue);`
`78`	`79`	`return $this->dom;`
`79`	`80`	`}`
`80`	`81`