Description: ------------ LC_ALL should not be use in Driver_Holidays::factory(). Instead LC_MESSAGES and/or the environment variable LANG should be used. It should further be considered that the locale might have a charset attached, e.g. de_DE.UTF-8.

string setlocale ( int $category , string $locale [, string $... ] ) If $locale is "0", the locale setting is not affected, only the current setting is returned. from http://au.php.net/setlocale At the moment, the package does: function factory($driverId, $year = null, $locale = null, $external = false) { .. snip .. if (is_null($locale)) { $locale = setlocale(LC_ALL, 0); } $driver->setLocale($locale); } So, if you want to change locale, you can override your locale settings in the factory() method; mainly for testing purposes I guess. The default behaviour is to do setlocale(LC_ALL, 0); // a stupid looking way to write getlocale(); So; is there something broken? Test steps if so?

php -r 'echo setlocale(LC_ALL, 0);' LC_CTYPE=de_DE.UTF-8;LC_NUMERIC=C;LC_TIME=C;LC_COLLATE=C;LC_MONETARY=C;LC_MESSAGES=C;LC_PAPER=C;LC_NAME=C;LC_ADDRESS=C;LC_TELEPHONE=C;LC_MEASUREMENT=C;LC_IDENTIFICATION=C

This bug has been fixed in CVS. If this was a documentation problem, the fix will appear on pear.php.net by the end of next Sunday (CET). If this was a problem with the pear.php.net website, the change should be live shortly. Otherwise, the fix will appear in the package's next release. Thank you for the report and for helping us make PEAR better.

Because LC_MESSAGES is only available if PHP was compiled with libintl, it's existence should be checked before use. $localeCat = (defined(LC_MESSAGES)) ? LC_MESSAGES : LC_ALL; $locale = setlocale($localeCat, 0);

Of course, I actually meant: $localeCat = (defined('LC_MESSAGES')) ? LC_MESSAGES : LC_ALL; (with the quotes) in my previous comment.