[TYPO3-core] RFC: Add new property TSFE.jumpURL_HTTPStatusCode
Martin Kutschker
Martin.Kutschker at n0spam-blackbox.net
Mon Apr 30 11:18:45 CEST 2007
Thorsten Kahler schrieb:
> Hi Masi,
>
> Martin Kutschker wrote on 30.04.2007 10:40:
>> Michael Stucki schrieb:
>>> This is a SVN patch request.
>>>
>>> Problem:
>>> JumpURL redirects are always forwarded using HTTP status 302 (which is the
>>> default when using header('Location: ...')).
>> Saw it working on T3DD07: +1
>
> I saw it working, too. But it's not 24 hours ago I heard you say that a
> review should be more than just a test if the code is working?
In what way should I comment the patch more? I don't object the code.
But here you are: The configuration via page TS seems to be in line with
the other jumpURL options thoug I think it would be possible to make the
status code configurable via setup.
>> PS: If 302 is default it may be removed from the switch.
>
> No, it should be marked as default case - if we stick to the switch statement.
Fine.
Masi
More information about the TYPO3-team-core
mailing list