r62479 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r62478‎ | r62479 | r62480 >
Date:21:23, 14 February 2010
Author:ialex
Status:ok
Tags:
Comment:
Fixed some doxygen warnings
Modified paths:
  • /trunk/phase3/includes/parser/CoreParserFunctions.php (modified) (history)

Diff [purge]

Index: trunk/phase3/includes/parser/CoreParserFunctions.php
@@ -234,9 +234,9 @@
235235 * Override the title of the page when viewed, provided we've been given a
236236 * title which will normalise to the canonical title
237237 *
238 - * @param Parser $parser Parent parser
239 - * @param string $text Desired title text
240 - * @return string
 238+ * @param $parser Parser: parent parser
 239+ * @param $text String: desired title text
 240+ * @return String
241241 */
242242 static function displaytitle( $parser, $text = '' ) {
243243 global $wgRestrictDisplayTitle;
@@ -492,11 +492,11 @@
493493 * Return the size of the given page, or 0 if it's nonexistent. This is an
494494 * expensive parser function and can't be called too many times per page.
495495 *
496 - * @FIXME This doesn't work correctly on preview for getting the size of
497 - * the current page.
498 - * @FIXME Title::getLength() documentation claims that it adds things to
499 - * the link cache, so the local cache here should be unnecessary, but in
500 - * fact calling getLength() repeatedly for the same $page does seem to
 496+ * @todo Fixme: This doesn't work correctly on preview for getting the size
 497+ * of the current page.
 498+ * @todo Fixme: Title::getLength() documentation claims that it adds things
 499+ * to the link cache, so the local cache here should be unnecessary, but
 500+ * in fact calling getLength() repeatedly for the same $page does seem to
501501 * run one query for each call?
502502 */
503503 static function pagesize( $parser, $page = '', $raw = null ) {

Status & tagging log