fix(NumberConstraint): force positive value on first arg

bighappyface · bighappyface · commit 999424181e0a · 2016-10-09T15:22:46.000-05:00
Resolves #315 - `multipleOf` is inconsistent, fails for negative numbers
diff --git a/.gitignore b/.gitignore
@@ -7,3 +7,4 @@ coverage
 .settings
 composer.lock
 docs-api
+phpunit.xml
diff --git a/src/JsonSchema/Constraints/NumberConstraint.php b/src/JsonSchema/Constraints/NumberConstraint.php
@@ -69,6 +69,7 @@ public function check($element, $schema = null, JsonPointer $path = null, $i = n
 
     private function fmod($number1, $number2)
     {
+        $number1 = abs($number1);
         $modulus = fmod($number1, $number2);
         $precision = abs(0.0000000001);
         $diff = (float)($modulus - $number2);

Original file line number	Diff line number	Diff line change
`@@ -69,6 +69,7 @@ public function check($element, $schema = null, JsonPointer $path = null, $i = n`
`69`	`69`
`70`	`70`	`private function fmod($number1, $number2)`
`71`	`71`	`{`
	`72`	`+ $number1 = abs($number1);`
`72`	`73`	`$modulus = fmod($number1, $number2);`
`73`	`74`	`$precision = abs(0.0000000001);`
`74`	`75`	`$diff = (float)($modulus - $number2);`