Permalink
Browse files

[CssSelector] fixed CssSelector::toXPath() when the CSS selector is a…

…n empty string
  • Loading branch information...
1 parent 1b9b428 commit a82737528ced35ae1ff2d71e14a48b49286237f8 @fabpot fabpot committed Mar 11, 2012
@@ -45,6 +45,10 @@ class CssSelector
static public function toXPath($cssExpr, $prefix = 'descendant-or-self::')
{
if (is_string($cssExpr)) {
+ if (!$cssExpr) {
+ return $prefix.'*';
+ }
+
if (preg_match('#^\w+\s*$#u', $cssExpr, $match)) {
return $prefix.trim($match[0]);
}
@@ -17,6 +17,7 @@ class CssSelectorTest extends \PHPUnit_Framework_TestCase
{
public function testCsstoXPath()
{
+ $this->assertEquals('descendant-or-self::*', CssSelector::toXPath(''));
$this->assertEquals('descendant-or-self::h1', CssSelector::toXPath('h1'));
$this->assertEquals("descendant-or-self::h1[@id = 'foo']", CssSelector::toXPath('h1#foo'));
$this->assertEquals("descendant-or-self::h1[contains(concat(' ', normalize-space(@class), ' '), ' foo ')]", CssSelector::toXPath('h1.foo'));

0 comments on commit a827375

Please sign in to comment.