r91142 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r91141‎ | r91142 | r91143 >
Date:23:35, 29 June 2011
Author:brion
Status:ok
Tags:
Comment:
Tweak doc comments on WebRequest::getRequestURL() & WebRequest::getFullRequestURL() per in-person discussion with preilly; the docs were unclear as to whether the query string is included (it is).
Modified paths:
  • /trunk/phase3/includes/WebRequest.php (modified) (history)

Diff [purge]

Index: trunk/phase3/includes/WebRequest.php
@@ -536,7 +536,8 @@
537537 }
538538
539539 /**
540 - * Return the path portion of the request URI.
 540+ * Return the path and query string portion of the request URI.
 541+ * This will be suitable for use as a relative link in HTML output.
541542 *
542543 * @return String
543544 */
@@ -574,7 +575,9 @@
575576 }
576577
577578 /**
578 - * Return the request URI with the canonical service and hostname.
 579+ * Return the request URI with the canonical service and hostname, path,
 580+ * and query string. This will be suitable for use as an absolute link
 581+ * in HTML or other output.
579582 *
580583 * @return String
581584 */

Status & tagging log