From 133f5952fd5fdfbe8e59656037d2b1026667b742 Mon Sep 17 00:00:00 2001 From: Santhosh Thottingal Date: Mon, 18 Mar 2013 16:37:28 +0530 Subject: [PATCH] Remove custom plurals for Nso and Sl in favour of CLDR Nso - Northern Sotho Sl - Slovenian Plural rules were not changed. They are same in CLDR and MW Change-Id: I0e0c84352de2de8f58af5a9147ba18b0fe1fb39a --- languages/classes/LanguageNso.php | 44 ---------------------- languages/classes/LanguageSl.php | 23 ----------- tests/phpunit/languages/LanguageSlTest.php | 4 +- 3 files changed, 2 insertions(+), 69 deletions(-) delete mode 100644 languages/classes/LanguageNso.php diff --git a/languages/classes/LanguageNso.php b/languages/classes/LanguageNso.php deleted file mode 100644 index 0c0867506e..0000000000 --- a/languages/classes/LanguageNso.php +++ /dev/null @@ -1,44 +0,0 @@ -preConvertPlural( $forms, 2 ); - - return ( $count <= 1 ) ? $forms[0] : $forms[1]; - } -} diff --git a/languages/classes/LanguageSl.php b/languages/classes/LanguageSl.php index c075e51542..77957b2b88 100644 --- a/languages/classes/LanguageSl.php +++ b/languages/classes/LanguageSl.php @@ -53,27 +53,4 @@ class LanguageSl extends Language { return $word; # this will return the original value for 'imenovalnik' (nominativ) and all undefined case values } - /** - * @param $count int - * @param $forms array - * - * @return string - */ - function convertPlural( $count, $forms ) { - if ( !count( $forms ) ) { return ''; } - $forms = $this->preConvertPlural( $forms, 5 ); - - if ( $count % 100 == 1 ) { - $index = 0; - } elseif ( $count % 100 == 2 ) { - $index = 1; - } elseif ( $count % 100 == 3 || $count % 100 == 4 ) { - $index = 2; - } elseif ( $count != 0 ) { - $index = 3; - } else { - $index = 4; - } - return $forms[$index]; - } } diff --git a/tests/phpunit/languages/LanguageSlTest.php b/tests/phpunit/languages/LanguageSlTest.php index 075e6af364..96c370f64b 100644 --- a/tests/phpunit/languages/LanguageSlTest.php +++ b/tests/phpunit/languages/LanguageSlTest.php @@ -11,13 +11,13 @@ class LanguageSlTest extends LanguageClassesTestCase { /** @dataProvider providerPlural */ function testPlural( $result, $value ) { - $forms = array( 'one', 'two', 'few', 'other', 'zero' ); + $forms = array( 'one', 'two', 'few', 'other' ); $this->assertEquals( $result, $this->getLang()->convertPlural( $value, $forms ) ); } function providerPlural() { return array( - array( 'zero', 0 ), + array( 'other', 0 ), array( 'one', 1 ), array( 'two', 2 ), array( 'few', 3 ), -- 2.20.1