Knah-Tsaeb/MyShaarli
Knah-Tsaeb/MyShaarli
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

Fix autoLocale error and cover it with unit tests

Browse source
This commit is contained in:
ArthurHoaro 2017-03-07 19:27:17 +01:00
parent 36c8fb1ef8
commit 03b9cb600a
4 changed files with 249 additions and 13 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

28
application/Utils.php
View file

@ -216,22 +216,30 @@ function is_session_id_valid($sessionId)
function autoLocale($headerLocale) function autoLocale($headerLocale)
{ {
// Default if browser does not send HTTP_ACCEPT_LANGUAGE // Default if browser does not send HTTP_ACCEPT_LANGUAGE
$attempts = array('en_US', 'en_US.utf8', 'en_US.UTF-8'); $locales = array('en_US', 'en_US.utf8', 'en_US.UTF-8');
if (isset($headerLocale)) { if (! empty($headerLocale)) {
// (It's a bit crude, but it works very well. Preferred language is always presented first.) if (preg_match_all('/([a-z]{2,3})[-_]?([a-z]{2})?,?/i', $headerLocale, $matches, PREG_SET_ORDER)) {
if (preg_match('/([a-z]{2,3})[-_]?([a-z]{2})?/i', $headerLocale, $matches)) { $attempts = [];
$first = [strtolower($matches[1]), strtoupper($matches[1])]; foreach ($matches as $match) {
$first = [strtolower($match[1]), strtoupper($match[1])];
$separators = ['_', '-']; $separators = ['_', '-'];
$encodings = ['utf8', 'UTF-8']; $encodings = ['utf8', 'UTF-8'];
if (!empty($matches[2])) { if (!empty($match[2])) {
$second = [strtoupper($matches[2]), strtolower($matches[2])]; $second = [strtoupper($match[2]), strtolower($match[2])];
$attempts = cartesian_product_generator([$first, $separators, $second, ['.'], $encodings]); $items = [$first, $separators, $second, ['.'], $encodings];
} else { } else {
$attempts = cartesian_product_generator([$first, $separators, $first, ['.'], $encodings]); $items = [$first, $separators, $first, ['.'], $encodings];
}
$attempts = array_merge($attempts, iterator_to_array(cartesian_product_generator($items)));
}
if (! empty($attempts)) {
$locales = array_merge(array_map('implode', $attempts), $locales);
} }
} }
} }
setlocale(LC_ALL, implode('implode', iterator_to_array($attempts)));
setlocale(LC_ALL, $locales);
} }
/** /**

76
tests/languages/de/UtilsDeTest.php
View file

@ -22,4 +22,80 @@ public function testDateFormatDefault()
$date = DateTime::createFromFormat('Ymd_His', '20170101_101112'); $date = DateTime::createFromFormat('Ymd_His', '20170101_101112');
$this->assertEquals('So 01 Jan 2017 10:11:12 EAT', format_date($date, false)); $this->assertEquals('So 01 Jan 2017 10:11:12 EAT', format_date($date, false));
} }
/**
* Test autoLocale with a simple value
*/
public function testAutoLocaleValid()
{
$current = setlocale(LC_ALL, 0);
$header = 'en-us';
autoLocale($header);
$this->assertEquals('en_US.utf8', setlocale(LC_ALL, 0));
setlocale(LC_ALL, $current);
}
/**
* Test autoLocale with an alternative locale value
*/
public function testAutoLocaleValidAlternative()
{
$current = setlocale(LC_ALL, 0);
$header = 'en_us.UTF8';
autoLocale($header);
$this->assertEquals('en_US.utf8', setlocale(LC_ALL, 0));
setlocale(LC_ALL, $current);
}
/**
* Test autoLocale with multiples value, the first one is valid
*/
public function testAutoLocaleMultipleFirstValid()
{
$current = setlocale(LC_ALL, 0);
$header = 'en-us,de-de';
autoLocale($header);
$this->assertEquals('en_US.utf8', setlocale(LC_ALL, 0));
setlocale(LC_ALL, $current);
}
/**
* Test autoLocale with multiples value, the second one is valid
*/
public function testAutoLocaleMultipleSecondValid()
{
$current = setlocale(LC_ALL, 0);
$header = 'pt_BR,fr-fr';
autoLocale($header);
$this->assertEquals('fr_FR.utf8', setlocale(LC_ALL, 0));
setlocale(LC_ALL, $current);
}
/**
* Test autoLocale without value: defaults to en_US.
*/
public function testAutoLocaleBlank()
{
$current = setlocale(LC_ALL, 0);
autoLocale('');
$this->assertEquals('en_US.utf8', setlocale(LC_ALL, 0));
setlocale(LC_ALL, $current);
}
/**
* Test autoLocale with an invalid value: defaults to en_US.
*/
public function testAutoLocaleInvalid()
{
$current = setlocale(LC_ALL, 0);
autoLocale('pt_BR');
$this->assertEquals('en_US.utf8', setlocale(LC_ALL, 0));
setlocale(LC_ALL, $current);
}
} }

