The rendering of autosummary in History and Diff pages did not properly escape the...

This was causing inconsistencies between anchors in editsummaries and those actually used in pages.
See also bug 18431

diff --git a/RELEASE-NOTES b/RELEASE-NOTES
index 965f800..e38f341 100644 (file)
--- a/RELEASE-NOTES
+++ b/RELEASE-NOTES
@@ -202,6 +202,7 @@ it from source control: http://www.mediawiki.org/wiki/Download_from_SVN
   a fatal error
 * (bug 23465) Don't ignore the predefined destination filename on 
   Special:Upload after following a red link
+* Correct the escaping of the autosummary URI fragments.
 
 === API changes in 1.17 ===
 * (bug 22738) Allow filtering by action type on query=logevent.

diff --git a/includes/Linker.php b/includes/Linker.php
index 79e417e..f6e45d1 100644 (file)
--- a/includes/Linker.php
+++ b/includes/Linker.php
@@ -1026,6 +1026,9 @@ class Linker {
                        $section = str_replace( '[[:', '', $section );
                        $section = str_replace( '[[', '', $section );
                        $section = str_replace( ']]', '', $section );
+
+                       # Most of Title:: expects fragments to be escaped
+                       $section = Title::escapeFragmentForURL( $section );
                        if ( $local ) {
                                $sectionTitle = Title::newFromText( '#' . $section );
                        } else {