Thijs/wiki.techinc.nl
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Sanitizer::normalizeWhitespace: simplify redundant preg_replace

Browse source 
The extraneous sequence \r\n is not required.
Avoid the use of hexcode to avoid future confusions.

Bug: T388733
Change-Id: I1092ff76ed5e8221e43ea7b70cf0c9d9d3abb1f3
(cherry picked from commit 6753123a0629de81ce4899958180272736e7ba61)
This commit is contained in:
David Causse 2025-03-18 10:36:36 +01:00 committed by Reedy
parent dccd645af7
commit 0f921b7878
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
includes/parser/Sanitizer.php
View file

@ -1136,7 +1136,7 @@ class Sanitizer {
}
private static function normalizeWhitespace( string $text ): string {
$normalized = preg_replace( '/(?:\r\n|[\x20\x0d\x0a\x09])+/', ' ', $text );
$normalized = preg_replace( '/[ \r\n\t]+/', ' ', $text );
if ( $normalized === null ) {
wfLogWarning( __METHOD__ . ': Failed to normalize whitespace: ' . preg_last_error() );
return '';