Download & Extend

Missing {braces} around table name on line 141 of signup_status_limit.module 6.x-1.x-dev

Project:Signup Status
Version:6.x-1.x-dev
Component:Signup status limit
Category:bug report
Priority:normal
Assigned:dww
Status:closed (fixed)

Issue Summary

Hello everyone,

I'm afraid I don't know how to produce patch files, but this is a very minor correction so hopefully it won't be too much of a problem for someone to help out.

I've spotted that the signup_status_limit module is missing its curly braces around the database table name on line 141 - this line:

<?php
$query
= db_query("SELECT COUNT(*) as total, status FROM signup_log WHERE nid = %d GROUP BY status", $node->nid);
?>

... should be:

<?php
$query
= db_query("SELECT COUNT(*) as total, status FROM {signup_log} WHERE nid = %d GROUP BY status", $node->nid);
?>

Hope this helps - I did try searching to see if anyone else had already spotted and fixed this, but I couldn't see anything in search.

Best regards,

Al

Comments

#1

Project:Signup» Signup Status
Version:6.x-1.x-dev» 6.x-1.x-dev
Component:Code» Signup status limit
Assigned to:Anonymous» dww
Status:needs review» fixed

Thanks, good catch. Committed to HEAD. Moving to the right queue for posterity.

#2

Status:fixed» closed (fixed)

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

nobody click here