From: Brion Vibber <brion@users.mediawiki.org>
Date: Fri, 21 Aug 2009 22:31:23 +0000 (+0000)
Subject: Revert r53667 "Enabled use of the greater-than sign (">") in page titles. There is... 
X-Git-Tag: 1.31.0-rc.0~40139
X-Git-Url: https://git.cyclocoop.org/%242?a=commitdiff_plain;h=4b8679df3c458cce744d3d399d3f183591750aaf;p=lhc%2Fweb%2Fwiklou.git

Revert r53667 "Enabled use of the greater-than sign (">") in page titles. There is actually no reason for this character to be restricted because it does not interfere with wiki syntax or HTML tags. This will have immediate applications for articles such as "M>Tram", ">play", "The Videos 86>98", "Tour 00 >> 01 Macabre", and "I>Télé", as well as redirects for "Inequality" and "Bitwise operation"."
Possibility of spitting raw >s into output unnerves me greatly, and could indeed be a problem depending on just where your text gets spat and what's around it. More generally allowing one half of a pair makes little sense and would simply confuse the issue.
---

diff --git a/includes/DefaultSettings.php b/includes/DefaultSettings.php
index 5b5c5f161e..fe24633aa1 100644
--- a/includes/DefaultSettings.php
+++ b/includes/DefaultSettings.php
@@ -259,7 +259,7 @@ $wgForeignFileRepos = array();
  *
  * Problematic punctuation:
  *  []{}|#    Are needed for link syntax, never enable these
- *  <         Causes problems with HTML escaping, don't use
+ *  <>        Causes problems with HTML escaping, don't use
  *  %         Enabled by default, minor problems with path to query rewrite rules, see below
  *  +         Enabled by default, but doesn't work with path to query rewrite rules, corrupted by apache
  *  ?         Enabled by default, but doesn't work with path to PATH_INFO rewrites
@@ -279,7 +279,7 @@ $wgForeignFileRepos = array();
  * Theoretically 0x80-0x9F of ISO 8859-1 should be disallowed, but
  * this breaks interlanguage links
  */
-$wgLegalTitleChars = " %!\"$&'()*,\\-.\\/0-9:;=?@A-Z\\\\^_`a-z~\\x80-\\xFF+>";
+$wgLegalTitleChars = " %!\"$&'()*,\\-.\\/0-9:;=?@A-Z\\\\^_`a-z~\\x80-\\xFF+";
 
 
 /**
diff --git a/maintenance/parserTests.txt b/maintenance/parserTests.txt
index 6afcef53a8..c94203b9b5 100644
--- a/maintenance/parserTests.txt
+++ b/maintenance/parserTests.txt
@@ -1453,7 +1453,7 @@ Link containing % as a double hex sequence interpreted to hex sequence
 !!end
 
 !! test
-Link containing "#<" and "#>" % as hex sequences- these are valid section anchors
+Link containing "#<" and "#>" % as a hex sequences- these are valid section anchors
 Example for such a section: == < ==
 !! input
 [[%23%3c]][[%23%3e]]
@@ -1463,11 +1463,11 @@ Example for such a section: == < ==
 !! end
 
 !! test
-Link containing "<#" and ">#" as hex sequences
+Link containing "<#" and ">#" as a hex sequences
 !! input
 [[%3c%23]][[%3e%23]]
 !! result
-<p>[[%3c%23]]<a href="/index.php?title=%3E&amp;action=edit&amp;redlink=1" class="new" title="> (page does not exist)">&gt;#</a>
+<p>[[%3c%23]][[%3e%23]]
 </p>
 !! end