r31782 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r31781‎ | r31782 | r31783 >
Date:14:18, 11 March 2008
Author:aaron
Status:old
Tags:
Comment:
*Don't autoreview outside the reviewable namespaces
Modified paths:
  • /trunk/extensions/FlaggedRevs/FlaggedArticle.php (modified) (history)

Diff [purge]

Index: trunk/extensions/FlaggedRevs/FlaggedArticle.php
@@ -890,6 +890,9 @@
891891
892892 if( $this->skipAutoReview || !$wgFlaggedRevsAutoReviewNew || !$user->isAllowed('review') )
893893 return true;
 894+ # Must be in reviewable namespace
 895+ if( !FlaggedRevs::isPageReviewable( $article->getTitle() ) )
 896+ return true;
894897 # Revision will be null for null edits
895898 if( !$rev ) {
896899 $this->skipReviewDiff = true; // Don't jump to diff...
@@ -917,6 +920,9 @@
918921
919922 if( $this->skipAutoReview || !$wgFlaggedRevsAutoReview || !$user->isAllowed('review') )
920923 return true;
 924+ # Must be in reviewable namespace
 925+ if( !FlaggedRevs::isPageReviewable( $article->getTitle() ) )
 926+ return true;
921927 # Revision will be null for null edits
922928 if( !$rev ) {
923929 $this->skipReviewDiff = true; // Don't jump to diff...
@@ -955,6 +961,9 @@
956962
957963 if( $this->skipAutoReview || !$wgFlaggedRevsAutoReview || !$user->isAllowed('review') )
958964 return true;
 965+ # Must be in reviewable namespace
 966+ if( !FlaggedRevs::isPageReviewable( $article->getTitle() ) )
 967+ return true;
959968 # Was this revision flagged?
960969 $frev = FlaggedRevs::getFlaggedRev( $article->getTitle(), $rev->getID() );
961970 if( is_null($frev) )

Status & tagging log