error

[talkster5]

Hi
I am having a problem on one of my scripts i am trying to run. When i go onto the page it gives me the 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 1"

But all line 1 is

<?php require_once('../Connections/Site_Database.php'); ?>

And everything inside that file is correct as it works for other pages that are using it.

Any ideas?

[tsinka]

Hi,

seems like this is a script generated error message and not a PHP error at all.

Check the SQL queries in Site_Database.php. At least one of them causes that error message.

Thomas

[talkster5]

I will check them in a minute,

Thanks

[talkster5]

This is my site database file:

<?php
# FileName="Connection_php_mysql.htm"
# Type="MYSQL"
# HTTP="true"
$hostname_Site_Database = "localhost";
$database_Site_Database = "deadlyh_site";
$username_Site_Database = "username";
$password_Site_Database = "password";
$Site_Database = mysql_pconnect($hostname_Site_Database, $username_Site_Database, $password_Site_Database) or trigger_error(mysql_error(),E_USER_ERROR); 
?>

And this is the query i am trying to run un view_ticket.php which calls the database file:

mysql_select_db($database_Site_Database, $Site_Database);
$sql    = "SELECT * FROM client_support WHERE id=$ticket"; 
$result = mysql_query($sql) or die(mysql_error()); 

[tsinka]

Ok,

change the code to:

mysql_select_db($database_Site_Database, $Site_Database);
$sql    = "SELECT * FROM client_support WHERE id=$ticket"; 
echo "Query: ".$sql."<br>\n";
$result = mysql_query($sql) or die(mysql_error());

to print the query. I think, that the problem might be that $ticket is empty or isn't defined at all.

A guess: If that variable is posted through a form or comes as a get variable and you try to access the variable by $ticket while register_globals is set to off the variable $ticket won't be set. In that case use $GET['ticket'] (get variable) or $POST['ticket'] (post variable) instead of $ticket.

Thomas

[talkster5]

It worked like that the other day but then i think i changed something and now it doesnt work. Because i have exactly the same script in another page and it works fine.

[tsinka]

How does the echoed sql query look like ?

Thomas

[talkster5]

This is the whole script

<?php
mysql_select_db($database_Site_Database, $Site_Database);
$sql    = "SELECT * FROM client_support WHERE id=$ticket"; 
$result = mysql_query($sql) or die(mysql_error()); 

$row = mysql_fetch_array($result);
$_SESSION['ticket'] = $ticket;    

echo "	<table width='200' border='1'>\n";
echo "          <tr>\n";
echo "            <td>Username:</td>\n";
echo "            <td>$row[username]</td>\n";
echo "          </tr>\n";
echo "          <tr>\n";
echo "            <td>Status:</td>\n";
echo "            <td>$row[status]</td>\n";
echo "          </tr>\n";
echo "          <tr>\n";
echo "            <td>Urgency:</td>\n";
echo "            <td>$row[urgency]</td>\n";
echo "          </tr>\n";
echo "          <tr>\n";
echo "            <td>Subject:</td>\n";
echo "            <td>$row[subject]</td>\n";
echo "          </tr>\n";
echo "          <tr>\n";
echo "        </table>\n";
echo "		<br>\n";
echo "      <br>\n";  

?>

[tsinka]

Where does $ticket come from ? Did you add the echo line to check how the query looks like ?

Thomas

[talkster5]

the link to the page is

support.php?ticket=1 or whatever ticket number the person wants to look at. It is all done by the previous page.

[tsinka]

Ok,

I guess that the query looks like
SELECT * FROM client_support WHERE id=
(without an id) if you add the echo statement mentioned in one of my last posts.

in this case add

$ticket=0;
if (isset($_GET['ticket']) && is_numeric($_GET['ticket'])) {
  $ticket = $_GET['ticket'];
}

on top of the script.

Thomas

[talkster5]

ok thanks,

i will try it in a minute.

[talkster5]

Sorry in the delay in getting back to you but it worked,

Thanks.