summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
Diffstat
m---------3rdparty0
-rw-r--r--core/js/config.php4
-rw-r--r--core/l10n/l10n-de.php7
-rw-r--r--core/l10n/l10n-en.php7
-rw-r--r--core/l10n/l10n-es.php7
-rw-r--r--lib/private/l10n.php106
-rw-r--r--tests/lib/l10n.php52
7 files changed, 94 insertions, 89 deletions
diff --git a/3rdparty b/3rdparty
-Subproject f6d7a519e4dca5189963abb15e5c9858b03bf98
+Subproject 0c0fb2a67dfa0392ebcea2fab558513f0000625
diff --git a/core/js/config.php b/core/js/config.php
index 6994f2ed8a2..52405725f23 100644
--- a/core/js/config.php
+++ b/core/js/config.php
@@ -39,7 +39,7 @@ $array = array(
"oc_isadmin" => OC_User::isAdminUser(OC_User::getUser()) ? 'true' : 'false',
"oc_webroot" => "\"".OC::$WEBROOT."\"",
"oc_appswebroots" => str_replace('\\/', '/', json_encode($apps_paths)), // Ugly unescape slashes waiting for better solution
- "datepickerFormatDate" => json_encode($l->l('jsdate', 'jsdate')),
+ "datepickerFormatDate" => json_encode($l->getDateFormat()),
"dayNames" => json_encode(
array(
(string)$l->t('Sunday'),
@@ -67,7 +67,7 @@ $array = array(
(string)$l->t('December')
)
),
- "firstDay" => json_encode($l->l('firstday', 'firstday')) ,
+ "firstDay" => json_encode($l->getFirstWeekDay()) ,
"oc_config" => json_encode(
array(
'session_lifetime' => min(\OCP\Config::getSystemValue('session_lifetime', ini_get('session.gc_maxlifetime')), ini_get('session.gc_maxlifetime')),
diff --git a/core/l10n/l10n-de.php b/core/l10n/l10n-de.php
deleted file mode 100644
index 77d35af4936..00000000000
--- a/core/l10n/l10n-de.php
+++ /dev/null
@@ -1,7 +0,0 @@
-<?php
-$LOCALIZATIONS = array(
- 'jsdate' => 'dd.mm.yy',
- 'date' => '%d.%m.%Y',
- 'datetime' => '%d.%m.%Y %H:%M:%S',
- 'time' => '%H:%M:%S',
- 'firstday' => 0 );
diff --git a/core/l10n/l10n-en.php b/core/l10n/l10n-en.php
deleted file mode 100644
index 9ee748bee23..00000000000
--- a/core/l10n/l10n-en.php
+++ /dev/null
@@ -1,7 +0,0 @@
-<?php
-$LOCALIZATIONS = array(
- 'jsdate' => 'MM d, yy',
- 'date' => '%B %e, %Y',
- 'datetime' => '%B %e, %Y %H:%M',
- 'time' => '%H:%M:%S',
- 'firstday' => 0 );
diff --git a/core/l10n/l10n-es.php b/core/l10n/l10n-es.php
deleted file mode 100644
index 13db2ec5d4c..00000000000
--- a/core/l10n/l10n-es.php
+++ /dev/null
@@ -1,7 +0,0 @@
-<?php
-$LOCALIZATIONS = array(
- 'jsdate' => "d 'de' MM 'de' yy",
- 'date' => '%e de %B de %Y',
- 'datetime' => '%e de %B de %Y %H:%M',
- 'time' => '%H:%M:%S',
- 'firstday' => 1 );
diff --git a/lib/private/l10n.php b/lib/private/l10n.php
index 57886a796cd..0b20eafea32 100644
--- a/lib/private/l10n.php
+++ b/lib/private/l10n.php
@@ -62,16 +62,6 @@ class OC_L10N implements \OCP\IL10N {
private $plural_form_function = null;
/**
- * Localization
- */
- private $localizations = array(
- 'jsdate' => 'dd.mm.yy',
- 'date' => '%d.%m.%Y',
- 'datetime' => '%d.%m.%Y %H:%M:%S',
- 'time' => '%H:%M:%S',
- 'firstday' => 0);
-
- /**
* get an L10N instance
* @param string $app
* @param string|null $lang
@@ -126,13 +116,10 @@ class OC_L10N implements \OCP\IL10N {
// Use cache if possible
if(array_key_exists($app.'::'.$lang, self::$cache)) {
-
$this->translations = self::$cache[$app.'::'.$lang]['t'];
- $this->localizations = self::$cache[$app.'::'.$lang]['l'];
- }
- else{
+ } else{
$i18ndir = self::findI18nDir($app);
- // Localization is in /l10n, Texts are in $i18ndir
+ // Texts are in $i18ndir
// (Just no need to define date/time format etc. twice)
if((OC_Helper::isSubDirectory($i18ndir.$lang.'.php', OC::$SERVERROOT.'/core/l10n/')
|| OC_Helper::isSubDirectory($i18ndir.$lang.'.php', OC::$SERVERROOT.'/lib/l10n/')
@@ -162,16 +149,7 @@ class OC_L10N implements \OCP\IL10N {
}
}
- if(file_exists(OC::$SERVERROOT.'/core/l10n/l10n-'.$lang.'.php') && OC_Helper::isSubDirectory(OC::$SERVERROOT.'/core/l10n/l10n-'.$lang.'.php', OC::$SERVERROOT.'/core/l10n/')) {
- // Include the file, save the data from $CONFIG
- include OC::$SERVERROOT.'/core/l10n/l10n-'.$lang.'.php';
- if(isset($LOCALIZATIONS) && is_array($LOCALIZATIONS)) {
- $this->localizations = array_merge($this->localizations, $LOCALIZATIONS);
- }
- }
-
self::$cache[$app.'::'.$lang]['t'] = $this->translations;
- self::$cache[$app.'::'.$lang]['l'] = $this->localizations;
}
}
@@ -313,17 +291,6 @@ class OC_L10N implements \OCP\IL10N {
}
/**
- * get localizations
- * @return array Fetch all localizations
- *
- * Returns an associative array with all localizations
- */
- public function getLocalizations() {
- $this->init();
- return $this->localizations;
- }
-
- /**
* Localization
* @param string $type Type of localization
* @param array|int|string $data parameters for this localization
@@ -334,45 +301,45 @@ class OC_L10N implements \OCP\IL10N {
* Implemented types:
* - date
* - Creates a date
- * - l10n-field: date
* - params: timestamp (int/string)
* - datetime
* - Creates date and time
- * - l10n-field: datetime
* - params: timestamp (int/string)
* - time
* - Creates a time
- * - l10n-field: time
* - params: timestamp (int/string)
*/
- public function l($type, $data) {
+ public function l($type, $data, $options = array()) {
+ if ($type === 'firstday') {
+ return $this->getFirstWeekDay();
+ }
+ if ($type === 'jsdate') {
+ return $this->getDateFormat();
+ }
+
$this->init();
+ $value = new DateTime();
+ if($data instanceof DateTime) {
+ $value = $data;
+ } elseif(is_string($data) && !is_numeric($data)) {
+ $data = strtotime($data);
+ $value->setTimestamp($data);
+ } else {
+ $value->setTimestamp($data);
+ }
+ $locale = self::findLanguage();
+ $options = array_merge(array('width' => 'long'), $options);
+ $width = $options['width'];
switch($type) {
- // If you add something don't forget to add it to $localizations
- // at the top of the page
case 'date':
+ return Punic\Calendar::formatDate($value, $width, $locale);
+ break;
case 'datetime':
+ return Punic\Calendar::formatDatetime($value, $width, $locale);
+ break;
case 'time':
- if($data instanceof DateTime) {
- $data = $data->getTimestamp();
- } elseif(is_string($data) && !is_numeric($data)) {
- $data = strtotime($data);
- }
- $locales = array(self::findLanguage());
- if (strlen($locales[0]) == 2) {
- $locales[] = $locales[0].'_'.strtoupper($locales[0]);
- }
- setlocale(LC_TIME, $locales);
- $format = $this->localizations[$type];
- // Check for Windows to find and replace the %e modifier correctly
- if (strtoupper(substr(PHP_OS, 0, 3)) == 'WIN') {
- $format = preg_replace('#(?<!%)((?:%%)*)%e#', '\1%#d', $format);
- }
- return strftime($format, $data);
+ return Punic\Calendar::formatTime($value, $width, $locale);
break;
- case 'firstday':
- case 'jsdate':
- return $this->localizations[$type];
default:
return false;
}
@@ -495,7 +462,7 @@ class OC_L10N implements \OCP\IL10N {
/**
* find the l10n directory
* @param string $app App that needs to be translated
- * @return directory
+ * @return string directory
*/
protected static function findI18nDir($app) {
// find the i18n dir
@@ -547,4 +514,21 @@ class OC_L10N implements \OCP\IL10N {
}
return false;
}
+
+ /**
+ * @return string
+ * @throws \Punic\Exception\ValueNotInList
+ */
+ public function getDateFormat() {
+ $locale = self::findLanguage();
+ return Punic\Calendar::getDateFormat('short', $locale);
+ }
+
+ /**
+ * @return int
+ */
+ public function getFirstWeekDay() {
+ $locale = self::findLanguage();
+ return Punic\Calendar::getFirstWeekday($locale);
+ }
}
diff --git a/tests/lib/l10n.php b/tests/lib/l10n.php
index 5ddf2290c35..26ad87b60f6 100644
--- a/tests/lib/l10n.php
+++ b/tests/lib/l10n.php
@@ -52,17 +52,59 @@ class Test_L10n extends PHPUnit_Framework_TestCase {
$this->assertEquals('5 oken', (string)$l->n('%n window', '%n windows', 5));
}
+ public function localizationDataProvider() {
+ return array(
+ // timestamp as string
+ array('February 13, 2009 at 11:31:30 PM GMT+0', 'en', 'datetime', '1234567890'),
+ array('13. Februar 2009 um 23:31:30 GMT+0', 'de', 'datetime', '1234567890'),
+ array('February 13, 2009', 'en', 'date', '1234567890'),
+ array('13. Februar 2009', 'de', 'date', '1234567890'),
+ array('11:31:30 PM GMT+0', 'en', 'time', '1234567890'),
+ array('23:31:30 GMT+0', 'de', 'time', '1234567890'),
+
+ // timestamp as int
+ array('February 13, 2009 at 11:31:30 PM GMT+0', 'en', 'datetime', 1234567890),
+ array('13. Februar 2009 um 23:31:30 GMT+0', 'de', 'datetime', 1234567890),
+ array('February 13, 2009', 'en', 'date', 1234567890),
+ array('13. Februar 2009', 'de', 'date', 1234567890),
+ array('11:31:30 PM GMT+0', 'en', 'time', 1234567890),
+ array('23:31:30 GMT+0', 'de', 'time', 1234567890),
+
+ // DateTime object
+ array('February 13, 2009 at 11:31:30 PM GMT+0', 'en', 'datetime', new DateTime('@1234567890')),
+ array('13. Februar 2009 um 23:31:30 GMT+0', 'de', 'datetime', new DateTime('@1234567890')),
+ array('February 13, 2009', 'en', 'date', new DateTime('@1234567890')),
+ array('13. Februar 2009', 'de', 'date', new DateTime('@1234567890')),
+ array('11:31:30 PM GMT+0', 'en', 'time', new DateTime('@1234567890')),
+ array('23:31:30 GMT+0', 'de', 'time', new DateTime('@1234567890')),
+ );
+ }
+
/**
- * Issue #4360: Do not call strtotime() on numeric strings.
+ * @dataProvider localizationDataProvider
*/
- public function testNumericStringToDateTime() {
+ public function testNumericStringLocalization($expectedDate, $lang, $type, $value) {
$l = new OC_L10N('test');
- $this->assertSame('February 13, 2009 23:31', $l->l('datetime', '1234567890'));
+ $l->forceLanguage($lang);
+ $this->assertSame($expectedDate, $l->l($type, $value));
+ }
+
+ public function firstDayDataProvider() {
+ return array(
+ array(1, 'de'),
+ array(0, 'en'),
+ );
}
- public function testNumericToDateTime() {
+ /**
+ * @dataProvider firstDayDataProvider
+ * @param $expected
+ * @param $lang
+ */
+ public function testFirstWeekDay($expected, $lang) {
$l = new OC_L10N('test');
- $this->assertSame('February 13, 2009 23:31', $l->l('datetime', 1234567890));
+ $l->forceLanguage($lang);
+ $this->assertSame($expected, $l->l('firstday', 'firstday'));
}
/**
generated by cgit v1.2.3 (git 2.39.1) at 2024-12-26 15:58:20 +0000