[TYPO3-core] FYI: #10183: [felogin] redirect doesn't work with return_url

Steffen Kamper info at sk-typo3.de
Mon Jan 19 10:10:06 CET 2009


moin Benni,

Benjamin Mack schrieb:
> Hey,
> 
> would this suffice?
> 
> $this->redirectUrl = (t3lib_div::_GP('return_url') ? 
> t3lib_div::_GP('return_url') : t3lib_div::_GP('redirect_url'));
> 
> Also, maybe you want to set a small comment on the top of the code, why 
> return_url is processed first here. Is that ok?
> 
> Apart from that... +1 after reading.
> 

as i did the commit for trunk and 4_2 already (i wanted that to be 
included in release tomorrow) i added the comment now.

Your suggestion in one line is ok too, but i didn't changed, as it's 
cosmetic and this way a bit better readable.

vg Steffen


More information about the TYPO3-team-core mailing list