bug #16873 Able to load big xml files with DomCrawler (zorn-v)

fabpot · fabpot · commit ec26f6ece07e · 2016-01-25T09:23:30.000+01:00
This PR was submitted for the 2.8 branch but it was merged into the 2.3 branch instead (closes #16873). Discussion ---------- Able to load big xml files with DomCrawler | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | | License | MIT | Doc PR | Commits ------- 3dae825 Able to load big xml files with DomCrawler
diff --git a/src/Symfony/Component/DomCrawler/Crawler.php b/src/Symfony/Component/DomCrawler/Crawler.php
@@ -230,7 +230,7 @@ public function addXmlContent($content, $charset = 'UTF-8')
 
         if ('' !== trim($content)) {
             // remove the default namespace to make XPath expressions simpler
-            @$dom->loadXML(str_replace('xmlns', 'ns', $content), LIBXML_NONET);
+            @$dom->loadXML(str_replace('xmlns', 'ns', $content), LIBXML_NONET | (defined('LIBXML_PARSEHUGE') ? LIBXML_PARSEHUGE : 0));
         }
 
         libxml_use_internal_errors($internalErrors);

Original file line number	Diff line number	Diff line change
`@@ -230,7 +230,7 @@ public function addXmlContent($content, $charset = 'UTF-8')`
`230`	`230`
`231`	`231`	`if ('' !== trim($content)) {`
`232`	`232`	`// remove the default namespace to make XPath expressions simpler`
`233`		`- @$dom->loadXML(str_replace('xmlns', 'ns', $content), LIBXML_NONET);`
	`233`	`+ @$dom->loadXML(str_replace('xmlns', 'ns', $content), LIBXML_NONET \| (defined('LIBXML_PARSEHUGE') ? LIBXML_PARSEHUGE : 0));`
`234`	`234`	`}`
`235`	`235`
`236`	`236`	`libxml_use_internal_errors($internalErrors);`