Last modified: 2012-10-16 15:31:32 UTC
Browsing more Parser.php r28588 code and found this inconsistency. Parser::doTableStuff() ( Parser.php:740 ) trim()s the line before attempting to match table tokens. Thereby allowing a table token to appear after whitespace or NUL characters. Eg: {| Bug #529 resolution in Parser::braceSubstitution() ( Parser.php:3354 ) doesn't take into account possible whitespace or NUL characters. if (!$piece['lineStart'] && preg_match('/^(?:{\\||:|;|#|\*)/', $text)) /*}*/{ $text = "\n" . $text; }
Current code says: https://gerrit.wikimedia.org/r/gitweb?p=mediawiki/core.git;a=blob;f=includes/parser/Parser.php;h=8671665ffc8cec343f8852dbdfa4e238a8591011;hb=HEAD#l3462 3463 && !$piece['lineStart'] 3464 && preg_match( '/^(?:{\\||:|;|#|\*)/', $text ) ) so I guess this is still valid...
