We shouldn't only check for variable_get('toboggan_redirect_on_register', '') but also whether $_GET['destination'] is set.

Comments

hunmonk’s picture

Status: Needs review » Postponed (maintainer needs more info)

this would mean that the destination string outranks the LT setting -- i'm unclear whether this is a good move or not. can you explain your use case and why you think this is the proper approach?

hunmonk’s picture

Status: Postponed (maintainer needs more info) » Closed (won't fix)

marking as won't fix unless/until i get an explanation why the proposed patch is better than the current approach.