From 3272f4a08a1965a0a9e2c7bd2261b16300bdadfa Mon Sep 17 00:00:00 2001 From: Aryeh Gregor Date: Mon, 5 Jan 2009 15:59:17 +0000 Subject: [PATCH] Add comment on urlencoding fragments --- includes/Title.php | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/includes/Title.php b/includes/Title.php index 95b2e51476..9fd4c79c85 100644 --- a/includes/Title.php +++ b/includes/Title.php @@ -452,6 +452,10 @@ class Title { */ static function escapeFragmentForURL( $fragment ) { global $wgEnforceHtmlIds; + # Note that we don't urlencode the fragment. urlencoded Unicode + # fragments appear not to work in IE (at least up to 7) or in at least + # one version of Opera 9.x. The W3C validator, for one, doesn't seem + # to care if they aren't encoded. return Sanitizer::escapeId( $fragment, $wgEnforceHtmlIds ? array() : 'xml' ); } -- 2.20.1