76
tests/languages/en/UtilsEnTest.php
View file

@ -22,4 +22,80 @@ public function testDateFormatDefault()
$date = DateTime::createFromFormat('Ymd_His', '20170101_101112'); $date = DateTime::createFromFormat('Ymd_His', '20170101_101112');
$this->assertEquals('Sun 01 Jan 2017 10:11:12 AM EAT', format_date($date, false)); $this->assertEquals('Sun 01 Jan 2017 10:11:12 AM EAT', format_date($date, false));
} }
/**
* Test autoLocale with a simple value
*/
public function testAutoLocaleValid()
{
$current = setlocale(LC_ALL, 0);
$header = 'de-de';
autoLocale($header);
$this->assertEquals('de_DE.utf8', setlocale(LC_ALL, 0));
setlocale(LC_ALL, $current);
}
/**
* Test autoLocale with an alternative locale value
*/
public function testAutoLocaleValidAlternative()
{
$current = setlocale(LC_ALL, 0);
$header = 'de_de.UTF8';
autoLocale($header);
$this->assertEquals('de_DE.utf8', setlocale(LC_ALL, 0));
setlocale(LC_ALL, $current);
}
/**
* Test autoLocale with multiples value, the first one is valid
*/
public function testAutoLocaleMultipleFirstValid()
{
$current = setlocale(LC_ALL, 0);
$header = 'de-de;en-us';
autoLocale($header);
$this->assertEquals('de_DE.utf8', setlocale(LC_ALL, 0));
setlocale(LC_ALL, $current);
}
/**
* Test autoLocale with multiples value, the second one is valid
*/
public function testAutoLocaleMultipleSecondValid()
{
$current = setlocale(LC_ALL, 0);
$header = 'pt_BR,fr-fr';
autoLocale($header);
$this->assertEquals('fr_FR.utf8', setlocale(LC_ALL, 0));
setlocale(LC_ALL, $current);
}
/**
* Test autoLocale without value: defaults to en_US.
*/
public function testAutoLocaleBlank()
{
$current = setlocale(LC_ALL, 0);
autoLocale('');
$this->assertEquals('en_US.utf8', setlocale(LC_ALL, 0));
setlocale(LC_ALL, $current);
}
/**
* Test autoLocale with an invalid value: defaults to en_US.
*/
public function testAutoLocaleInvalid()
{
$current = setlocale(LC_ALL, 0);
autoLocale('pt_BR');
$this->assertEquals('en_US.utf8', setlocale(LC_ALL, 0));
setlocale(LC_ALL, $current);
}
} }

76
tests/languages/fr/UtilsFrTest.php
View file

@ -22,4 +22,80 @@ public function testDateFormatDefault()
$date = DateTime::createFromFormat('Ymd_His', '20170101_101112'); $date = DateTime::createFromFormat('Ymd_His', '20170101_101112');
$this->assertEquals('dim. 01 janv. 2017 10:11:12 EAT', format_date($date, false)); $this->assertEquals('dim. 01 janv. 2017 10:11:12 EAT', format_date($date, false));
} }
/**
* Test autoLocale with a simple value
*/
public function testAutoLocaleValid()
{
$current = setlocale(LC_ALL, 0);
$header = 'de-de';
autoLocale($header);
$this->assertEquals('de_DE.utf8', setlocale(LC_ALL, 0));
setlocale(LC_ALL, $current);
}
/**
* Test autoLocale with an alternative locale value
*/
public function testAutoLocaleValidAlternative()
{
$current = setlocale(LC_ALL, 0);
$header = 'de_de.UTF8';
autoLocale($header);
$this->assertEquals('de_DE.utf8', setlocale(LC_ALL, 0));
setlocale(LC_ALL, $current);
}
/**
* Test autoLocale with multiples value, the first one is valid
*/
public function testAutoLocaleMultipleFirstValid()
{
$current = setlocale(LC_ALL, 0);
$header = 'de-de;en-us';
autoLocale($header);
$this->assertEquals('de_DE.utf8', setlocale(LC_ALL, 0));
setlocale(LC_ALL, $current);
}
/**
* Test autoLocale with multiples value, the second one is valid
*/
public function testAutoLocaleMultipleSecondValid()
{
$current = setlocale(LC_ALL, 0);
$header = 'pt_BR,de-de';
autoLocale($header);
$this->assertEquals('de_DE.utf8', setlocale(LC_ALL, 0));
setlocale(LC_ALL, $current);
}
/**
* Test autoLocale without value: defaults to en_US.
*/
public function testAutoLocaleBlank()
{
$current = setlocale(LC_ALL, 0);
autoLocale('');
$this->assertEquals('en_US.utf8', setlocale(LC_ALL, 0));
setlocale(LC_ALL, $current);
}
/**
* Test autoLocale with an invalid value: defaults to en_US.
*/
public function testAutoLocaleInvalid()
{
$current = setlocale(LC_ALL, 0);
autoLocale('pt_BR');
$this->assertEquals('en_US.utf8', setlocale(LC_ALL, 0));
setlocale(LC_ALL, $current);
}
} }