Merge remote-tracking branch 'slusarz/float_fix'

* slusarz/float_fix:
  Fix Size output for locales that use non-period decimal separators

Closes MyIntervals#61 as fixed

Author: sabberworm

CHANGELOG.md

@@ -31,6 +31,12 @@
 * *No backwards-incompatible changes*
 * *No deprecations*
 
+### 5.0.4 (2013-03-21)
+
+* Don’t output floats with locale-aware separator chars
+* *No backwards-incompatible changes*
+* *No deprecations*
+
 ## 4.0
 
 ### 4.0.0 (2013-03-19)

lib/Sabberworm/CSS/Value/Size.php

@@ -60,7 +60,8 @@ public function isRelative() {
 	}
 
 	public function __toString() {
-		return str_replace('0.', '.', $this->fSize) . ($this->sUnit === null ? '' : $this->sUnit);
+		$l = localeconv();
+		return str_replace(array($l['decimal_point'], '0.'), '.', $this->fSize) . ($this->sUnit === null ? '' : $this->sUnit);
 	}
 
 }

tests/Sabberworm/CSS/RuleSet/LenientParsingTest.php

@@ -20,7 +20,15 @@ public function testFaultToleranceOn() {
 		$sFile = dirname(__FILE__) . '/../../../files' . DIRECTORY_SEPARATOR . "fault-tolerance.css";
 		$oParser = new Parser(file_get_contents($sFile), Settings::create()->withLenientParsing(true));
 		$oResult = $oParser->parse();
-		$this->assertSame('.test1 {}'."\n".'.test2 {hello: 2;}'."\n", $oResult->__toString());
+		$this->assertSame('.test1 {}'."\n".'.test2 {hello: 2.2;hello: 200000000000.2;}'."\n", $oResult->__toString());
+	}
+
+	public function testLocaleTrap() {
+		setlocale(LC_ALL, "pt_PT", "no");
+		$sFile = dirname(__FILE__) . '/../../../files' . DIRECTORY_SEPARATOR . "fault-tolerance.css";
+		$oParser = new Parser(file_get_contents($sFile), Settings::create()->withLenientParsing(true));
+		$oResult = $oParser->parse();
+		$this->assertSame('.test1 {}'."\n".'.test2 {hello: 2.2;hello: 200000000000.2;}'."\n", $oResult->__toString());
 	}
 
 	public function testCaseInsensitivity() {

tests/files/fault-tolerance.css

@@ -4,5 +4,6 @@
 
 .test2 {
 	*hello: 1;
-	hello: 2;
+	hello: 2.2;
+	hello: 2000000000000.2;
 }