From: Ilmari Karonen <vyznev@users.mediawiki.org>
Date: Sun, 17 Dec 2006 16:51:01 +0000 (+0000)
Subject: Change the regexen in doBlockLevels() to handle <center> tags the same as
X-Git-Tag: 1.31.0-rc.0~54863
X-Git-Url: http://git.cyclocoop.org/%22%20.%20generer_url_ecrire%28%22calendrier%22%2C%22type=semaine%22%29%20.%20%22?a=commitdiff_plain;h=7325234d67e8c6d1759186d6aff92dee74910e82;p=lhc%2Fweb%2Fwiklou.git

Change the regexen in doBlockLevels() to handle <center> tags the same as
<div>.  This fixes bug 8293: Use of center tag ruins paragraph formatting,
but COULD SOMEONE PLEASE EXPLAIN TO ME WHY??!?  Trying to make sense of
that code gives me a headache.  (Ps. See also related change in r14690.)
---

diff --git a/RELEASE-NOTES b/RELEASE-NOTES
index 1c63875b31..83400e3aa1 100644
--- a/RELEASE-NOTES
+++ b/RELEASE-NOTES
@@ -336,6 +336,8 @@ it from source control: http://www.mediawiki.org/wiki/Download_from_SVN
 * (bug 8274) Wrap edit tools in a <div> with a specified class
 * Detect PHP 5.0.x 64-bit bug and abort in WebStart.php; too many things break
   mysteriously otherwise (detection code copied from install-utils.inc)
+* (bug 8295) Change handling of <center> tags in doBlockLevels() to match that
+  of <div>
 
 == Languages updated ==
 
diff --git a/includes/Parser.php b/includes/Parser.php
index 7e594f1bbe..9a0331da9e 100644
--- a/includes/Parser.php
+++ b/includes/Parser.php
@@ -2123,10 +2123,10 @@ class Parser
 				wfProfileIn( "$fname-paragraph" );
 				# No prefix (not in list)--go to paragraph mode
 				// XXX: use a stack for nestable elements like span, table and div
-				$openmatch = preg_match('/(<table|<blockquote|<h1|<h2|<h3|<h4|<h5|<h6|<pre|<tr|<p|<ul|<ol|<li|<\\/center|<\\/tr|<\\/td|<\\/th)/iS', $t );
+				$openmatch = preg_match('/(<table|<blockquote|<h1|<h2|<h3|<h4|<h5|<h6|<pre|<tr|<p|<ul|<ol|<li|<\\/tr|<\\/td|<\\/th)/iS', $t );
 				$closematch = preg_match(
 					'/(<\\/table|<\\/blockquote|<\\/h1|<\\/h2|<\\/h3|<\\/h4|<\\/h5|<\\/h6|'.
-					'<td|<th|<div|<\\/div|<hr|<\\/pre|<\\/p|'.$this->mUniqPrefix.'-pre|<\\/li|<\\/ul|<\\/ol|<center)/iS', $t );
+					'<td|<th|<\\/?div|<hr|<\\/pre|<\\/p|'.$this->mUniqPrefix.'-pre|<\\/li|<\\/ul|<\\/ol|<\\/?center)/iS', $t );
 				if ( $openmatch or $closematch ) {
 					$paragraphStack = false;
 					# TODO bug 5718: paragraph closed