Merge pull request #152 from alex-storchak/fix-cases-for-cardinal-num…

…-1000-russian Removed precalculated cases for cardinal number 1000 in Russian\CardinalNumeralGenerator
wapmorgan · Sep 26, 2024 · d228767 · d228767
2 parents 857ce52 + df01b61
commit d228767
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 8 deletions.
diff --git a/src/Russian/CardinalNumeralGenerator.php b/src/Russian/CardinalNumeralGenerator.php
@@ -171,14 +171,6 @@ class CardinalNumeralGenerator extends NumeralGenerator implements Cases
             self::TVORIT  => 'восьмистами',
             self::PREDLOJ => 'восьмистах',
         ],
-        'тысяча'      => [
-            self::IMENIT  => 'тысяча',
-            self::RODIT   => 'тысяч',
-            self::DAT     => 'тысячам',
-            self::VINIT   => 'тысяч',
-            self::TVORIT  => 'тысячей',
-            self::PREDLOJ => 'тысячах',
-        ],
     ];
 
     /**

diff --git a/tests/Russian/CardinalNumeralTest.php b/tests/Russian/CardinalNumeralTest.php
@@ -60,6 +60,16 @@ public function numbersProvider()
                 'тремястами сорока четырьмя',
                 'трехстах сорока четырех',
             ],
+            [
+                1000,
+                NumeralGenerator::FEMALE,
+                'тысяча',
+                'тысячи',
+                'тысяче',
+                'тысячу',
+                'тысячей',
+                'тысяче',
+            ],
             [
                 1007,
                 NumeralGenerator::MALE,