When a new user is registered, the next cron crun sends him an account_reminder mail.
I'm not sure that mail is sent to user because i cannot access mail logs but i suppose it from the bcc email (configured in account_reminder options) i recieve.
A wathcdog message is also logged: Reminder email send to [new user mail]

"Days till initial reminder:" is 7 days.

I consider it a bug because a new user already recieve a mail with login instruction from drupal registration system.

Comments

jaydub’s picture

jaydub commented

did you install the module as a new install or did you upgrade over the old version? If you
upgraded you will definitely have to run the update to get the change in the one of the
database columns.

if you have access to your database can you select the row for one of the users in
question to see what is written to the account_reminder table?

select * from account_reminder where uid = X

jaydub’s picture

jaydub commented
Status: Active » Fixed

I made some changes that should address this problem. Please try the newest
-dev version.

samuelet’s picture

samuelet commented

I've installed it but after a cron run a reminder mail was sent to two active account users.
I could be wrong and they could exist before upgrading, but I guess that was your patch fault because "select * from account_reminder" displays a lists sorted by id execept for those two users ids:

46	1207141202	1
48	1207141202	1
49	1207141203	1
95	1207141205	1
127	1207141205	1
168	1207334403	1
45	1207474802	1   //active user
75	1207474802	1   //active user

These are their created and access time (from user table):
45 1193938607 1193938673
75 1194359198 1203074997

In my first bug reporting, the module was installed for the first time.

samuelet’s picture

samuelet commented

I guess that the fault is the u.login = 0 in the sql query at line 168 of account_reminder.module.
It should be u.access = 0.

samuelet’s picture

samuelet commented

Forget last reply, i just see that login is a numeric field and i don't know what it differs from access field.

jaydub’s picture

jaydub commented

There were two bugs that have been addressed in the latest code. The first was my bug
regarding the sending out of reminder emails before the initial period of days was
supposed to have passed.

The second was an issue of the original module design. The use of the access column of the
user table should have been the login column instead. The true test for whether a user
has used their account should be based on whether or not they have ever logged in.
In fact the use of the access column created its own problem because the generation of
any reminder email sets a new temporary password for the user. In doing this and calling
user_save to save the new password, the access column for that user will be updated
regardless of whether or not a user has ever logged in.

So the current -dev code should handle your problem now. Please help test by creating
a user to make sure that no reminder is sent out earlier than it should. thanks.

jaydub’s picture

jaydub commented
Status: Fixed » Active
samuelet’s picture

samuelet commented

I confirm that the reminder mail for new users is not sent anymore

jaydub’s picture

jaydub commented
Status: Active » Closed (fixed)