Missing use of db_prefix
Xen.dk - February 9, 2008 - 15:45
| Project: | Comment Notify |
| Version: | 5.x-1.4 |
| Component: | Code |
| Category: | bug report |
| Priority: | normal |
| Assigned: | greggles |
| Status: | closed |
Description
In comment_notify_page:
<?php
db_query("UPDATE {comments} c, users u
SET c.notify = 0
WHERE u.uid = c.uid
AND md5(concat(c.mail, ifnull(u.mail, u.init), c.uid, c.name, c.nid))\
= '%s'",
$arg
);
?>it should be "UPDATE {comments} c, {users} u" in order for the tables to be prefixed properly.
#1
How does this look?
#2
xen commented that it looked fine in IRC so I've committed this.
#3
Hear hear. I looked long and hard at it, and can't see any possibility for error..
#4
Automatically closed -- issue fixed for two weeks with no activity.