There are many situations where it may be desirable to have the destination parameter (i.e. "https://example.com/user/login?destination=/node/2932") take precedence over the redirects configured in the Login Redirect Per Role module.

For example, if a user tries to access a page on the site that requires a login and we want to redirect them back to the page after logging in.

I'm attaching a patch which provides a configuration option that will allow for this.

CommentFileSizeAuthor
#2 allow-destination-redirect-3020104-2.patch1.86 KBnplowman
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

nplowman created an issue. See original summary.

nplowman’s picture

nplowman CreditAttribution: nplowman at FFW commented
FileSize
allow-destination-redirect-3020104-2.patch1.86 KB
JeroenT’s picture

JeroenT
Primary language Dutch
Location 🇧🇪
CreditAttribution: JeroenT at iO commented
Status: Active » Needs review

  • Antonnavi committed 15464ff on 8.x-1.x authored by nplowman 
    Issue #3020104 by nplowman: Allow destination parameter to override...
Antonnavi’s picture

Antonnavi
Primary language Ukrainian
Location Berlin
CreditAttribution: Antonnavi at Skilld commented
Status: Needs review » Fixed

Works/looks fine for me.

Antonnavi’s picture

Antonnavi
Primary language Ukrainian
Location Berlin
CreditAttribution: Antonnavi at Skilld commented
Assigned: nplowman » Unassigned
pratik.mehta19’s picture

pratik.mehta19 CreditAttribution: pratik.mehta19 as a volunteer commented
Status: Fixed » Closed (fixed)
GNUMatrix’s picture

GNUMatrix CreditAttribution: GNUMatrix commented
Component: Code » Documentation
Priority: Normal » Minor

Perfect. If someone could add a note at the top of the module config UI to indicate that this is exactly what the "destination" checkbox is for, that would have saved me the trouble of figuring it out by finding this issue in the queue :-)