converts '@fixme' to '@todo FIXME'
@fixme is simply not recognized by doxygen whereas @todo is used to generate a nice ... todo list!! Change-Id: If956c0a164373126ce48b791d45c56962034eecd
This commit is contained in:
parent
d5737f8f17
commit
12e608e3d5
12 changed files with 14 additions and 13 deletions
|
|
@ -80,8 +80,8 @@ class Cookie {
|
|||
* A better method might be to use a blacklist like
|
||||
* http://publicsuffix.org/
|
||||
*
|
||||
* @fixme fails to detect 3-letter top-level domains
|
||||
* @fixme fails to detect 2-letter top-level domains for single-domain use (probably not a big problem in practice, but there are test cases)
|
||||
* @todo fixme fails to detect 3-letter top-level domains
|
||||
* @todo fixme fails to detect 2-letter top-level domains for single-domain use (probably not a big problem in practice, but there are test cases)
|
||||
*
|
||||
* @param $domain String: the domain to validate
|
||||
* @param $originDomain String: (optional) the domain the cookie originates from
|
||||
|
|
|
|||
|
|
@ -156,7 +156,7 @@ class Http {
|
|||
*
|
||||
* file:// should not be allowed here for security purpose (r67684)
|
||||
*
|
||||
* @fixme this is wildly inaccurate and fails to actually check most stuff
|
||||
* @todo FIXME this is wildly inaccurate and fails to actually check most stuff
|
||||
*
|
||||
* @param $uri Mixed: URI to check for validity
|
||||
* @return Boolean
|
||||
|
|
|
|||
|
|
@ -939,7 +939,7 @@ class Preferences {
|
|||
|
||||
if ( $wgEnableAPI ) {
|
||||
# Some random gibberish as a proposed default
|
||||
// @fixme This should use CryptRand but we may not want to read urandom on every view
|
||||
// @todo Fixme: this should use CryptRand but we may not want to read urandom on every view
|
||||
$hash = sha1( mt_rand() . microtime( true ) );
|
||||
|
||||
$defaultPreferences['watchlisttoken'] = array(
|
||||
|
|
|
|||
|
|
@ -363,7 +363,7 @@ abstract class Skin extends ContextSource {
|
|||
* inside ->getOutput() is deprecated. The $out arg is kept
|
||||
* for compatibility purposes with skins.
|
||||
* @param $out OutputPage
|
||||
* @delete
|
||||
* @todo delete
|
||||
*/
|
||||
abstract function setupSkinUserCss( OutputPage $out );
|
||||
|
||||
|
|
|
|||
|
|
@ -359,7 +359,7 @@ class LogEventsList extends ContextSource {
|
|||
}
|
||||
|
||||
/**
|
||||
* @TODO: split up!
|
||||
* @todo split up!
|
||||
*
|
||||
* @param $row
|
||||
* @param Title $title
|
||||
|
|
|
|||
|
|
@ -261,7 +261,7 @@ class BitmapHandler extends ImageHandler {
|
|||
* @param $params array Array with scaler params
|
||||
* @return ThumbnailImage
|
||||
*
|
||||
* @fixme no rotation support
|
||||
* @todo fixme: no rotation support
|
||||
*/
|
||||
protected function getClientScalingThumbnailImage( $image, $params ) {
|
||||
return new ThumbnailImage( $image, $image->getURL(),
|
||||
|
|
|
|||
|
|
@ -199,7 +199,8 @@ class ExifBitmapHandler extends BitmapHandler {
|
|||
*
|
||||
* @param string $data
|
||||
* @return int 0, 90, 180 or 270
|
||||
* @fixme orientation can include flipping as well; see if this is an issue!
|
||||
* @todo FIXME orientation can include flipping as well; see if this is an
|
||||
* issue!
|
||||
*/
|
||||
protected function getRotationForExif( $data ) {
|
||||
if ( !$data ) {
|
||||
|
|
|
|||
|
|
@ -3907,7 +3907,7 @@ class Language {
|
|||
/**
|
||||
* Decode an expiry (block, protection, etc) which has come from the DB
|
||||
*
|
||||
* @FIXME: why are we returnings DBMS-dependent strings???
|
||||
* @todo FIXME: why are we returnings DBMS-dependent strings???
|
||||
*
|
||||
* @param $expiry String: Database expiry String
|
||||
* @param $format Bool|Int true to process using language functions, or TS_ constant
|
||||
|
|
|
|||
|
|
@ -35,7 +35,7 @@ class JSParseHelper extends Maintenance {
|
|||
if ( $this->hasArg() ) {
|
||||
$files = $this->mArgs;
|
||||
} else {
|
||||
$this->maybeHelp( true ); // @fixme this is a lame API :)
|
||||
$this->maybeHelp( true ); // @todo fixme this is a lame API :)
|
||||
exit( 1 ); // it should exit from the above first...
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -189,7 +189,7 @@ class ImageBuilder extends Maintenance {
|
|||
$filename = $altname;
|
||||
$this->output( "Estimating transcoding... $altname\n" );
|
||||
} else {
|
||||
# @FIXME: create renameFile()
|
||||
# @todo FIXME: create renameFile()
|
||||
$filename = $this->renameFile( $filename );
|
||||
}
|
||||
}
|
||||
|
|
|
|||
|
|
@ -478,7 +478,7 @@ more stuff
|
|||
$this->assertEquals( $expected, $text );
|
||||
}
|
||||
|
||||
/* @FIXME: fix this!
|
||||
/* @todo FIXME: fix this!
|
||||
public function testGetUndoText() {
|
||||
global $wgDiff3;
|
||||
|
||||
|
|
|
|||
|
|
@ -596,7 +596,7 @@ class NewParserTest extends MediaWikiTestCase {
|
|||
* Run a fuzz test series
|
||||
* Draw input from a set of test files
|
||||
*
|
||||
* @todo @fixme Needs some work to not eat memory until the world explodes
|
||||
* @todo fixme Needs some work to not eat memory until the world explodes
|
||||
*
|
||||
* @group ParserFuzz
|
||||
*/
|
||||
|
|
|
|||
Loading…
Reference in a new issue