There are several other modules that attempt to perform a redirection upon user login, such as login_destination and total_control. These modules generally provide an option to not perform the redirection if the ?destination query param has been set.

I think that the one time login links generated by this module should use the destination paramater to determine where to redirect the user to, instead of additional URL arguments. This would make it play a lot better with other modules.

CommentFileSizeAuthor
#1 1127436-login_one_time-destination.patch1.22 KBmrfelton
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mrfelton’s picture

mrfelton CreditAttribution: mrfelton commented
Status: Active » Needs review
FileSize
1127436-login_one_time-destination.patch1.22 KB
danielb’s picture

danielb CreditAttribution: danielb commented
Status: Needs review » Fixed

I originally did it my way because "destination?=" looks a bit ugly, but if it has functional benefits when there is a conflict, then lets stick it in. Cheers. Will blame the ugly on core developers in the future.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.