what's wrong with this query?

[sabret00the]

even when i use the echo to read it via the top of the page it stops at the table prefix.

SELECT username, userid, email
FROM prs_users
LEFT JOIN " . TABLE_PREFIX . "user AS user ON (user.userid = prs_users.userid)
WHERE prs_users.subscriptions LIKE = "% $display[pieceid] %" AND userid != $getparentinfo[userid]


and i'm calling it like this $thesubscribed = $DB_site->query("
SELECT username, userid, email
FROM prs_users
LEFT JOIN " . TABLE_PREFIX . "user AS user ON (user.userid = prs_user.userid)
WHERE prs_users.subscriptions LIKE = "% $display[pieceid] %" AND userid != $getparentinfo[userid]
");

[Colin F]

Could you post the exact error message?

Also, could it be that prs_users also needs a TABLE_PREFIX?

[sabret00the]

here you go Database error in vBulletin 3.0.7:

Invalid SQL:
SELECT username, userid, email
FROM prs_users
LEFT JOIN
mysql error: 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 3

mysql error number: 1064

and nope prs_users has no table prefix what so ever :(

[The Geek]

Tough to say, however it would at least read a little better and be easier to troubleshoot written like this:


$thesubscribed = $DB_site->query("
SELECT p.username, p.userid, p.email
FROM prs_users p
LEFT JOIN " . TABLE_PREFIX . "user u ON p.userid = u.userid
WHERE p.subscriptions LIKE '%$display[pieceid]%' AND u.userid != $getparentinfo[userid]
");


I dont know if thats right - nor do I know if I got the fields on the right aliases (p for prd_users and u for users).
Hell, this may actually make it harder for you to troubleshoot - I just thought I would trow it in as its the style I would use.

I point to note is that "% $display[pieceid] %" would have given you problems for a number of reasons (in fact, it shouldnt have compiled).

HTH's m8

[sabret00the]

funnily enough i just changed it to read a bit better with this
$thesubscribed = $DB_site->query("
SELECT user.username, prs_users.userid, user.email
FROM prs_users
LEFT JOIN " . TABLE_PREFIX . "user AS user ON (user.userid = prs_users.userid)
WHERE prs_users.subscriptions LIKE = "% $display[pieceid] %" AND prs_users.userid != $getparentinfo[userid]
");


and i know all the colums are there so it's just confusing me.

I point to note is that "% $display[pieceid] %" would have given you problems for a number of reasons (in fact, it shouldnt have compiled).
you're right the single quotes fixed it, thank you :)

though now i'm getting Database error in vBulletin 3.0.7:

Invalid SQL:
SELECT user.username, prs_users.userid, user.email
FROM prs_users
LEFT JOIN user AS user ON (user.userid = prs_users.userid)
WHERE prs_users.subscriptions LIKE = '% 1 %' AND prs_users.userid != 0

mysql error: 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 '= '% 1 %' AND prs_users.userid != 0' at line 4

mysql error number: 1064 not sure what that means :(

nevermind the problem was the = <--- thanks again :)

[akanevsky]

The best and most valid way to write this query would be:

$thesubscribed = $DB_site->query
("
SELECT p.username, p.userid, p.email
FROM prs_users p
LEFT JOIN " . TABLE_PREFIX . "user AS user ON (user.userid = p.userid)
WHERE p.subscriptions LIKE '%" . $display[pieceid] . "%' AND p.userid != '" . $getparentinfo[userid] . "'
");

[Colin F]

The best and most valid way to write this query would be:

$thesubscribed = $DB_site->query
("
SELECT p.username, p.userid, p.email
FROM prs_users p
LEFT JOIN " . TABLE_PREFIX . "user AS user ON (user.userid = p.userid)
WHERE p.subscriptions LIKE '%" . $display[pieceid] . "%' AND p.userid != '" . $getparentinfo[userid] . "'
");
I like to have "FROM prs_users AS p", but it's not as if it would make a difference.

[Marco van Herwaarden]

The best and most valid way to write this query would be: $thesubscribed = $DB_site->query("SELECT p.username, p.userid, p.emailFROM prs_users pLEFT JOIN " . TABLE_PREFIX . "user AS user ON (user.userid = p.userid)WHERE p.subscriptions LIKE '%" . $display[pieceid] . "%' AND p.userid != '" . $getparentinfo[userid] . "'");Most valid would be using also $display['pieceid'] instead of $display[pieceid]. And depending on the column type and where that value is coming from, maybe also measurements to avoid injections, and amybe more but that is difficult to tell based on the info we have. ;)


ohh and yes like mentioned before, TABLE_PREFIX on all tables.