r46641 MediaWiki - Code Review archive

Repository:MediaWiki
Revision:r46640‎ | r46641 | r46642 >
Date:13:05, 31 January 2009
Author:catrope
Status:deferred
Tags:
Comment:
API: (bug 17182) Fix pretty printer so URLs with parentheses in them are autolinked correctly
Modified paths:
  • /trunk/phase3/RELEASE-NOTES (modified) (history)
  • /trunk/phase3/includes/api/ApiFormatBase.php (modified) (history)

Diff [purge]

Index: trunk/phase3/includes/api/ApiFormatBase.php
@@ -209,7 +209,7 @@
210210 // identify URLs
211211 $protos = implode("|", $wgUrlProtocols);
212212 # This regex hacks around bug 13218 (" included in the URL)
213 - $text = preg_replace("#(($protos).*?)(&quot;)?([ \\'\"()<\n])#", '<a href="\\1">\\1</a>\\3\\4', $text);
 213+ $text = preg_replace("#(($protos).*?)(&quot;)?([ \\'\"<\n])#", '<a href="\\1">\\1</a>\\3\\4', $text);
214214 // identify requests to api.php
215215 $text = preg_replace("#api\\.php\\?[^ \\()<\n\t]+#", '<a href="\\0">\\0</a>', $text);
216216 if( $this->mHelp ) {
Index: trunk/phase3/RELEASE-NOTES
@@ -1,4 +1,4 @@
2 - = MediaWiki release notes =
 2+= MediaWiki release notes =
33
44 Security reminder: MediaWiki does not require PHP's register_globals
55 setting since version 1.2.0. If you have it on, turn it *off* if you can.
@@ -137,6 +137,8 @@
138138 * (bug 16483) Kill filesort in ApiQueryBacklinks caused by missing parentheses.
139139 Building query properly now using makeList()
140140 * Add "check" parameter to action=email
 141+* (bug 17182) Fix pretty printer so URLs with parentheses in them are
 142+ autolinked correctly
141143
142144 === Languages updated in 1.15 ===
143145

Status & tagging log