The code is not following drupal coding guidelines to include all tables in curly brackets.
E.g

select count(*) as count from users where status = 1 and login != 0

Need to be

select count(*) as count from {users} where status = 1 and login != 0

This is not just a purist point. If you have tables with prefix defined, the module does not work due to above issue.

Comments

yelvington’s picture

yelvington commented
Assigned: Unassigned » yelvington

You're right. I thought I had fixed that. I'm traveling right now; will look at this when I get back.

ken hawkins’s picture

ken hawkins commented
Status: Active » Needs work

Just dove into the code. Looks like they're all fixed except for one in the "usercount_blocked" function.

At line 163

    $sql = "select count(*) as count from users where status = 0 and login != 0";
yelvington’s picture

yelvington commented
Issue summary: View changes
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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