From 3012b8df3808cf1489a1ecbc4a3e037eca49a8f9 Mon Sep 17 00:00:00 2001 From: Sam Reed Date: Fri, 13 Jan 2012 21:42:31 +0000 Subject: [PATCH] () are valid in URLs, not sure why we're using them as a finishing point in ApiFormatBase Fixes fixme on r95572 as urls are now complete with () in them --- includes/api/ApiFormatBase.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/includes/api/ApiFormatBase.php b/includes/api/ApiFormatBase.php index 674763299b..543c90ce08 100644 --- a/includes/api/ApiFormatBase.php +++ b/includes/api/ApiFormatBase.php @@ -264,7 +264,7 @@ See complete documentation, or // This regex hacks around bug 13218 (" included in the URL) $text = preg_replace( "#(($protos).*?)(")?([ \\'\"<>\n]|<|>|")#", '\\1\\3\\4', $text ); // identify requests to api.php - $text = preg_replace( "#api\\.php\\?[^ \\()<\n\t]+#", '\\0', $text ); + $text = preg_replace( "#api\\.php\\?[^ <\n\t]+#", '\\0', $text ); if ( $this->mHelp ) { // make strings inside * bold $text = preg_replace( "#\\*[^<>\n]+\\*#", '\\0', $text ); -- 2.20.1