Standardize @todo

Always use lowercase @todo
Changed some "@FIXME" to "@todo FIXME: "

Change-Id: I016d1f9511e6f3bbd22427f0fffeccd77ab5e0db

includes/MediaWiki.php

@@ -558,7 +558,7 @@ class MediaWiki {
 					// to fail due to post data being lost, but let's try anyway
 					// and just log the instance.
 					//
-					// @todo @fixme See if we could issue a 307 or 308 here, need
+					// @todo FIXME: See if we could issue a 307 or 308 here, need
 					// to see how clients (automated & browser) behave when we do
 					wfDebugLog( 'RedirectedPosts', "Redirected from HTTP to HTTPS: $oldUrl" );
 				}

includes/cache/LocalisationCache.php

@@ -1186,7 +1186,7 @@ class LCStoreDB implements LCStore {
 
 		// We must keep a separate connection to MySQL in order to avoid breaking
 		// main transactions. However, SQLite deadlocks when using two connections.
-		// @TODO: get this trick to work on PostgreSQL too
+		// @todo get this trick to work on PostgreSQL too
 		if ( wfGetDB( DB_MASTER )->getType() == 'mysql' ) {
 			$lb = wfGetLBFactory()->newMainLB();
 			$this->dbw = $lb->getConnection( DB_MASTER );

includes/content/ContentHandler.php

@@ -837,7 +837,7 @@ abstract class ContentHandler {
 	 * @return mixed String containing deletion reason or empty string, or
 	 *    boolean false if no revision occurred
 	 *
-	 * @XXX &$hasHistory is extremely ugly, it's here because
+	 * @todo &$hasHistory is extremely ugly, it's here because
 	 * WikiPage::getAutoDeleteReason() and Article::generateReason()
 	 * have it / want it.
 	 */

includes/diff/DifferenceEngine.php

@@ -972,7 +972,7 @@ class DifferenceEngine extends ContextSource {
 		}
 
 		// Sanity: don't show the notice if too many rows must be scanned
-		// @TODO: show some special message for that case
+		// @todo show some special message for that case
 		$nEdits = $this->mNewPage->countRevisionsBetween( $oldRev, $newRev, 1000 );
 		if ( $nEdits > 0 && $nEdits <= 1000 ) {
 			$limit = 100; // use diff-multi-manyusers if too many users

includes/filebackend/FileBackendStore.php

@@ -69,7 +69,7 @@ abstract class FileBackendStore extends FileBackend {
 		$this->mimeCallback = isset( $config['mimeCallback'] )
 			? $config['mimeCallback']
 			: function ( $storagePath, $content, $fsPath ) {
-				// @TODO: handle the case of extension-less files using the contents
+				// @todo handle the case of extension-less files using the contents
 				return StreamFile::contentTypeFromPath( $storagePath ) ?: 'unknown/unknown';
 			};
 		$this->memCache = new EmptyBagOStuff(); // disabled by default

includes/filebackend/lockmanager/DBLockManager.php

@@ -108,7 +108,7 @@ abstract class DBLockManager extends QuorumLockManager {
 		$this->session = wfRandomString( 31 );
 	}
 
-	// @TODO: change this code to work in one batch
+	// @todo change this code to work in one batch
 	protected function getLocksOnServer( $lockSrv, array $pathsByType ) {
 		$status = Status::newGood();
 		foreach ( $pathsByType as $type => $paths ) {

includes/objectcache/SqlBagOStuff.php

@@ -145,7 +145,7 @@ class SqlBagOStuff extends BagOStuff {
 			} else {
 				// We must keep a separate connection to MySQL in order to avoid deadlocks
 				// However, SQLite has an opposite behavior.
-				// @TODO: get this trick to work on PostgreSQL too
+				// @todo get this trick to work on PostgreSQL too
 				if ( wfGetDB( DB_MASTER )->getType() == 'mysql' ) {
 					$lb = wfGetLBFactory()->newMainLB();
 					$db = $lb->getConnection( DB_MASTER );

includes/profiler/Profiler.php

@@ -397,7 +397,7 @@ class TransactionProfiler {
 	public function recordFunctionCompletion( $method, $realtime ) {
 		if ( !$this->mDBTrxHoldingLocks ) {
 			return; // short-circuit
-		// @TODO: hardcoded check is a tad janky (what about FOR UPDATE?)
+		// @todo hardcoded check is a tad janky (what about FOR UPDATE?)
 		} elseif ( !preg_match( '/^query-m: (?!SELECT)/', $method )
 			&& $realtime < $this->mDBLockThreshold
 		) {

includes/specials/SpecialChangePassword.php

@@ -260,7 +260,7 @@ class SpecialChangePassword extends FormSpecialPage {
 			);
 		}
 
-		// @TODO Make these separate messages, since the message is written for both cases
+		// @todo Make these separate messages, since the message is written for both cases
 		if ( !$user->checkTemporaryPassword( $oldpass ) && !$user->checkPassword( $oldpass ) ) {
 			wfRunHooks( 'PrefsPasswordAudit', array( $user, $newpass, 'wrongpassword' ) );
 			throw new PasswordError( $this->msg( 'resetpass-wrong-oldpass' )->text() );

includes/specials/SpecialListusers.php

@@ -193,7 +193,7 @@ class UsersPager extends AlphabeticPager {
 		$edits = '';
 		global $wgEdititis;
 		if ( !$this->including && $wgEdititis ) {
-			// @fixme i18n issue: Hardcoded square brackets.
+			// @todo fixme i18n issue: Hardcoded square brackets.
 			$edits = ' [' .
 				$this->msg( 'usereditcount' )->numParams( $row->edits )->escaped() .
 				']';

includes/specials/SpecialMergeHistory.php

@@ -477,7 +477,7 @@ class SpecialMergeHistory extends SpecialPage {
 			array( $destTitle->getPrefixedText(), $timestampLimit ), $this->getUser()
 		);
 
-		# @TODO: message should use redirect=no
+		# @todo message should use redirect=no
 		$this->getOutput()->addWikiMsg( 'mergehistory-success',
 			$targetTitle->getPrefixedText(), $destTitle->getPrefixedText(), $count );
 

languages/Language.php

@@ -2396,7 +2396,7 @@ class Language {
 			// Timestamps are in different years: use full timestamp
 			// Also do full timestamp for future dates
 			/**
-			 * @FIXME Add better handling of future timestamps.
+			 * @todo FIXME: Add better handling of future timestamps.
 			 */
 			$format = $this->getDateFormatString( 'both', $user->getDatePreference() ?: 'default' );
 			$ts = $this->sprintfDate( $format, $ts->getTimestamp( TS_MW ) );

maintenance/eraseArchivedFile.php

@@ -27,7 +27,7 @@ require_once __DIR__ . '/Maintenance.php';
 /**
  * Maintenance script to delete archived (non-current) files from storage.
  *
- * @TODO: Maybe add some simple logging
+ * @todo Maybe add some simple logging
  *
  * @ingroup Maintenance
  * @since 1.22

maintenance/purgeChangedPages.php

@@ -160,7 +160,7 @@ class PurgeChangedPages extends Maintenance {
 	 * If this returns an empty array for a non-empty query result, then all the rows
 	 * had the same column value and the query should be repeated with a higher LIMIT.
 	 *
-	 * @TODO: move this elsewhere
+	 * @todo move this elsewhere
 	 *
 	 * @param ResultWrapper $res Query result sorted by $column (ascending)
 	 * @param string $column

tests/phpunit/includes/GlobalFunctions/GlobalTest.php

@@ -721,5 +721,5 @@ class GlobalTest extends MediaWikiTestCase {
 			),
 		);
 	}
-	/* @TODO many more! */
+	/* @todo many more! */
 }

tests/phpunit/includes/MWNamespaceTest.php

@@ -9,7 +9,7 @@
  * Test class for MWNamespace.
  * Generated by PHPUnit on 2011-02-20 at 21:01:55.
  * @todo covers tags
- * @FIXME this test file is a mess
+ * @todo FIXME: this test file is a mess
  *
  */
 class MWNamespaceTest extends MediaWikiTestCase {

tests/phpunit/includes/SanitizerValidateEmailTest.php

@@ -2,7 +2,7 @@
 
 /**
  * @covers Sanitizer::validateEmail
- * @TODO all test methods in this class should be refactored and...
+ * @todo all test methods in this class should be refactored and...
  *    use a single test method and a single data provider...
  */
 class SanitizerValidateEmailTest extends MediaWikiTestCase {