// 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

eaton’s picture

thanks for the report. Just checked a fix for this in, and it will be in the dev version.

eaton’s picture

Status: Active » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)

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