[16:42:40] Reedy: Non-query modules would generally call ApiMain->setCacheMode() to set the cache mode. ApiQueryBase::getCacheMode() is used by ApiQuery::execute() in determining what it should pass to ApiMain->setCacheMode(). [16:43:18] So yeah, the function is doing nothing [16:44:10] So whether we need to replace it with a setCacheMode call instead... [16:45:55] The default if setCacheMode() isn't called is 'private'. You'd only need to set a different mode if you want to allow clients to request cache headers be set. [16:46:56] So that's a no, just remove that redundant function. Cheers :) [16:47:14] (ApiQuery, on the other hand, effectively defaults to 'public') [16:47:29] Yeah, which makes sense [17:06:54] https://gerrit.wikimedia.org/r/#/c/mediawiki/extensions/OATHAuth/+/550513/ removes it [22:54:10] with the retirement of mwrepl in production, is there any known methods for setting breakpoints and debugging php7 in prod on mwdebug* ?