Aaron Schulz has submitted this change and it was merged. Change subject: ValidationStatistics: Showing unreviewed pages depends on config ......................................................................
ValidationStatistics: Showing unreviewed pages depends on config Special:UnreviewedPages does not exists on testwiki due to wgFlaggedRevsProtection = true Follow-Up: I63c2919587f858f91a2e9fe868b3cfb87b0650e0 Change-Id: I6c779b9ab425deaea9cab8ea4813563c095db889 --- M frontend/specialpages/reports/ValidationStatistics_body.php 1 file changed, 13 insertions(+), 4 deletions(-) Approvals: Aaron Schulz: Verified; Looks good to me, approved diff --git a/frontend/specialpages/reports/ValidationStatistics_body.php b/frontend/specialpages/reports/ValidationStatistics_body.php index 6bd25ed..48aef40 100644 --- a/frontend/specialpages/reports/ValidationStatistics_body.php +++ b/frontend/specialpages/reports/ValidationStatistics_body.php @@ -8,7 +8,7 @@ } public function execute( $par ) { - global $wgContLang, $wgFlaggedRevsStats; + global $wgContLang, $wgFlaggedRevsStats, $wgFlaggedRevsProtection; $out = $this->getOutput(); $lang = $this->getLanguage(); @@ -80,7 +80,10 @@ // validationstatistics-ns, validationstatistics-total, validationstatistics-stable, // validationstatistics-latest, validationstatistics-synced, validationstatistics-old, // validationstatistics-unreviewed - $msgs = array( 'ns', 'total', 'stable', 'latest', 'synced', 'old', 'unreviewed' ); // our headings + $msgs = array( 'ns', 'total', 'stable', 'latest', 'synced', 'old' ); // our headings + if ( !$wgFlaggedRevsProtection ) { + $msgs[] = 'unreviewed'; + } foreach ( $msgs as $msg ) { $out->addHTML( '<th>' . $this->msg( "validationstatistics-$msg" )->parse() . '</th>' ); @@ -149,14 +152,20 @@ array(), array( 'namespace' => $namespace ) ) . - "</td> + "</td>" + ); + if ( !$wgFlaggedRevsProtection ) { + $out->addHTML( " <td>" . Linker::linkKnown( SpecialPage::getTitleFor( 'UnreviewedPages' ), htmlspecialchars( $unreviewed ), array(), array( 'namespace' => $namespace ) ) . - "</td> + "</td>" + ); + } + $out->addHTML( " </tr>" ); } -- To view, visit https://gerrit.wikimedia.org/r/91205 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6c779b9ab425deaea9cab8ea4813563c095db889 Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/FlaggedRevs Gerrit-Branch: master Gerrit-Owner: Umherirrender <umherirrender_de...@web.de> Gerrit-Reviewer: Aaron Schulz <asch...@wikimedia.org> Gerrit-Reviewer: Alex Monk <kren...@gmail.com> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits