[] 54 wfGetDB() (use LoadBalancer::getConnection() on an injected instance of LoadBalancer instead)
[] $rc->getTitle() (use getPage() instead)
[] $article->mParserOutput
[] User::newFromID()
[] $page->doDeleteArticleReal() (Use DeletePage instead. Calling ::deleteIfAllowed and letting DeletePage handle permission checks is preferred over doing permission checks yourself and then calling ::deleteUnsafe. Note that DeletePage returns a good status with false value in case of scheduled deletion, instead of a status with a warning. Also, the new method doesn't have an $error parameter, since any error is added to the returned Status.)
[] $page->doUserEditContent() (use PageUpdater::saveRevision instead. Note that the new method expects callers to take care of checking EDIT_MINOR against the minoredit right, and to apply the autopatrol right as appropriate.)
[] Add these to static code analyzer CI? to prevent these from coming back
Note: Refactoring will be safer once we have some happy path Selenium tests up and running (T320924)