Permalink
Browse files

jLocale - adds the possibility to have dash in a property name

  • Loading branch information...
1 parent f88f38e commit 55f2b209bba80f826f337bfce01467ed9ff80e96 @laurentj laurentj committed Oct 29, 2012
View
2 ext/jelix/jelix.c
@@ -1126,7 +1126,7 @@ PHP_FUNCTION(jelix_scan_locale_sel)
} else if(!( ( *cursor >= 'a' && *cursor <= 'z')
|| ( *cursor >= 'A' && *cursor <= 'Z')
|| ( *cursor >= '0' && *cursor <= '9')
- || *cursor == '_' || *cursor == '.')){
+ || *cursor == '_' || *cursor == '-' || *cursor == '.')){
RETURN_FALSE;
}
resource_length ++;
View
2 lib/jelix/core/jelix_api.php
@@ -95,7 +95,7 @@ function jelix_scan_class_sel($selStr, $selObj) {
}
function jelix_scan_locale_sel($selStr, $selObj) {
- if (preg_match("/^(([a-zA-Z0-9_\.]+)~)?([a-zA-Z0-9_]+)\.([a-zA-Z0-9_\.]+)$/", $selStr, $m)) {
+ if (preg_match("/^(([a-zA-Z0-9_\.]+)~)?([a-zA-Z0-9_]+)\.([a-zA-Z0-9_\-\.]+)$/", $selStr, $m)) {
if ($m[1]!='' && $m[2]!='') {
$selObj->module = $m[2];
}
View
3 testapp/modules/jelix_tests/locales/fr_FR/tests1.UTF-8.properties
@@ -4,4 +4,5 @@ multiline.locale.with.accent = Chaîne à\
tester
multiline.locale.with.accent2 = Chaîne à\
tester à \
- foison
+ foison
+first-dash-locale = ceci est une phrase fr_FR avec tiret
View
1 testapp/tests-jelix/jelix/core/jLocaleTest.php
@@ -92,6 +92,7 @@ public function testBundle(){
function testSimpleLocale(){
jApp::config()->locale = 'fr_FR';
$this->assertEquals('ceci est une phrase fr_FR',jLocale::get('tests1.first.locale'));
+ $this->assertEquals('ceci est une phrase fr_FR avec tiret',jLocale::get('tests1.first-dash-locale'));
$this->assertEquals('ceci est une phrase fr_FR',jLocale::get('tests1.first.locale', null, 'fr_FR'));
$this->assertEquals('Chaîne à tester',jLocale::get('tests1.multiline.locale.with.accent'));
$this->assertEquals('Chaîne à tester à foison',jLocale::get('tests1.multiline.locale.with.accent2'));

0 comments on commit 55f2b20

Please sign in to comment.