[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