Cron - PDOException: SQLSTATE[42000]: Syntax error or access violation [#1617488]

Hi,
I use the shoutbox module on a block and on a page and it goes well.
In my "recent log messages" I get an error message everytime cron is running.

PDOException: SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'FROM shoutbox s WHERE (s.created < '1307347830')' at line 1: SELECT FROM {shoutbox} s WHERE (s.created < :db_condition_placeholder_0) ; Array ( [:db_condition_placeholder_0] => 1307347830 ) in shoutbox_cron() (line 133 of /var/www/virtual/MY_URL/MY_SUBDOMAIN/htdocs/sites/all/modules/shoutbox/shoutbox.module).

Is this a bug or a false config for shoutbox or cron?

rappelkiste_src

Comment	File	Size	Author
#5	addfieldlist-1617488-4.patch	610 bytes	justdave

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Comment #1

Taxoman CreditAttribution: Taxoman commented 15 June 2012 at 14:25

I just enabled 7.x-1.0-alpha2 on a web site with many other modules, but not using JavaScript aggregation, and cron runs fine without errors. (Drupal 7.14 / PHP 5.3.10). I am not using auto-updates in Shoutbox, though.

Comment #2

rappelkiste_src CreditAttribution: rappelkiste_src commented 19 June 2012 at 11:37

Hi Taxoman,

I use shoutbox 7.x-1.0-alpha2, too. Auto-refresh for the Shoutbox is set to "0". Javascript aggregation is off.
The error message from my first post comes up by every run of cron. Drupal ist in 7.14.

Any other tipps?

rappelkiste_src

Comment #3

justdave CreditAttribution: justdave commented 12 September 2012 at 06:31

I'm using 7.x-1.2 and getting this also.

Comment #4

justdave CreditAttribution: justdave commented 12 September 2012 at 06:49

Did a little troubleshooting and found the error. It's in the code that prunes expired shouts if you have expiration enabled. The db_select() call is failing to specify which fields it's intending to retrieve (which is why the SQL syntax error since you're essentially doing "SELECT FROM" instead of "SELECT {fieldlist} FROM" or equivalent.

I'll have a patch coming as soon as I get a shout to expire so I can make sure it actually works.

Comment #5

justdave CreditAttribution: justdave commented 12 September 2012 at 07:06

File	Size
addfieldlist-1617488-4.patch	610 bytes

Comment #6

justdave CreditAttribution: justdave commented 19 September 2012 at 02:03

Status:

Active

» Needs review

Comment #7

rappelkiste_src CreditAttribution: rappelkiste_src commented 20 September 2012 at 14:24

Hello justdave,
thanks for the patch.
Unfortunately, I can still not install a patch.
That's on my list, just behind the point to get learn better English. :-)
Sorry I can therefore give no return info, today.

Comment #8

floppy CreditAttribution: floppy commented 3 October 2012 at 20:48

Hi Justdave,

The patch from #5 (i.e. my implementation by simple editing the file, adding the missing line) worked fine for me. Cron no longer gives the PDOException. Thanks, and for the first time old shouts got removed too!

View changes

#5 working fine.. this should be committed.

Comment #12

justdave CreditAttribution: justdave commented 7 March 2014 at 00:58

I filed https://drupal.org/node/2207965 a couple days ago completely forgetting about this bug. Yeah, the bug is still here, and this patch still fixes it.