Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.When creating a login destination with a trigger of "One-time login link," the trigger is incorrectly identified as "Login" due to a $_SESSION parameter that is not present. I recommend a new way of detecting the one-time login link trigger that is in line with the Drupal 8 way of doing things. I'll post my patch in the next comment.
| Comment | File | Size | Author |
|---|---|---|---|
| #2 | login_destination-one_time_login-2884820-2-8.x-1.x.dev_.patch | 1.01 KB | pjohn |
Comments
Comment #2
pjohn CreditAttribution: pjohn as a volunteer commentedHere's the patch I propose for this. Can someone please review it?
Comment #3
ddrozdik CreditAttribution: ddrozdik as a volunteer and at Five Jars commentedHi @pjohn,
Your patch looks good. In case you would like to get a credit after applying the patch please create a patch in the correct format. Refer this link for details.
Thanks.
Comment #4
ddrozdik CreditAttribution: ddrozdik as a volunteer and at Five Jars commentedNevermind, I see that fix has been already added in this issue https://www.drupal.org/project/login_destination/issues/2852881.
Closing, since this issue is fixed.