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

Comments

mrfelton’s picture

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

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.