PHP5 MySQL weirdness

[bucksta001]

Ok, I have a weird problem with my page... I'm putting info into my database from a form.

function save_game() {
    $game_name = addslashes($_REQUEST['game_name']);
    $contact = addslashes($_REQUEST['contact']);

    $query = "INSERT INTO game_data (game_name, contact) VALUES ('$game_name','$contact')";

  	global $dbaddress;
  	global $dbuser;
  	global $dbpass;
  	global $dbname;

  	$dbc = new mysqli($dbaddress, $dbuser, $dbpass, $dbname);
  	$dbc->query($query) or die($dbc->error());
  	$dbc->close();

    echo $query;

    $content = "Game added successfully";

When I submit the data, it's inserted into the database, however MySQL spits back an error:

"INSERT INTO game_data (game_name, contact) VALUES ('Blah','Blah')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"

When I copy / paste the query into phpmyadmin, it works just fine.

Any clue what's going on here? This is on my Test server, which is running Debian64, Apache2, PHP5, and MySQL5.

Thanks!

[sneakyimp]

Given that you are using some object we know nothing about, we don't know if you are using it correctly. Does the $dbc->query() method actually return anything? Perhaps you should be looking for an error condition in the object instead.

[bucksta001]

the $dbc->query() does nothing but query the database with my query, as described above. All I'm trying to do is insert a record into my database, and not return anything.

I've tried inserting the record into the database using both methods of programming, I just happened to post the OO version of it. Both methods give the same error even though the record is actually inserted into the database.

[sneakyimp]

Try checking for error conditions explicitly not only when you query but when you connect. I don't have much experience with the mysqli OO approach so I can't seem to find the constructor function in the documentation. I'm assuming it tries to connect to your DB in the constructor. It may not.

if ($dbc->connect_error) {
    die('Connect Error: ' . $dbc->connect_error);
}

$query_result = $dbc->query($query) or die($dbc->error());
if (is_null($query_result)) {
    echo "dbc->query is null!\n";
    die("this might return an error:" . mysqli_error());
} else {
    die("dbc->query actually returned something and it is:" . $query_result);
}

[bucksta001]

Result:

"dbc->query actually returned something and it is:1"

I assume this is just saying that the query was successful.

[sneakyimp]

if dbc-query returns 1 then I don't understand why the 'or die' part is executing. perhaps try changing this line:

    $dbc->query($query) or die($dbc->error());

to this:

    if (!$dbc->query($query)) {
        die('dbc->query returned an empty value:' . $dbc->error());
    }