When we disable a user account (blocked but not completely deleted), this message shows up in the log for each account cancellation:

Warning: Parameter 1 to account_reminder_user_cancel() expected to be a reference, value given in module_invoke_all() (line 850 of /home/user/public_html/includes/module.inc).

CommentFileSizeAuthor
#3 account_reminder-hook_user_cancel-1733848-1.patch1.13 KBjaydub
#1 account_reminder-hook_user_cancel-1733848-1.patch1.13 KBjaydub
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jaydub’s picture

jaydub CreditAttribution: jaydub commented
Status: Active » Needs review
FileSize
account_reminder-hook_user_cancel-1733848-1.patch1.13 KB

Can you try the attached patch?

dhalbert’s picture

dhalbert CreditAttribution: dhalbert commented
Status: Needs review » Active

What service! Unfortunately the patch does not apply. I pulled the latest with git, but I get an error. Maybe you generated a patch against a newer version?

# git apply -v account_reminder-hook_user_cancel-1733848-1.patch
Checking patch account_reminder.module...
Hunk #1 succeeded at 239 (offset 86 lines).
error: while searching for:
}

/**
 * Implement hook_cron().
 */
function account_reminder_cron() {

error: patch failed: account_reminder.module:165
error: account_reminder.module: patch does not apply

In the account_reminder.module I have, line 165 is:

function account_reminder_variable_info($options) {
jaydub’s picture

jaydub CreditAttribution: jaydub commented
FileSize
account_reminder-hook_user_cancel-1733848-1.patch1.13 KB

Let's try that again, I hadn't updated my tree :)

dhalbert’s picture

dhalbert CreditAttribution: dhalbert commented
Status: Active » Reviewed & tested by the community

Works! No more error. Thank you.

jaydub’s picture

jaydub CreditAttribution: jaydub commented
Status: Reviewed & tested by the community » Fixed

Committed fix

Status: Fixed » Closed (fixed)

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