Revert r35857, 35858, 35859 for the moment. Some notes:
authorBrion Vibber <brion@users.mediawiki.org>
Wed, 4 Jun 2008 16:56:31 +0000 (16:56 +0000)
committerBrion Vibber <brion@users.mediawiki.org>
Wed, 4 Jun 2008 16:56:31 +0000 (16:56 +0000)
commit0103d07da488529eecc94990cbbc641661c6017d
tree76ce263476ef2b2fc819f5130d05fc3e96e9d62b
parent07303079232cbe36bee515d37124848ea6063703
Revert r35857, 35858, 35859 for the moment. Some notes:

* Calling $user->isActiveUser() looks a bit redundant. :) Consider either shortening it to isActive() or changing the last word to be clearer, say $user->isActiveEditor()
* $wgActiveUserEditcount <- consider fully casing here; "editcount" isn't a word. :D -> $wgActiveUserEditCount
* $oldTime is dumped into SQL without any escaping or quoting. This will happen to work on MySQL but will fail on PostgreSQL, and is generally a bad practice. Use addQuotes() to ensure the formatted timestamp string is escaped and quoted for your raw SQL construct
* the database result object is not freed, which will leak resources if used in a long-running script
RELEASE-NOTES
includes/DefaultSettings.php
includes/User.php