Closed (fixed)
Project:
Simple Cart
Version:
6.x-1.2
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Reporter:
Created:
7 Jul 2011 at 10:21 UTC
Updated:
29 Dec 2011 at 09:00 UTC
Jump to comment: Most recent file
Thanks for this module. I install it today and all work fine (i'm check issues and fix two of problems by hand).
But i notice many records like this in my site's log:
Unknown column 'sc.session_id' in 'on clause' query: SELECT sc.sid FROM simple_cart sc LEFT JOIN sessions s ON (sc.session_id = s.sid) WHERE s.sid IS NULL в файле /home/cat/modules6/simple_cart/simple_cart.module в строке 153.
Looks like just typo in query text.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | simple_cart.module.patch | 659 bytes | ami7878 |
Comments
Comment #1
zet commentedsame on manual cron run:
user warning: Unknown column 'sc.session_id' in 'on clause' query: SELECT sc.sid FROM simple_cart sc LEFT JOIN sessions s ON (sc.session_id = s.sid) WHERE s.sid IS NULL in /xxxx/drupal6/sites/all/modules/simple_cart/simple_cart.module on line 154.
Comment #2
johnaut commentedI have changed line 152 in /xxxx/drupal6/sites/all/modules/simple_cart/simple_cart.module to
$sql = "SELECT sc.sid FROM {%s} sc LEFT JOIN {sessions} s ON (sc.sid = s.sid) WHERE s.sid IS NULL";
and now it works for me (the name of the field containing the id of the session in table sessions is sid and not session_id ==> sc.sid instead of sc.session_id).
Comment #3
zet commentedYeah, I've changed that too after cheking the namings in the database and is working, but I didn't have time to report back until now. Waiting for the author to commit this.
Comment #4
visualfox commentedsubscribing - did the same as #2 and #3 - just reporting back
Comment #5
vladsavitsky commentedFixed in HEAD.
Comment #6
ami7878 commentedHad the same issue and thanks to johnaut problem solved.
I've added a patch with johnaut's fix.