Closed (fixed)
Project:
Twitter
Version:
6.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
28 Jun 2008 at 01:20 UTC
Updated:
5 Sep 2008 at 02:52 UTC
// Nuke old statuses.
if ($age = variable_get('twitter_expire', 0)) {
db_query('DELETE {twitter} WHERE created_time < %d', time() - $age);
}
Shouldn't this be
// Nuke old statuses.
if ($age = variable_get('twitter_expire', 0)) {
db_query('DELETE FROM {twitter} WHERE created_time < %d', time() - $age);
}
the 'FROM' is missing in the sql-statement - unless there's a feature in sql I didn't know yet :)
Note: I have not even tested this module yet, I just got a question from a friend who automatically wants to post his blog entries to twitter and browsing around I bumped on this module and found this error (I think) after a quick look at the code.
Note 2: Did not check the D5 version at this point, but you never know it's in there too :)
Comments
Comment #1
eaton commentedthanks for the report. Just checked a fix for this in, and it will be in the dev version.
Comment #2
eaton commentedComment #3
Anonymous (not verified) commentedAutomatically closed -- issue fixed for two weeks with no activity.