Fixes for r28797.
authorAryeh Gregor <simetrical@users.mediawiki.org>
Sun, 23 Dec 2007 19:53:49 +0000 (19:53 +0000)
committerAryeh Gregor <simetrical@users.mediawiki.org>
Sun, 23 Dec 2007 19:53:49 +0000 (19:53 +0000)
commit1ed4c2f2a3035a3b0d77f7e227cd974784512a29
tree24bd66f09f2dc7064914ba4c47ba66bc38ee9c62
parentb503237a8b81d85949c8ccc3d9f72e6037de969a
Fixes for r28797.
* Mark private methods private using a keyword.
* Reject arrays with count == 2: these will fail when you do array_slice( ... , 1 ).
* Treat xor consistent with the other operations: if there's only one parameter the result should just evaluate that, not always return false; and any number of parameters should be allowed.
* Fail fast on bad input: throw an exception if Autopromote encounters a condition it can't understand (after asking extensions).
* Code documentation!  There were five lines of comments in the original commit.
* APCONDS_INGROUPS is not used, or for that matter defined.
* Editcount should use >=, not >, for consistency with past behavior and intuitiveness.
* "autopromoteUser" sounds like it's actually promoting the user somehow.  Renamed the function to getAutopromoteGroups.
* Make sure we don't return the same group more than once, when we're returning a group.  Probably not going to hurt, but may as well be clean.
RELEASE-NOTES
includes/Autopromote.php
includes/DefaultSettings.php
includes/Defines.php
includes/User.php