Hi,
i have a problem, because after register with the terms of use accepted. I get a second form for accepting the terms of use again.

So my question is this correct in legal.module?

// Don't insert if user is already registered (administrator).
if (!empty($user->uid) )

The user is registered at this point, because the hook_user_insert() is called after user save.
So is this way better?

if (!empty($user->uid) && $user->uid != $account->uid)

This works for me ;-)

Thanks a lot
Dirk

Comments

robert castelo’s picture

robert castelo commented

What are the steps to reproduce this issue?

Pi-XL’s picture

Pi-XL commented

Here the steps to reproduce it:

  • Install the module
  • Add legal to the registartion form
  • Register new user, with confirmation (This confirmation is not set!)
  • Login first time, you must confirm terms & conditions for the second time

I hope this helps.

Dirk

robert castelo’s picture

robert castelo commented
Status: Active » Postponed (maintainer needs more info)

"Add legal to the registartion form"

Do you mean add terms and conditions text in Legal configuration, or something else?

I couldn't reproduce this issue.

Maybe you have another module enabled that's causing problems with Legal module?

sunriser’s picture

sunriser commented

I have the same issue using the logintoboggan module
The solution that Pi-XL provides works for me as well

marcoka’s picture

marcoka commented

i can not reproduce that neither.
sunriser, Pi-XL did not provide a solution, it is a way to reproduce his problem.

i am interested if both of you use logintobogan, because that module does a lot of stuff with the registration forms/process. i dont use it and do not have this problem.

can you check with logintobogan disabled?

avpaderno’s picture

avpaderno
he/him
Primary language Italian
Location Brescia, 🇮🇹 🇪🇺
commented
Version: 7.x-1.2 » 7.x-1.x-dev
Issue summary: View changes
Status: Postponed (maintainer needs more info) » Closed (cannot reproduce)

Given the comments from the project owner and another user, I am closing this issue as not reproducible. Feel free to re-open it, if you can provide the steps to reproduce the issue.