In D7, user_mail_token() takes a first parameter by reference instead of a returning value, as in D6 (ref: http://api.drupal.org/api/drupal/modules--user--user.module/function/use...).

Function account_reminder_mail() needs to update its call to user_mail_tokens() [line 211] to reflect this change.

Currently, the missing argument causes a fatal error (see: http://drupal.org/node/1326934).

CommentFileSizeAuthor
#2 account_reminder-user_mail_tokens-1365638-v02.patch625 bytesmesr01
#1 account_reminder-user_mail_tokens-1365638.patch602 bytesmesr01
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mesr01’s picture

mesr01 CreditAttribution: mesr01 commented
FileSize
account_reminder-user_mail_tokens-1365638.patch602 bytes

Here's a patch that needs testing.

mesr01’s picture

mesr01 CreditAttribution: mesr01 commented
FileSize
account_reminder-user_mail_tokens-1365638-v02.patch625 bytes

Previous patch is incorrect. Use this one instead.

jaydub’s picture

jaydub CreditAttribution: jaydub commented

Thanks for the patch. I've done some more substantial refactoring of the d7 version to bring it up to snuff and this error shouldn't happen anymore with the new changes. Please try out the current d7 code and let me know if it works for you.

jaydub’s picture

jaydub CreditAttribution: jaydub commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.