Fix geo russian names

wapmorgan · Nov 17, 2017 · 190d7e1 · 190d7e1
1 parent e4e1f7e
commit 190d7e1
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 3 deletions.
diff --git a/src/Russian/GeographicalNamesInflection.php b/src/Russian/GeographicalNamesInflection.php
@@ -22,6 +22,10 @@ class GeographicalNamesInflection extends \morphos\BaseInflection implements Cas
         '-на-'
     );
 
+    protected static $ovAbnormalExceptions = [
+        'осташков',
+    ];
+
     /**
      * Проверяет, склоняемо ли название
      * @param string $name Название
@@ -88,6 +92,7 @@ public static function getCases($name)
             ]);
         }
 
+        // Сложное название через пробел или через '-на-'
         foreach (self::$delimiters as $delimiter) {
             if (strpos($name, $delimiter) !== false) {
                 $parts = explode($delimiter, $name);
@@ -100,7 +105,7 @@ public static function getCases($name)
         }
 
         // Сложное название города из нескольких слов через тире.
-        // Ростов-на-Дону, Переславль-Залесский
+        // Ростов-на-Дону, Переславль-Залесский и т.д.
         if (strpos($name, '-') !== false) {
             $parts = explode('-', $name);
             $result = array(
@@ -179,7 +184,7 @@ public static function getCases($name)
                     self::TVORIT => $prefix.'ем',
                     self::PREDLOJ => $prefix.'е',
                 );
-            } elseif (self::isConsonant(S::slice($name, -1)) && S::slice($name, -2) != 'ов') {
+            } elseif (self::isConsonant(S::slice($name, -1)) && !in_array($name, self::$ovAbnormalExceptions)) {
                 // Париж, Валаам, Киев
                 $prefix = S::name($name);
                 return array(

diff --git a/tests/Russian/GeographicalNamesInflectionTest.php b/tests/Russian/GeographicalNamesInflectionTest.php
@@ -43,13 +43,14 @@ public function wordsProvider()
             ['Осташков', 'Осташкова', 'Осташкову', 'Осташков', 'Осташковым', 'Осташкове'],
             ['Грозный', 'Грозного', 'Грозному', 'Грозный', 'Грозным', 'Грозном'],
             ['Благодарный', 'Благодарного', 'Благодарному', 'Благодарный', 'Благодарным', 'Благодарном'],
+            ['Псков', 'Пскова', 'Пскову', 'Псков', 'Псковом', 'Пскове'],
+            ['Киров', 'Кирова', 'Кирову', 'Киров', 'Кировом', 'Кирове'],
 
             // сложные названия
             ['Санкт-Петербург', 'Санкт-Петербурга', 'Санкт-Петербургу', 'Санкт-Петербург', 'Санкт-Петербургом', 'Санкт-Петербурге'],
             ['Ростов-на-Дону', 'Ростова-на-Дону', 'Ростову-на-Дону', 'Ростов-на-Дону', 'Ростовом-на-Дону', 'Ростове-на-Дону'],
             ['Нижний Новгород', 'Нижнего Новгорода', 'Нижнему Новгороду', 'Нижний Новгород', 'Нижним Новгородом', 'Нижнем Новгороде'],
             ['Набережные Челны', 'Набережных Челнов', 'Набережным Челнам', 'Набережные Челны', 'Набережными Челнами', 'Набережных Челнах'],
-            ['Ростов-на-Дону', 'Ростова-на-Дону', 'Ростову-на-Дону', 'Ростов-на-Дону', 'Ростовом-на-Дону', 'Ростове-на-Дону'],
 
             // N край
             ['Краснодарский край', 'Краснодарского края', 'Краснодарскому краю', 'Краснодарский край', 'Краснодарским краем', 'Краснодарском крае'],