Browse Source

intl message parser now handles too many or too less args

tags/2.0.0-beta
Carsten Brandt 11 years ago
parent
commit
fa9a975dbb
  1. 46
      framework/yii/i18n/MessageFormatter.php
  2. 32
      tests/unit/framework/i18n/MessageFormatterTest.php

46
framework/yii/i18n/MessageFormatter.php

@ -65,7 +65,8 @@ class MessageFormatter extends \MessageFormatter
{ {
$map = array_flip(array_keys($args)); $map = array_flip(array_keys($args));
// parsing http://icu-project.org/apiref/icu4c/classMessageFormat.html#details // parsing pattern base on ICU grammar:
// http://icu-project.org/apiref/icu4c/classMessageFormat.html#details
$parts = explode('{', $pattern); $parts = explode('{', $pattern);
$c = count($parts); $c = count($parts);
$pattern = $parts[0]; $pattern = $parts[0];
@ -73,26 +74,29 @@ class MessageFormatter extends \MessageFormatter
$stack = array(); $stack = array();
for($i = 1; $i < $c; $i++) { for($i = 1; $i < $c; $i++) {
if (preg_match('~^\A(\s*)([\d\w]+)(\s*)([},])(\s*)(.*)\z$~u', $parts[$i], $matches)) { if (preg_match('~^\A(\s*)([\d\w]+)(\s*)([},])(\s*)(.*)\z$~u', $parts[$i], $matches)) {
$d++; // if we are not inside a plural or select this is a message
// replace normal arg if it was set if (!isset($stack[$d]) || $stack[$d] != 'plural' && $stack[$d] != 'select') {
if (isset($map[$matches[2]])) { $d++;
$q = ''; // replace normal arg if it is available
$pattern .= '{' . $matches[1] . $map[$matches[2]] . $matches[3]; if (isset($map[$matches[2]])) {
} else { $q = '';
// quote unused args $pattern .= '{' . $matches[1] . $map[$matches[2]] . $matches[3];
$q = '';//($matches[4] == '}' && isset($stack[$d]) && !($stack[$d] == 'plural' || $stack[$d] == 'select')) ? "'" : ""; } else {
$pattern .= "$q{" . $matches[1] . $matches[2] . $matches[3]; // quote unused args
} $q = ($matches[4] == '}') ? "'" : "";
$pattern .= ($term = $matches[4] . $q . $matches[5] . $matches[6]); $pattern .= "$q{" . $matches[1] . $matches[2] . $matches[3];
// check type current level }
$stack[$d] = ($matches[4] == ',') ? substr($matches[6], 0, 6) : 'none'; $pattern .= ($term = $matches[4] . $q . $matches[5] . $matches[6]);
// if it's plural or select, the next bracket is NOT begin of a message then! // store type of current level
if ($stack[$d] == 'plural' || $stack[$d] == 'select') { $stack[$d] = ($matches[4] == ',') ? substr($matches[6], 0, 6) : 'none';
$i++; // if it's plural or select, the next bracket is NOT begin of a message then!
$d -= substr_count($term, '}'); if ($stack[$d] == 'plural' || $stack[$d] == 'select') {
} else { $i++;
$d -= substr_count($term, '}'); $d -= substr_count($term, '}');
continue; } else {
$d -= substr_count($term, '}');
continue;
}
} }
} }
$pattern .= '{' . $parts[$i]; $pattern .= '{' . $parts[$i];

32
tests/unit/framework/i18n/MessageFormatterTest.php

@ -65,6 +65,38 @@ _MSG_;
'gender' => 'male', 'gender' => 'male',
)); ));
$this->assertEquals('Alexander is male and he loves Yii!', $result); $this->assertEquals('Alexander is male and he loves Yii!', $result);
// verify pattern in select does not get replaced
$pattern = '{name} is {gender} and {gender, select, female{she} male{he} other{it}} loves Yii!';
$result = MessageFormatter::formatMessage('en_US', $pattern, array(
'name' => 'Alexander',
'gender' => 'male',
// following should not be replaced
'he' => 'wtf',
'she' => 'wtf',
'it' => 'wtf',
));
$this->assertEquals('Alexander is male and he loves Yii!', $result);
// verify pattern in select message gets replaced
$pattern = '{name} is {gender} and {gender, select, female{she} male{{he}} other{it}} loves Yii!';
$result = MessageFormatter::formatMessage('en_US', $pattern, array(
'name' => 'Alexander',
'gender' => 'male',
'he' => 'wtf',
'she' => 'wtf',
));
$this->assertEquals('Alexander is male and wtf loves Yii!', $result);
// some parser specific verifications
$pattern = '{gender} and {gender, select, female{she} male{{he}} other{it}} loves {nr, number} is {gender}!';
$result = MessageFormatter::formatMessage('en_US', $pattern, array(
'nr' => 42,
'gender' => 'male',
'he' => 'wtf',
'she' => 'wtf',
));
$this->assertEquals('male and wtf loves 42 is male!', $result);
} }
public function testInsufficientArguments() public function testInsufficientArguments()

Loading…
Cancel
Save