| Index: trunk/extensions/Translate/scripts/sync-group.php |
| — | — | @@ -190,7 +190,7 @@ |
| 191 | 191 | continue; |
| 192 | 192 | } |
| 193 | 193 | |
| 194 | | - // FIXME: temporary exception. Should be fixed elsewhere more generically |
| | 194 | + // @todo Temporary exception. Should be fixed elsewhere more generically. |
| 195 | 195 | if ( $translation == '{{PLURAL:GETTEXT|}}' ) { |
| 196 | 196 | return; |
| 197 | 197 | } |
| Index: trunk/extensions/Translate/scripts/poimport.php |
| — | — | @@ -181,9 +181,10 @@ |
| 182 | 182 | private $group = null; |
| 183 | 183 | |
| 184 | 184 | /** |
| 185 | | - * @param $changes Array of key/langcode => translation. |
| 186 | | - * @param $user User who makes the edits in wiki. |
| 187 | | - * @param $dryrun Don't do anything that affects the database. |
| | 185 | + * @param $changes \array Array of key/langcode => translation. |
| | 186 | + * @param $groupId \string Group ID. |
| | 187 | + * @param $user \string User who makes the edits in wiki. |
| | 188 | + * @param $dryrun \bool Do not do anything that affects the database. |
| 188 | 189 | */ |
| 189 | 190 | public function __construct( $changes, $groupId, $user, $dryrun = true ) { |
| 190 | 191 | $this->changes = $changes; |
| Index: trunk/extensions/Translate/scripts/autoexport.php |
| — | — | @@ -85,7 +85,7 @@ |
| 86 | 86 | $group = false; |
| 87 | 87 | $code = false; |
| 88 | 88 | |
| 89 | | - // TODO: fixme |
| | 89 | + // @todo Fixme |
| 90 | 90 | list( $pieces, ) = explode( '/', $wgContLang->lcfirst( $row->rc_title ), 2 ); |
| 91 | 91 | |
| 92 | 92 | $mg = TranslateUtils::messageKeyToGroup( $row->rc_namespace, $pieces ); |
| Index: trunk/extensions/Translate/tag/TranslatablePage.php |
| — | — | @@ -379,7 +379,7 @@ |
| 380 | 380 | /** |
| 381 | 381 | * Removes all page translation feature data from the database. |
| 382 | 382 | * Does not remove translated sections or translation pages. |
| 383 | | - * FIXME: Change name to something better. |
| | 383 | + * @todo Change name to something better. |
| 384 | 384 | */ |
| 385 | 385 | public function removeTags() { |
| 386 | 386 | $dbw = wfGetDB( DB_MASTER ); |
| Index: trunk/extensions/Translate/tag/PageTranslationHooks.php |
| — | — | @@ -141,8 +141,10 @@ |
| 142 | 142 | $article->doPurge(); |
| 143 | 143 | } |
| 144 | 144 | |
| | 145 | + /** |
| | 146 | + * @todo NOOP now. |
| | 147 | + */ |
| 145 | 148 | public static function addSidebar( $out, $tpl ) { |
| 146 | | - // TODO: fixme |
| 147 | 149 | /* |
| 148 | 150 | global $wgLang; |
| 149 | 151 | |
| — | — | @@ -152,7 +154,7 @@ |
| 153 | 155 | foreach ( $status as $code => $percent ) { |
| 154 | 156 | $name = TranslateUtils::getLanguageName( $code, false, $wgLang->getCode() ); |
| 155 | 157 | $percent = $wgLang->formatNum( round( 100 * $percent ) ); |
| 156 | | - $label = "$name ($percent%)"; // FIXME: i18n |
| | 158 | + $label = "$name ($percent%)"; // @todo i18n missing. |
| 157 | 159 | |
| 158 | 160 | $_title = TranslateTagUtils::codefyTitle( $title, $code ); |
| 159 | 161 | |
| — | — | @@ -221,8 +223,8 @@ |
| 222 | 224 | |
| 223 | 225 | $percent = Xml::element( 'img', array( |
| 224 | 226 | 'src' => TranslateUtils::assetPath( "images/prog-$image.png" ), |
| 225 | | - 'alt' => "$percent%", // FIXME: i18n |
| 226 | | - 'title' => "$percent%", // FIXME: i18n |
| | 227 | + 'alt' => "$percent%", // @todo i18n missing. |
| | 228 | + 'title' => "$percent%", // @todo i18n missing. |
| 227 | 229 | 'width' => '9', |
| 228 | 230 | 'height' => '9', |
| 229 | 231 | ) ); |
| Index: trunk/extensions/Translate/groups/ComplexMessages.php |
| — | — | @@ -458,7 +458,7 @@ |
| 459 | 459 | |
| 460 | 460 | // Remove duplicated entries, causes problems with magic words |
| 461 | 461 | // Just to be sure, it should not be possible to save invalid data anymore |
| 462 | | - $val = array_unique( $val /*FIXME:SORT_REGULAR*/ ); |
| | 462 | + $val = array_unique( $val /* @todo SORT_REGULAR */ ); |
| 463 | 463 | |
| 464 | 464 | // So do empty elements... |
| 465 | 465 | foreach ( $val as $k => $v ) { |
| Index: trunk/extensions/Translate/groups/MediaWiki/Checker.php |
| — | — | @@ -13,7 +13,7 @@ |
| 14 | 14 | * uses and vice versa. |
| 15 | 15 | */ |
| 16 | 16 | protected function wikiParameterCheck( $messages, $code, &$warnings ) { |
| 17 | | - // FIXME: gives false positive on (some?) languages for which a |
| | 17 | + // @todo Gives false positive on (some?) languages for which a |
| 18 | 18 | // parameter is completely left out. Example: |
| 19 | 19 | // http://translatewiki.net/w/i.php?title=MediaWiki:Configure-ext-ext-dependencies/zh-hant&action=edit |
| 20 | 20 | foreach ( $messages as $message ) { |
| — | — | @@ -220,7 +220,7 @@ |
| 221 | 221 | $warnings[$key][] = array( |
| 222 | 222 | array( 'miscmw', $subcheck, $key, $code ), |
| 223 | 223 | 'translate-checks-format', |
| 224 | | - "Parameter count is $traCount; should be $defCount", // FIXME: missing i18n. |
| | 224 | + "Parameter count is $traCount; should be $defCount", // @todo Missing i18n. |
| 225 | 225 | ); |
| 226 | 226 | continue; |
| 227 | 227 | } |
| — | — | @@ -234,7 +234,7 @@ |
| 235 | 235 | $warnings[$key][] = array( |
| 236 | 236 | array( 'miscmw', $subcheck, $key, $code ), |
| 237 | 237 | 'translate-checks-format', |
| 238 | | - "<nowiki>$traArray[$i]</nowiki> is malformed", // FIXME: missing i18n. |
| | 238 | + "<nowiki>$traArray[$i]</nowiki> is malformed", // @todo Missing i18n. |
| 239 | 239 | ); |
| 240 | 240 | continue; |
| 241 | 241 | } |
| Index: trunk/extensions/Translate/utils/TranslationStats.php |
| — | — | @@ -15,16 +15,16 @@ |
| 16 | 16 | * Returns translated percentage for message group in given |
| 17 | 17 | * languages |
| 18 | 18 | * |
| 19 | | - * @param $group String: Unique key identifying the group |
| 20 | | - * @param $languages Array: array of language codes |
| 21 | | - * @param $threshold Int: Minimum required percentage translated to |
| | 19 | + * @todo Add ability to return localised fuzzy percentage. |
| | 20 | + * @param $group \string Unique key identifying the group |
| | 21 | + * @param $languages \array List of language codes |
| | 22 | + * @param $threshold \int Minimum required percentage translated to |
| 22 | 23 | * return. Other given language codes will not be returned. |
| 23 | | - * @param $simple Bool: Return only codes or code/pecentage pairs |
| | 24 | + * @param $simple \bool Return only codes or code/pecentage pairs |
| 24 | 25 | * |
| 25 | | - * @return Array: array of key value pairs code (string)/percentage |
| | 26 | + * @return \array Array of key value pairs code (string)/percentage |
| 26 | 27 | * (float) or array of codes, depending on $simple |
| 27 | 28 | */ |
| 28 | | - // FIXME: add ability to return fuzzy percentage. |
| 29 | 29 | public static function getPercentageTranslated( $group, $languages, $threshold = false, $simple = false ) { |
| 30 | 30 | $stats = array(); |
| 31 | 31 | |
| Index: trunk/extensions/Translate/TranslateUtils.php |
| — | — | @@ -269,7 +269,7 @@ |
| 270 | 270 | /** |
| 271 | 271 | * Escapes the message, and does some mangling to whitespace, so that it is |
| 272 | 272 | * preserved when outputted as-is to html page. Line feeds are converted to |
| 273 | | - * <br /> and occurances of leading and trailing and multiple consecutive |
| | 273 | + * \<br /> and occurances of leading and trailing and multiple consecutive |
| 274 | 274 | * spaces to non-breaking spaces. |
| 275 | 275 | * |
| 276 | 276 | * @param $msg Plain text string. |
| Index: trunk/extensions/Translate/ffs/Wiki.php |
| — | — | @@ -15,8 +15,7 @@ |
| 16 | 16 | /** |
| 17 | 17 | * Reads all \@author tags from the file and returns array of authors. |
| 18 | 18 | * |
| 19 | | - * @param $filename From which file to get the authors. |
| 20 | | - * @return Array of authors. |
| | 19 | + * @return \array List of authors. |
| 21 | 20 | */ |
| 22 | 21 | public function parseAuthors() { |
| 23 | 22 | if ( $this->filename === false ) { |
| — | — | @@ -36,7 +35,7 @@ |
| 37 | 36 | |
| 38 | 37 | $contents = file_get_contents( $this->filename ); |
| 39 | 38 | |
| 40 | | - /** FIXME: handle the case where the first comment is missing */ |
| | 39 | + // @todo Handle the case where the first comment is missing */ |
| 41 | 40 | // $dollarstart = strpos( $contents, '$' ); |
| 42 | 41 | |
| 43 | 42 | $start = strpos( $contents, '*/' ); |
| — | — | @@ -149,7 +148,8 @@ |
| 150 | 149 | * Preprocesses MessageArray to suitable format and filters things that should |
| 151 | 150 | * not be exported. |
| 152 | 151 | * |
| 153 | | - * @param $array Reference of MessageArray. |
| | 152 | + * @param $messages MessageCollection Reference of MessageArray. |
| | 153 | + * @return Array of key-translation pairs. |
| 154 | 154 | */ |
| 155 | 155 | public function makeExportArray( MessageCollection $messages ) { |
| 156 | 156 | // We copy only relevant translations to this new array |