Merge pull request #91 from b1rdex/fix-floats

Fix floats
wapmorgan · Aug 3, 2020 · 5a8b5b0 · 5a8b5b0
2 parents 2976835 + 059c66a
commit 5a8b5b0
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/src/Russian/MoneySpeller.php b/src/Russian/MoneySpeller.php
@@ -45,7 +45,9 @@ public static function spell(
         $currency = static::canonizeCurrency($currency);
 
         $integer = floor($value);
-        $fractional = floor(($value * 100) % 100);
+        $fractional = fmod($value, $integer);
+        $fractional = round($fractional, 2);
+        $fractional *= 100;
 
         switch ($format) {
             case static::SHORT_FORMAT:

diff --git a/tests/Russian/MoneySpellerTest.php b/tests/Russian/MoneySpellerTest.php
@@ -48,6 +48,12 @@ public function moneyAmountsProvider()
             [123, Currency::PESO, MoneySpeller::SHORT_FORMAT, '123 песо'],
             [123, Currency::FRANC, MoneySpeller::SHORT_FORMAT, '123 франка'],
             [123, Currency::LIRA, MoneySpeller::SHORT_FORMAT, '123 лиры'],
+            [256.4, Currency::RUBLE, MoneySpeller::SHORT_FORMAT, '256 рублей 40 копеек'],
+            [65536.4, Currency::RUBLE, MoneySpeller::SHORT_FORMAT, '65536 рублей 40 копеек'],
+            [16777216.4, Currency::RUBLE, MoneySpeller::SHORT_FORMAT, '16777216 рублей 40 копеек'],
+            [4294967296.4, Currency::RUBLE, MoneySpeller::SHORT_FORMAT, '4294967296 рублей 40 копеек'],
+            [4294967297.4, Currency::RUBLE, MoneySpeller::SHORT_FORMAT, '4294967297 рублей 40 копеек'],
+            [4294967296.4, Currency::RUBLE, MoneySpeller::DUPLICATION_FORMAT, 'четыре миллиарда двести девяносто четыре миллиона девятьсот шестьдесят семь тысяч двести девяносто шесть (4294967296) рублей сорок (40) копеек'],
         ];
     }