r89158 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r89157‎ | r89158 | r89159 >
Date:08:06, 30 May 2011
Author:aaron
Status:ok
Tags:
Comment:
Added function docs
Modified paths:
  • /trunk/extensions/FlaggedRevs/dataclasses/FlaggedRevision.php (modified) (history)

Diff [purge]

Index: trunk/extensions/FlaggedRevs/dataclasses/FlaggedRevision.php
@@ -794,6 +794,13 @@
795795 return ( $deleted || $updated );
796796 }
797797
 798+ /*
 799+ * Fetch pending template changes for this reviewed page
 800+ * version against a list of current versions of templates.
 801+ * See findPendingTemplateChanges() for details.
 802+ *
 803+ * @return array of (title, rev ID in reviewed version, has stable rev) tuples
 804+ */
798805 public function findTemplateChanges( array $newTemplates ) {
799806 if ( FlaggedRevs::inclusionSetting() == FR_INCLUDES_CURRENT ) {
800807 return array(); // short-circuit
@@ -822,6 +829,14 @@
823830 return $tmpChanges;
824831 }
825832
 833+ /*
 834+ * Fetch pending file changes for this reviewed page
 835+ * version against a list of current versions of files.
 836+ * See findPendingFileChanges() for details.
 837+ *
 838+ * @param string $noForeign Using 'noForeign' skips foreign file updates (bug 15748)
 839+ * @return array of (title, MW file timestamp in reviewed version, has stable rev) tuples
 840+ */
826841 public function findFileChanges( array $newFiles, $noForeign = false ) {
827842 if ( FlaggedRevs::inclusionSetting() == FR_INCLUDES_CURRENT ) {
828843 return array(); // short-circuit
@@ -846,7 +861,7 @@
847862 }
848863 return $fileChanges;
849864 }
850 -
 865+
851866 /**
852867 * Get flags for a revision
853868 * @param Title $title

Status & tagging log