Merge pull request #15 from v-technologies/fix-document-clean-flaot-inf

fix Document::_clean()
v-technologies · May 23, 2016 · 7a07b0e · 7a07b0e
2 parents 5a81444 + a60c88b
commit 7a07b0e
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/lib/Simples/Document.php b/lib/Simples/Document.php
@@ -329,7 +329,7 @@ protected function _clean(& $data, array $options = array(), $path = array()) {
 					} else {
 						unset($data[$key]) ;
 					}
-				} elseif (is_numeric($value)) {
+				} elseif (is_numeric($value) && (float)$value !== INF) {
 					$data[$key] = (float) $value ;
 				}
 			}

diff --git a/tests/lib/Simples/DocumentTest.php b/tests/lib/Simples/DocumentTest.php
@@ -123,4 +123,17 @@ public function testClean() {
 		$res = $document->to('array', array('clean' => true, 'cast' => array('sub.string' => 'float'))) ;
 		$this->assertTrue($res['sub'][0]['string'] === 9.0);
 	}
+
+	public function testCleanFloatInf() {
+		// Test clean
+		$document = new Simples_Document(array(
+			'string' => '3E7210'
+		)) ;
+		$res = $document->to('array', array('clean' => true)) ;
+
+		$this->assertTrue($res['string'] === '3E7210');
+
+		$res = $document->to('json', array('clean' => true)) ;
+		$this->assertEquals($res, '{"string":"3E7210"}');
+	}
 }