r71361 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r71360‎ | r71361 | r71362 >
Date:14:54, 20 August 2010
Author:siebrand
Status:deferred
Tags:
Comment:
Fix doxygen issues.
Modified paths:
  • /trunk/extensions/Translate/MessageGroups.php (modified) (history)
  • /trunk/extensions/Translate/SpecialTranslationChanges.php (modified) (history)
  • /trunk/extensions/Translate/Translate.php (modified) (history)
  • /trunk/extensions/Translate/TranslateEditAddons.php (modified) (history)
  • /trunk/extensions/Translate/tag/SpecialPageTranslation.php (modified) (history)

Diff [purge]

Index: trunk/extensions/Translate/MessageGroups.php
@@ -857,7 +857,7 @@
858858 if ( $code === 'en' ) {
859859 $stuff = $this->load( 'en' );
860860 /**
861 - * @todo Throws PHP Notice: Undefined index: <key>
 861+ * @todo Throws PHP Notice: Undefined index: \<key>
862862 * when keys are added, but createMessageIndex.php is
863863 * not run (like when a translatable page from page
864864 * translation was added).
Index: trunk/extensions/Translate/tag/SpecialPageTranslation.php
@@ -204,7 +204,7 @@
205205 continue;
206206 }
207207
208 - /* Ignore pages which have had <translate> at some point, but which
 208+ /* Ignore pages which have had \<translate> at some point, but which
209209 * have never been marked. */
210210 if ( $page['tp:tag'] !== $page['latest'] ) {
211211 unset( $pages[$index] );
Index: trunk/extensions/Translate/SpecialTranslationChanges.php
@@ -228,7 +228,7 @@
229229 * Generate HTML for an arrow or placeholder graphic
230230 * @param $dir \string One of '', 'd', 'l', 'r'
231231 * @param $alt \string Alt text
232 - * @return string HTML "img" tag
 232+ * @return string HTML \<img> tag
233233 */
234234 private function arrow( $dir, $alt = '' ) {
235235 global $wgStylePath;
@@ -240,7 +240,7 @@
241241 /**
242242 * Generate HTML for a right- or left-facing arrow,
243243 * depending on language direction.
244 - * @return string HTML "img" tag
 244+ * @return string HTML \<img> tag
245245 */
246246 private function sideArrow() {
247247 global $wgContLang;
@@ -251,7 +251,7 @@
252252 /**
253253 * Generate HTML for a down-facing arrow
254254 * depending on language direction.
255 - * @return string HTML <img> tag
 255+ * return string HTML \<img> tag
256256 */
257257 private function downArrow() {
258258 return $this->arrow( 'd', '-' );
Index: trunk/extensions/Translate/TranslateEditAddons.php
@@ -257,7 +257,7 @@
258258 * Check if a string contains the fuzzy string.
259259 *
260260 * @param $text \string Arbitrary text
261 - * return \bool If string contains fuzzy string.
 261+ * @return \bool If string contains fuzzy string.
262262 */
263263 public static function hasFuzzyString( $text ) {
264264 return strpos( $text, TRANSLATE_FUZZY ) !== false;
@@ -266,7 +266,7 @@
267267 /** Check if a title is marked as fuzzy.
268268 *
269269 * @param $title Title
270 - * return \bool If title is marked fuzzy.
 270+ * @return \bool If title is marked fuzzy.
271271 */
272272 public static function isFuzzy( Title $title ) {
273273 $dbr = wfGetDB( DB_SLAVE );
@@ -291,7 +291,7 @@
292292 /** Check if a title is in a message namespace.
293293 *
294294 * @param $title Title
295 - * return \bool If title is in a message namespace.
 295+ * @return \bool If title is in a message namespace.
296296 */
297297 public static function isMessageNamespace( Title $title ) {
298298 global $wgTranslateMessageNamespaces; ;
Index: trunk/extensions/Translate/Translate.php
@@ -506,17 +506,17 @@
507507 # $wgHooks['SkinTemplateOutputPageBeforeExec'][] = 'TranslateTagHooks::addSidebar';
508508
509509 /**
510 - # Register <languages/>
 510+ * Register \<languages/>
511511 */
512512 $wgHooks['ParserFirstCallInit'][] = 'efTranslateInitTags';
513513
514514 /**
515 - * Strip <translate> tags etc. from source pages when rendering
 515+ * Strip \<translate> tags etc. from source pages when rendering
516516 */
517517 $wgHooks['ParserBeforeStrip'][] = 'PageTranslationHooks::renderTagPage';
518518
519519 /**
520 - * Check syntax for <translate>
 520+ * Check syntax for \<translate>
521521 */
522522 $wgHooks['ArticleSave'][] = 'PageTranslationHooks::tpSyntaxCheck';
523523 $wgHooks['EditFilterMerged'][] = 'PageTranslationHooks::tpSyntaxCheckForEditPage';

Status & tagging log