[TYPO3-core] RFC: #7350:PHP5ize class.t3lib_div.php

Martin Kutschker martin.kutschker-n0spam at no5pam-blackbox.net
Sun Feb 3 19:08:04 CET 2008


Michael Stucki schrieb:
> Hi Masi,
> 
>> The final patch is attached. I did the same to split_tag_attributes and
>> added some code cleanup.
> 
> Just one question:
> 
> You added this line:
> 
> +       if (preg_match('/^\s*<\?xml([^>]*)\?\>/', $xmlData, $match))    {
> 
> There is no sense in allowing whitespaces before the opening tag, right?

It was there before. The old code had a trim() before doing the match. 
I'm not sure if any white space before the prologue is allowed (I think 
not), but I think it's better to be a bit relaxed in what we accept as 
input.

Masi


More information about the TYPO3-team-core mailing list