Download & Extend
Flag » Issues

Error in SQL Syntax regarding SID

Posted by derhasi on September 28, 2011 at 12:18pm
Project:Flag
Version:6.x-2.x-dev
Component:Flag core
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed (fixed)

Issue Summary

As follow up to #271582: Allow anonymous users to flag content, regarding Comment #20, I had the same error with the current dev.

user warning: 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 '' at line 1 query: SELECT * FROM flag_content WHERE content_type = 'node' AND content_id = 2 AND (uid = 0 OR uid = 0) AND sid =

This error is simply resolved by replacing the last two %s placeholders with the %d (as it is done in the rest of the module). Patch for this will be attached in first comment.

Login or register to post comments

Comments

#1

Posted by derhasi on September 28, 2011 at 12:22pm

And here comes the patch.

AttachmentSize
flag_sql_error_sid-1293448-1.patch 1.71 KB

#2

Posted by derhasi on September 28, 2011 at 12:23pm
Status:active» needs review

#3

Posted by BenK on September 28, 2011 at 5:38pm

Sub

#4

Posted by quicksketch on September 29, 2011 at 9:56pm
Status:needs review» reviewed & tested by the community

Nice one! Thanks for the patch @derhasi. I'll test and commit next time I'm working on Flag.

#5

Posted by quicksketch on October 2, 2011 at 4:18am
Title:error in SQL Syntax regarding SID» Error in SQL Syntax regarding SID
Status:reviewed & tested by the community» fixed

Committed. Thanks @derhasi!

#6

Posted by System Message on October 16, 2011 at 4:21am
Status:fixed» closed (fixed)

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