[DomCrawler] Improve the sprintf() call in selectLink()

dunglas · dunglas · commit e00b3a7b3ef4 · 2018-03-11T10:04:59.000+01:00
diff --git a/Crawler.php b/Crawler.php
@@ -710,10 +710,9 @@ public function filter($selector)
      */
     public function selectLink($value)
     {
-        $xpath = sprintf('descendant-or-self::a[contains(concat(\' \', normalize-space(string(.)), \' \'), %s) ', static::xpathLiteral(' '.$value.' ')).
-                            sprintf('or ./img[contains(concat(\' \', normalize-space(string(@alt)), \' \'), %s)]]', static::xpathLiteral(' '.$value.' '));
-
-        return $this->filterRelativeXPath($xpath);
+        return $this->filterRelativeXPath(
+            sprintf('descendant-or-self::a[contains(concat(\' \', normalize-space(string(.)), \' \'), %1$s) or ./img[contains(concat(\' \', normalize-space(string(@alt)), \' \'), %1$s)]]', static::xpathLiteral(' '.$value.' '))
+        );
     }
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -710,10 +710,9 @@ public function filter($selector)`
`710`	`710`	`*/`
`711`	`711`	`public function selectLink($value)`
`712`	`712`	`{`
`713`		`- $xpath = sprintf('descendant-or-self::a[contains(concat(\' \', normalize-space(string(.)), \' \'), %s) ', static::xpathLiteral(' '.$value.' ')).`
`714`		`- sprintf('or ./img[contains(concat(\' \', normalize-space(string(@alt)), \' \'), %s)]]', static::xpathLiteral(' '.$value.' '));`
`715`		`-`
`716`		`- return $this->filterRelativeXPath($xpath);`
	`713`	`+ return $this->filterRelativeXPath(`
	`714`	`+ sprintf('descendant-or-self::a[contains(concat(\' \', normalize-space(string(.)), \' \'), %1$s) or ./img[contains(concat(\' \', normalize-space(string(@alt)), \' \'), %1$s)]]', static::xpathLiteral(' '.$value.' '))`
	`715`	`+ );`
`717`	`716`	`}`
`718`	`717`
`719`	`718`	`/**`