User:Aaron Schulz/code review

Code review (core and WMF extensions) (Up to revision 36861):


 * Major (OK): [36631,36632,36637,36642,36728], 36668 (hard to avoid duplication), [36682,36727], [36758,36760]
 * Major (check): 36648 (looks fine), [36664,36752], [36772 - 36844] catTree stuff
 * Iffy: 36600 (by ref?), 36608 (???), 36692 (won't this block the same IP several times?, what is the use of $ignoreErrors?, $editor in getBaseRevision looks bogus)
 * Things to look at: 36730, [36796,36797], 36802, 36814

Note: 36685 reverted by committer, OK? (other changes pending anyway)

Still need to look more at CategoryTree stuff and look closer at RC changes. Prolly should go on testwp first.