Index: trunk/extensions/FlaggedRevs/FlaggedArticle.php |
— | — | @@ -8,7 +8,6 @@ |
9 | 9 | # Process cache variables |
10 | 10 | protected $stableRev = null; |
11 | 11 | protected $pageConfig = null; |
12 | | - protected $flags = array(); |
13 | 12 | |
14 | 13 | /** |
15 | 14 | * Get a FlaggedArticle for a given title |
— | — | @@ -106,7 +105,6 @@ |
107 | 106 | $srev = FlaggedRevision::newFromStable( $this->getTitle(), $flags ); |
108 | 107 | if ( $srev ) { |
109 | 108 | $this->stableRev = $srev; |
110 | | - $this->flags[$srev->getRevId()] = $srev->getTags(); |
111 | 109 | return $srev; |
112 | 110 | } else { |
113 | 111 | $this->stableRev = false; |
Index: trunk/extensions/FlaggedRevs/FlaggedArticleView.php |
— | — | @@ -120,7 +120,7 @@ |
121 | 121 | * Output review notice |
122 | 122 | */ |
123 | 123 | public function displayTag() { |
124 | | - global $wgOut, $wgRequest; |
| 124 | + global $wgOut; |
125 | 125 | $this->load(); |
126 | 126 | // Sanity check that this is a reviewable page |
127 | 127 | if ( $this->article->isReviewable() ) { |
— | — | @@ -182,7 +182,7 @@ |
183 | 183 | * Adds a quick review form on the bottom if needed |
184 | 184 | */ |
185 | 185 | public function setPageContent( &$outputDone, &$pcache ) { |
186 | | - global $wgRequest, $wgOut, $wgUser, $wgLang, $wgContLang; |
| 186 | + global $wgRequest, $wgOut, $wgLang, $wgContLang; |
187 | 187 | $this->load(); |
188 | 188 | # Only trigger on article view for content pages, not for protect/delete/hist... |
189 | 189 | $action = $wgRequest->getVal( 'action', 'view' ); |
— | — | @@ -596,7 +596,7 @@ |
597 | 597 | * @returns bool, diff added to output |
598 | 598 | */ |
599 | 599 | protected function maybeShowTopDiff( $srev, $quality ) { |
600 | | - global $wgUser, $wgOut, $wgMemc; |
| 600 | + global $wgUser, $wgOut; |
601 | 601 | $this->load(); |
602 | 602 | if ( !$wgUser->getBoolOption( 'flaggedrevsviewdiffs' ) ) |
603 | 603 | return false; // nothing to do here |