Followup to r38525 (tweaks from r38400) -- don't put fragments on redlinks; they...

(I suppose it's dubious whether we should do the same to nonexitent special pages.... but fragments there wouldn't be too helpful either so no need for extra logic to add it back. :)

diff --git a/includes/Linker.php b/includes/Linker.php
index 4e4b5f5..b913ebf 100644 (file)
--- a/includes/Linker.php
+++ b/includes/Linker.php
@@ -522,11 +522,6 @@ class Linker {
                        $q = 'action=edit&redlink=1&'.$query;
                }
                $u = $nt->escapeLocalURL( $q );
-               if( $nt->getFragmentForURL() !== '' ) {
-                       # Might seem pointless to have a fragment on a redlink, but let's
-                       # be obedient.
-                       $u .= $nt->getFragmentForURL();
-               }
 
                $titleText = $nt->getPrefixedText();
                if ( '' == $text ) {

diff --git a/maintenance/parserTests.txt b/maintenance/parserTests.txt
index 0c36f57..46f12f4 100644 (file)
--- a/maintenance/parserTests.txt
+++ b/maintenance/parserTests.txt
@@ -1345,7 +1345,7 @@ Nonexistent special page link with fragment
 !! input
 [[Special:ThisNameWillHopefullyNeverBeUsed#anchor]]
 !! result
-<p><a href="/wiki/Special:ThisNameWillHopefullyNeverBeUsed#anchor" class="new" title="Special:ThisNameWillHopefullyNeverBeUsed (not yet written)">Special:ThisNameWillHopefullyNeverBeUsed#anchor</a>
+<p><a href="/wiki/Special:ThisNameWillHopefullyNeverBeUsed" class="new" title="Special:ThisNameWillHopefullyNeverBeUsed (not yet written)">Special:ThisNameWillHopefullyNeverBeUsed#anchor</a>
 </p>
 !! end