Update russian GeographicalNamesInflection::isMutable() and fix 'Респ…

…ублика' case
wapmorgan · Jul 28, 2018 · 6227400 · 6227400
1 parent 58101ed
commit 6227400
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/src/Russian/GeographicalNamesInflection.php b/src/Russian/GeographicalNamesInflection.php
@@ -49,7 +49,10 @@ public static function isMutable($name)
         }
 
         if (strpos($name, ' ') !== false) {
-            list($first_part, $last_part) = explode(' ', $name, 2);
+            // explode() is not applicable because Geographical unit may have few words
+            $first_part = S::slice($name, 0, S::findFirstPosition($name, ' '));
+            $last_part = S::slice($name,
+                S::findLastPosition($name, ' ') + 1);
 
             // город N, село N, хутор N, район N, поселок N, округ N, республика N
             // N область, N край
@@ -90,7 +93,9 @@ public static function getCases($name)
             if (in_array($first_part, ['город', 'село', 'хутор', 'пгт', 'район', 'поселок', 'округ', 'республика'], true)) {
                 if ($first_part !== 'пгт')
                     return self::composeCasesFromWords([
-                        NounDeclension::getCases($first_part),
+                        $first_part !== 'республика'
+                            ? NounDeclension::getCases($first_part)
+                            : array_map(['\\morphos\\S', 'name'], NounDeclension::getCases($first_part)),
                         array_fill_keys(self::getAllCases(), S::name(S::slice($name, S::length($first_part) + 1)))
                     ]);
                 else

diff --git a/tests/Russian/GeographicalNamesInflectionTest.php b/tests/Russian/GeographicalNamesInflectionTest.php
@@ -68,7 +68,7 @@ public function wordsProvider()
             ['село Привольное', 'села Привольное', 'селу Привольное', 'село Привольное', 'селом Привольное', 'селе Привольное'],
 
             // республика N
-            ['республика Крым', 'республики Крым', 'республике Крым', 'республику Крым', 'республикой Крым', 'республике Крым'],
+            ['Республика Крым', 'Республики Крым', 'Республике Крым', 'Республику Крым', 'Республикой Крым', 'Республике Крым'],
 
             // исключения
             ['Великие Луки', 'Великих Лук', 'Великим Лукам', 'Великие Луки', 'Великими Луками', 'Великих Луках'],