| Index: trunk/extensions/FlaggedRevs/FlaggedRevs.hooks.php |
| — | — | @@ -1689,17 +1689,18 @@ |
| 1690 | 1690 | } |
| 1691 | 1691 | |
| 1692 | 1692 | public static function addToHistLine( $history, $row, &$s, &$liClasses ) { |
| | 1693 | + $fa = FlaggedArticle::getArticleInstance( $history->getArticle() ); |
| | 1694 | + if( !$fa->isReviewable() ) { |
| | 1695 | + return true; // nothing to do here |
| | 1696 | + } |
| 1693 | 1697 | $title = $history->getArticle()->getTitle(); |
| 1694 | | - if( !FlaggedRevs::isPageReviewable( $title ) ) { |
| 1695 | | - return false; // nothing to do here |
| 1696 | | - } |
| 1697 | 1698 | # Fetch and process cache the stable revision |
| 1698 | 1699 | if( !isset($history->fr_stableRevId) ) { |
| 1699 | | - $frev = FlaggedRevision::newFromStable( $title ); |
| | 1700 | + $frev = $fa->getStableRev(); |
| 1700 | 1701 | $history->fr_stableRevId = $frev ? $frev->getRevId() : 0; |
| 1701 | 1702 | } |
| 1702 | 1703 | if( !$history->fr_stableRevId ) { |
| 1703 | | - return false; // nothing to do here |
| | 1704 | + return true; // nothing to do here |
| 1704 | 1705 | } |
| 1705 | 1706 | // Unreviewed revision: highlight if pending |
| 1706 | 1707 | $link = $class = ''; |
| — | — | @@ -1782,6 +1783,7 @@ |
| 1783 | 1784 | return true; |
| 1784 | 1785 | } |
| 1785 | 1786 | |
| | 1787 | + // diff=review param (bug 16923) |
| 1786 | 1788 | public static function checkDiffUrl( $titleObj, &$mOldid, &$mNewid, $old, $new ) { |
| 1787 | 1789 | if( $new === 'review' && isset($titleObj) ) { |
| 1788 | 1790 | $frev = FlaggedRevision::newFromStable( $titleObj ); |