tonyair
/
silverstripe-framework
mirror of https://github.com/silverstripe/silverstripe-framework
1
0
Fork 0
Code Issues Projects Releases Wiki Activity

FIX IntlLocales::validate when lang and region are the same e.g. de_DE

This commit is contained in:
Robbie Averill 2017-04-27 18:31:03 +12:00
parent 16aebb83e4
commit 3a372a1f41
2 changed files with 15 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/i18n/Data/Intl/IntlLocales.php
View File

@ -1605,6 +1605,18 @@ class IntlLocales implements Locales, Resettable
if (!$lang || !$region) {
return false;
}
// Check the configurable whitelist
$localeCode = strtolower($lang) . '_' . strtoupper($region);
$locales = $this->getLocales();
if (array_key_exists($localeCode, $locales)
|| array_key_exists(strtolower($localeCode), $locales)
) {
return true;
}
// Fallback
return strcasecmp($lang, $region)
&& strcasecmp($lang, $locale)
&& strcasecmp($region, $locale);

3
tests/php/i18n/i18nTest.php
View File

@ -376,6 +376,9 @@ class i18nTest extends SapphireTest
$this->assertFalse(i18n::getData()->validate('en'), 'Short lang format is not valid');
$this->assertFalse(i18n::getData()->validate('xx_XX'), 'Unknown locale in correct format is not valid');
$this->assertFalse(i18n::getData()->validate(''), 'Empty string is not valid');
$this->assertTrue(i18n::getData()->validate('de_DE'), 'Known locale where language is same as region');
$this->assertTrue(i18n::getData()->validate('fr-FR'), 'Known locale where language is same as region');
$this->assertTrue(i18n::getData()->validate('zh_cmn'), 'Known locale with all lowercase letters');
}
public function testTranslate()