[TYPO3-core] RFC: #10327: [felogin] redirecting to referer does not work

[Xavier Perseguers]

On 03/12/10 16:51 , Xavier Perseguers wrote:
> Hi,
>
> Please find enclosed a follow-up for this bug (committed as revision
> 6638). Lines around 468 that were added do not allow redirect_url to be
> defined when a user is not yet logged in ($this->logintype==='login'),
> and as such modifying line 477 should not have happened.

Just to describe the problem... When you use EXT:seminar for instance, and want to redirect to a page that needs login while not being logged, you're redirected to the login form with a url of the form:

http://typo3website.tld/path/to/login?redirect_url=<encoded_url_to_redirect_to_after_login>

You have the login form, log in but you are not redirected although you setup the plugin to redirect according to GET/POST redirect_url.

Problem is that the login form does not include the redirect_url parameter in its hidden fields due to the problem that patch solves. A few lines above, testing against $this->loggintype === 'login' 
clears the redirectUrl that was properly retrieved from GET parameter as when not yet logged in, $this->logintype is empty. Reverting the line that was "simplified" with changeset 6638 fixes this 
issue and then all is (hopefully) working great.

Cheers

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