Quick hack to fix the broken hack noted in bug 15301
authorBrion Vibber <brion@users.mediawiki.org>
Tue, 26 Aug 2008 18:51:27 +0000 (18:51 +0000)
committerBrion Vibber <brion@users.mediawiki.org>
Tue, 26 Aug 2008 18:51:27 +0000 (18:51 +0000)
commite394f6a2d8ff89ad8d9d68dc3aa62ed2680e84e2
tree19552525812f8ace16c9870761e72ab374d56d3e
parentb03a2c72074d41dd7454a9b15e1e2b7a9ccf99c5
Quick hack to fix the broken hack noted in bug 15301

Use of:
$action_desc[0] = strtolower($action_desc[0]);
is horrible wrong for a few reasons:

1) The first-byte match fails for all non-ASCII characters -- use lcfirst()!
2) System strtolower() or lcfirst() may not work for UTF-8 or lang-specific issues properly; use $wgLang->lcfirst()
3) You shouldn't be forcing things to lowercase here anyway, you can't know it's correct gramattically.

This whole message output should be redone cleanly; either all the rights info messages should be designed to work in an inline list sentence, or they should be formatted as stand-alone text strings using a <ul> or something.
includes/OutputPage.php