Hi,
I installed and enabled 8.x-1.0-beta1 version.
I noticed that redirect on Registration is not working when the email verification is required ("Require email verification when a visitor creates an account" is checked in Account settings).
I resolved this by creating a Rule to redirect after a new user is saved. However, I think would be important to solve it within the Login Destination module.
I raise this as bug, even if not sure this is the expected behavior (in case you can eventually change it to Feature request).

Regards

CommentFileSizeAuthor
#6 3131342-6.patch658 byteschucksimply
#2 3131342-2.patch654 byteshimanshu_sindhwani

Comments

axel80 created an issue. See original summary.

himanshu_sindhwani’s picture

Category: Bug report » Support request
Status: Active » Needs review
StatusFileSize
new654 bytes

I checked the code of this module and evert redirect is written in hook_user_login(), so whenever a user registers on the website and "Require email verification when a visitor creates an account" the redirections are never called since the user is not logged into the website. Here I have created a patch that can extend the functionality of this module. This should not be considered under the bug report as the name of the module itself suggests login destination. So you may add redirections after login but in this case, you are never logged in.
Nevertheless, you can use this path to add redirections.

axel80’s picture

Patch worked for me.
here the test conditions

  • "Require email verification when a visitor creates an account" set to true
  • New Location Destination rule created with "Redirect upon triggers" = Registration

Result: after the user submits the email address he's redirected to the destination page connfigured in the rule

axel80’s picture

Status: Needs review » Reviewed & tested by the community
olisb’s picture

Thanks for contributing this!
The Patch worked for me in version 8.x-1.0-beta2
This should be rolled in - as the name and set up of the module implies this functionality should work out of the box.

chucksimply’s picture

StatusFileSize
new658 bytes

#2 didn't work for me on 8.x-2.0-beta1. Adding a new patch that applied cleanly on this module version.

chucksimply’s picture

Version: 8.x-1.0-beta1 » 8.x-2.0-beta1
chucksimply’s picture

Well my above patch applied cleanly, but doesn't work as noted in #3

rsvelko’s picture

Status: Reviewed & tested by the community » Needs work
gaurav.kapoor’s picture

Status: Needs work » Reviewed & tested by the community

The patch resolved the issue for me and we should not be worried about failing test cases as of now, most of the test cases have any way deprecated code which is being handled in a different issue. I will take a note of updating this (behavior of Login Destination in case email verification has been enabled) in the README file.

chucksimply’s picture

Still not working for me... not sure why this is marked as reviewed and tested by the community.

Creating a one-time login destination doesn't redirect when the user clicks the registration confirmation link in their email.

Needs to be reopened

bas123’s picture

Version: 8.x-2.0-beta1

First off, some sort of basic instructions for setting up this module would be helpful, and this could potentially prevent the need for this sort of post.

I am a bit confused by this set of patches and the conditions being applied!

In my use case, we have a registration process where both the email confirmation is sent, but the registration itself is contingent upon an Admin approving the account. This triggers the temporary login link that allows the 'approved' registrant to set their password to complete the process.

The redirect does not seem to work with Redirect upon triggers: Registration
With the following in the module's configuration fields:

  • Lable: Signup Redirect
  • Destination: Internal destination
  • Triggers: Registration
  • Pages: No pages
  • Language: All languages
  • Roles: All roles
  • Redirect from specific pages: (Tried 'All pages except those listed' and 'Only the listed pages' with and without a URL)

However, no redirect takes place when an applicant submits a complete (required fields) Registration

--------------------------------------

Also, while it does work with this configuration for Redirect upon triggers: One time login, I would like to know if a specific configuration is possible.

  • Lable: One time login destination
  • Destination: Internal destination
  • Triggers: One-time login link
  • Pages: All pages
  • Language: All languages
  • Roles: Authenticated user, Verified User
  • Redirect from specific pages: All pages except those listed (no URL's listed)

Unfortunately, while the Login the above does work for both "Approved Applicants" and "Lost Password?" I would like to separate the destination pages for each, and setting: Redirect from specific pages "Only the listed pages" doesn't seem possible as the system seems to use the same URL for both. Having this available would allow us to have a landing page specific to the situation rather than Welcoming new members and people who have recovered their existing accounts on the same page.

If there is a specific token that could be added to a URL to create the distinction between the two, I would appreciate knowing what that would be!

chucksimply’s picture

Status: Reviewed & tested by the community » Needs work
plousia’s picture

Can confirm this patch doesn't work for me either. Creating a redirect on registration rule with email verification required does not result in user being redirected on registering.

hockey2112’s picture

Same issue here. does not redirect when I have "email verification required" checked.