Description: ------------ The CSS tokenizer fails to turn quoted urls into type T_URL. It's a simple fix in Tokenizer/CSS.PHP: Line 245: change "$x + 2" to "$x + 1" Line 257: change ".=" to "=" Test script: --------------- .test { background-image: url("foobar.png"); } Expected result: ---------------- expect the tokenizer to create a T_URL token with value "foobar.png" Actual result: -------------- The tokenizer just creates a T_CONSTANT_ENCAPSED_STRING

I see the error, but your fix actually puts the tokens in incorrect positions for me (at the end). Something else obviously needs fixing as well, so let me look into it.

Found the problem. If the contnet inside the "url()" is in double quotes there will only be one token and so we don't have to do anything except change its type. If it is not empty, we need to do some token merging. What it was doing is just not changing the token type unless there was more than one token inside the "url()". So no changes to the merging code needed at all - just needed to move a couple of lines out of the if statement.

This bug has been fixed in SVN. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.