r62536 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r62535‎ | r62536 | r62537 >
Date:18:49, 15 February 2010
Author:aaron
Status:ok
Tags:
Comment:
Follow-up r61741: made load() more resistant to infinite loops if bad code uses it
Modified paths:
  • /trunk/extensions/FlaggedRevs/FlaggedArticleView.php (modified) (history)

Diff [purge]

Index: trunk/extensions/FlaggedRevs/FlaggedArticleView.php
@@ -29,11 +29,11 @@
3030 */
3131 protected function load() {
3232 if ( !$this->loaded ) {
 33+ $this->loaded = true;
3334 $this->article = self::globalArticleInstance();
3435 if ( $this->article == null ) {
3536 throw new MWException( 'FlaggedArticleViewer has no context article!' );
3637 }
37 - $this->loaded = true;
3838 }
3939 }
4040

Past revisions this follows-up on

RevisionCommit summaryAuthorDate
r61741Clean up silly bug in load()aaron10:04, 31 January 2010

Status & tagging log