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.
I am experiencing a problem when try to specify the redirect page as "current".
When do this, give me a error: 'Incorrect path, Please Enter valid Path".
Comment | File | Size | Author |
---|---|---|---|
#4 | login_destination-fix-current-path-validation-2617156-4.patch | 699 bytes | lucass |
Comments
Comment #2
lucass CreditAttribution: lucass commentedComment #3
lucass CreditAttribution: lucass commentedComment #4
lucass CreditAttribution: lucass commentedFollow the patch to fix this problem.
Comment #5
izmeez CreditAttribution: izmeez commentedSame problem here, redirect to
<current>
fails with incorrect path.Patch works perfectly, thanks.
It's a simple fix but I'm almost inclined to call the bug major since it breaks a major use for this module.
Comment #7
ddrozdik CreditAttribution: ddrozdik as a volunteer commentedComment #8
izmeez CreditAttribution: izmeez commentedThanks for the commit. Do you think a new release tag is in order? 7.x-1.1 is almost three years old.
Comment #9
ddrozdik CreditAttribution: ddrozdik as a volunteer commentedYeah, new version will be released soon. Just want to finish with a few other bugs before next release.
Comment #10
izmeez CreditAttribution: izmeez commentedThe commit #9ccc2e3 is not correct, it is not the same as the patch and with the commit enabling the module now causes a WSOD. Also on closer examination the permissions on the files are not correct.
Comment #11
ddrozdik CreditAttribution: ddrozdik as a volunteer commented@izmeez, I have added your patch to the new structure of the code, because your patch has been added after applying changes in other issue.
But you still can see that if you will add or destination then function will finish work.
Also I have checked module on the real sites before committing the changes, and did not saw any WSOD. Please check, that you have installed latest version of the module, and if module was installed earlier you have done update.php. If error still appear please provide log messages from your site.
Comment #12
izmeez CreditAttribution: izmeez commentedok I will take another look and try to test it again.
Also the patch is from @lucass
Thanks.
Comment #13
ddrozdik CreditAttribution: ddrozdik as a volunteer commented@izmeez any news?
Comment #14
izmeez CreditAttribution: izmeez commentedSorry to not get back earlier.
Versions 7.x-1.3 and the latest 7.x-1.x-dev work fine.
I've changed the status to Closed (fixed).
Thanks.