merged branch cloppy/Dom-Crawler-fix (PR symfony#7127)

fabpot · fabpot · commit cf15198faa1c · 2013-02-19T21:07:22.000+01:00
This PR was merged into the 2.0 branch. Commits ------- cb03074 [DomCrawler] lowered parsed protocol string (fixes symfony#6986) Discussion ---------- [DomCrawler] lowered parsed protocol string (fixes symfony#6986) | Q | A | ------------- | --- | Bug fix? | yes | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | symfony#6986 | License | MIT | Doc PR | symfony/symfony-docs#123 Are UTs needed for this?
diff --git a/src/Symfony/Component/DomCrawler/Link.php b/src/Symfony/Component/DomCrawler/Link.php
@@ -46,7 +46,7 @@ class Link
      */
     public function __construct(\DOMNode $node, $currentUri, $method = 'GET')
     {
-        if (!in_array(substr($currentUri, 0, 4), array('http', 'file'))) {
+        if (!in_array(strtolower(substr($currentUri, 0, 4)), array('http', 'file'))) {
             throw new \InvalidArgumentException(sprintf('Current URI must be an absolute URL ("%s").', $currentUri));
         }
 

Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ class Link`
`46`	`46`	`*/`
`47`	`47`	`public function __construct(\DOMNode $node, $currentUri, $method = 'GET')`
`48`	`48`	`{`
`49`		`- if (!in_array(substr($currentUri, 0, 4), array('http', 'file'))) {`
	`49`	`+ if (!in_array(strtolower(substr($currentUri, 0, 4)), array('http', 'file'))) {`
`50`	`50`	`throw new \InvalidArgumentException(sprintf('Current URI must be an absolute URL ("%s").', $currentUri));`
`51`	`51`	`}`
`52`	`52`