| Index: trunk/extensions/FlaggedRevs/FlaggedRevs.hooks.php |
| — | — | @@ -619,7 +619,7 @@ |
| 620 | 620 | } |
| 621 | 621 | # All cases below require auto-review of edits to be enabled |
| 622 | 622 | if ( !FlaggedRevs::autoReviewEnabled() ) { |
| 623 | | - return true; |
| | 623 | + return true; // short-circuit |
| 624 | 624 | } |
| 625 | 625 | # If a $baseRevId is passed in, the edit is using an old revision's text |
| 626 | 626 | $isOldRevCopy = (bool)$baseRevId; // null edit or rollback |
| — | — | @@ -670,7 +670,7 @@ |
| 671 | 671 | # (a) this is a rollback to the stable version |
| 672 | 672 | # (b) this is a self-reversion to the stable version |
| 673 | 673 | # These are subcases of making a new revision based on an old, reviewed, revision. |
| 674 | | - } elseif ( $fa->getStableRev( FR_MASTER ) ) { |
| | 674 | + } elseif ( FlaggedRevs::autoReviewEdits() && $fa->getStableRev( FR_MASTER ) ) { |
| 675 | 675 | $srev = $fa->getStableRev(); |
| 676 | 676 | # Check for rollbacks... |
| 677 | 677 | $reviewableChange = ( |