[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