Restore previous state for libxml option

greg0ire · fabpot · commit dde74aa8c936 · 2019-04-06T17:26:32.000+02:00
Whenever libxml_use_internal_errors() is called, the previous value for it should be restored. This used to be the case in this piece of code, but it was wrongly removed in e53bf58 , which has the nasty side effect of making the Validator component test suite break with this message: Validation failed: No DTD found!
diff --git a/src/Symfony/Component/Translation/Command/XliffLintCommand.php b/src/Symfony/Component/Translation/Command/XliffLintCommand.php
@@ -115,7 +115,7 @@ private function validate($content, $file = null)
             return ['file' => $file, 'valid' => true];
         }
 
-        libxml_use_internal_errors(true);
+        $internal = libxml_use_internal_errors(true);
 
         $document = new \DOMDocument();
         $document->loadXML($content);
@@ -143,6 +143,9 @@ private function validate($content, $file = null)
             ];
         }
 
+        libxml_clear_errors();
+        libxml_use_internal_errors($internal);
+
         return ['file' => $file, 'valid' => 0 === \count($errors), 'messages' => $errors];
     }
 

Original file line number	Diff line number	Diff line change
`@@ -115,7 +115,7 @@ private function validate($content, $file = null)`
`115`	`115`	`return ['file' => $file, 'valid' => true];`
`116`	`116`	`}`
`117`	`117`
`118`		`- libxml_use_internal_errors(true);`
	`118`	`+ $internal = libxml_use_internal_errors(true);`
`119`	`119`
`120`	`120`	`$document = new \DOMDocument();`
`121`	`121`	`$document->loadXML($content);`
`@@ -143,6 +143,9 @@ private function validate($content, $file = null)`
`143`	`143`	`];`
`144`	`144`	`}`
`145`	`145`
	`146`	`+ libxml_clear_errors();`
	`147`	`+ libxml_use_internal_errors($internal);`
	`148`	`+`
`146`	`149`	`return ['file' => $file, 'valid' => 0 === \count($errors), 'messages' => $errors];`
`147`	`150`	`}`
`148`	`151`