In account_reminder.module, there is this code ...

// reset notifications if recent user activity
    $users = db_fetch_object(db_query('SELECT uid FROM {inactive_users} WHERE uid <> 1'));
    if ($users) {
      foreach ($users as $uid) {
      ...

This is wrong as db_fetch_object will only return a single row at a time. However, the line foreach ($users as $uid) suggest that it is expected by the developer of the module to be an array of all the results.

Cheers

Asiby

Comments

jaydub’s picture

Status: Active » Postponed (maintainer needs more info)

Line number please? There is no table {inactive_users} created or queried against in Account Reminder module.
Perhaps you mean the Inactive User module?

jaydub’s picture

Project: Account reminder » Inactive User
Version: 6.x-1.2 » 7.x-1.x-dev
Status: Postponed (maintainer needs more info) » Active

yes I verified that you are talking about code in another module. Changing project to that module.

asiby’s picture

Sorry for the confusion I have cause. I thank you for your timely response (less than 24 hours).

Cheers

deekayen’s picture

Issue summary: View changes
Status: Active » Closed (cannot reproduce)

The code doesn't look like that anymore. Not sure when it changed in relation to this report.