[TYPO3-core] RFC #13732: External URL only indexes first page [indexed_search]

Xavier Perseguers typo3 at perseguers.ch
Tue Mar 23 11:22:44 CET 2010


Hi,

> I am +1 to your patch by reading. After it is committed I am going to
> rework my patch to include your code.
>
> There are two small things that I did not like about the patch. May be
> you can have a look at them:
> - HTML parse is quite huge to get just<base>. May be a simple regular
> expression can be better? /<base[^>]+href="([^"]+)/i should do the trick.
> - there are two return statements. If I remember correctly, only one
> should be in the function.

I used same methodology as other part of the code but I agree this is 
suboptimal. Will change this to some simple regular expression instead 
while committing.

> These two issues do not look major to me, therefore I am still willing
> to give +1 to the current version of the patch.

Are +1 by testing required? If so, anyone willing to test?

-- 
Xavier Perseguers
http://xavier.perseguers.ch/en


More information about the TYPO3-team-core mailing list