| Novem_Linguae | |
| Nov 18 2022, 10:33 PM |
| F35796194: image.png | |
| Nov 18 2022, 10:33 PM |
I assume these...
private static function shouldNoIndexForNewArticleReasons( WikiPage $page ) {
global $wgPageTriageNoIndexUnreviewedNewArticles, $wgPageTriageMaxAge;
Should be changed to something like...
$config = MediaWikiServices::getInstance()->getMainConfig(); $pageTriageDraftNamespaceId = $config->get( 'PageTriageDraftNamespaceId' );
@Novem_Linguae yeah that would be good. Even better would be to pass Config $config to the static method (or pass e.g. int $PageTriageMaxAge). Ideally the method is passed whatever it needs to do its work, rather than "reaching outside" (e.g. invoke the service to get the config object); that tends to help with readability & makes testing easier as well.