For instance, the getLocale() function should be modified similar to this:
public function getLocale()
{
// Zend version incorrectly ignores HTTP_ACCEPT_LANGUAGE value when create locale object
if (null === $this->_locale) {
$options = $this->getOptions();
if (isset($options['default'])) { // Set the fall-back locale Zend_Locale::setDefault($options['default']); } else { // Should the web server's locale be set as a default when none provided? }
// Now create the locale with automatic locale discovery enabled (the default method)
$this->_locale = new Zend_Locale();
$key = (isset($options['registry_key']) && !is_numeric($options['registry_key']))
? $options['registry_key']
: self::DEFAULT_REGISTRY_KEY;
Zend_Registry::set($key, $this->_locale);
}
return $this->_locale;
}
Comments
Posted by Jolly Blume (jollyblume) on 2009-12-01T12:16:58.000+0000
When the application.ini file sets a default (fall-back) locale, Zend_Application_Resource_Locale incorrectly creates the locale object with it. This default should be set before creating the locale object and then create the object using automatic locale discovery.
For instance, the getLocale() function should be modified similar to this: public function getLocale() { // Zend version incorrectly ignores HTTP_ACCEPT_LANGUAGE value when create locale object if (null === $this->_locale) { $options = $this->getOptions(); if (isset($options['default'])) { // Set the fall-back locale Zend_Locale::setDefault($options['default']); } else { // Should the web server's locale be set as a default when none provided? } // Now create the locale with automatic locale discovery enabled (the default method) $this->_locale = new Zend_Locale();
Posted by Dolf Schimmel (Freeaqingme) (freak) on 2009-12-01T12:23:51.000+0000
I edited the issue itself a bit, so that the issue itself makes immediately clear what the supposed problem would be, please check if I summarized and interpreted your problem correctly.
Posted by Jolly Blume (jollyblume) on 2009-12-02T21:30:47.000+0000
Here is some code notes:
[configs/application.ini] resources.locale.default = en_US
[index/index.phtml] <?php echo '
HTTP_ACCEPT_LANGUAGE: ' . Zend_Controller_Front::getInstance()->getRequest()->getServer('HTTP_ACCEPT_LANGUAGE'); $locale = Zend_Registry::get('Zend_Locale'); $defaultLocale = $locale->getDefault(); foreach ($defaultLocale as $key => $value) { echo '
Default locale: ' . $key . ' (' . $value. ')'; } echo '
Language: ' . $locale->getLanguage(); echo '
Region: ' . $locale->getRegion(); ?>
[output with Zend version of Locale resource's getLocale()] HTTP_ACCEPT_LANGUAGE: de-de,en-us;q=0.7,en;q=0.3 Default locale: en_US (1) Language: en Region: US
(Notice that even though the preferred language in HTTP_ACCEPT_LANGUAGE is "de-DE", the language set in the Locale is "en_US". This is because the existing Locale resource sets the locale language to the default setting, instead of allowing it to detect the language using Zend_Locale automatic locale discovery)
[output using getLocale() based on code in the above comment] HTTP_ACCEPT_LANGUAGE: de-de,en-us;q=0.7,en;q=0.3 Default locale: en_US (1) Language: de Region: DE
(Notice that the Locale created by the modified Locale resource is correctly set to "de_DE" because the locale is being created using automatic locale discovery. Automatic locale discovery ($locale = new Zend_Locale();) will detect the locale by first looking at HTTP_ACCEPT_LANGUAGE. If automatic discovery fails, it will ultimately use the locale default.)
Posted by Jolly Blume (jollyblume) on 2009-12-02T21:31:55.000+0000
Also, this problem exists in the current (1.9.4) release.
Posted by Jolly Blume (jollyblume) on 2009-12-02T21:33:35.000+0000
Ignore the previous comment. It is a problem in the current release (1.9.6)
Posted by Jolly Blume (jollyblume) on 2009-12-02T21:48:07.000+0000
I just checkout the SVN trunk code. The problem exists in trunk.
Note: this is a problem with Zend_Application_Resource_Locale::getLocale().
Posted by Dolf Schimmel (Freeaqingme) (freak) on 2010-01-02T16:33:05.000+0000
Should be fixed now, thank you for reporting. If you still encounter this issue, please reopen it (and add a comment).