whats wrong with this sql statement

[leocrawf]

i have this simple script that recieve name/value pairs from a flash movie that is suppose to update a feild in a database table. here is it:

<?php
  include_once("conn.php");
  
  $active = ($_GET["active"]);
  $ipc = ($_GET["ipc"]);
  $ipc = ereg_replace("[^A-Za-z0-9 ]", "", $ipc); 
  $active = ereg_replace("[^A-Za-z0-9 ]", "", $active); 



// This is the SQL statement that will update the appropriate table in the database
$query = "UPDATE ipcinfo SET active ='$active', WHERE ipc ='$ipc' , LIMIT = 1";
if (!$result = mysql_query($query)) {
  die(mysql_error());
}
$num_result = mysql_num_rows($result);

 

  if ($num_result == 1) {
  if($row = mysql_fetch_array($result));
   echo"success= 1";
   echo"ipc=$ipc";
   echo"active=$active";
   }
  else
  {
    echo( "success=0" );
  }

  // Clean up
  mysql_close($dblink);
?>

when i run it i get this 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 'WHERE ipc ='' , LIMIT = 1' at line 1"

i am not a php/mysql buff so therefore i dont know what is wrong, because it seems ok. Any help is appreciated.

[Blunty Killer]

// This is the SQL statement that will update the appropriate table in the database
$query = "UPDATE ipcinfo SET active ='$active', WHERE ipc ='$ipc' , LIMIT = 1";

I don't think those commas after the SET and WHERE clause are supposed to be there. Try taking them out and see what happens.

[leocrawf]

i tried it and i am still getting that error.

[Arch_Angel]

Can you print the output of the query unto your page?
I guess something like:

echo $query 

after you construct your query.
But I don't know php, so you would have to manage to print it out.
Should be able to determine from the query why the server doesn't like it.

[Blunty Killer]

One last thing, with the commas out, remove the '=' after LIMIT. SQL usually take those operators without the equal sign. Ex, ORDER BY field, LIMIT no, OFFSET no.

Taken from MySQL SELECT Syntax

SELECT
[ALL | DISTINCT | DISTINCTROW ]
[HIGH_PRIORITY]
[STRAIGHT_JOIN]
[SQL_SMALL_RESULT] [SQL_BIG_RESULT] [SQL_BUFFER_RESULT]
[SQL_CACHE | SQL_NO_CACHE] [SQL_CALC_FOUND_ROWS]
select_expr, ...
[FROM table_references
[WHERE where_condition]
[GROUP BY {col_name | expr | position}
[ASC | DESC], ... [WITH ROLLUP]]
[HAVING where_condition]
[ORDER BY {col_name | expr | position}
[ASC | DESC], ...]
[LIMIT {[offset,] row_count | row_count OFFSET offset}]
[PROCEDURE procedure_name(argument_list)]
[INTO OUTFILE 'file_name' export_options
| INTO DUMPFILE 'file_name'
| INTO @var_name [, @var_name]]
[FOR UPDATE | LOCK IN SHARE MODE]]

[leocrawf]

Thanks Agian For Your Help.

[Arch_Angel]

One last thing, with the commas out, remove the '=' after LIMIT. SQL usually take those operators without the equal sign. Ex, ORDER BY field, LIMIT no, OFFSET no.

You're right.

$query = "UPDATE ipcinfo SET active ='$active' WHERE ipc ='$ipc' LIMIT 1"; 

would be the proper query.

[leocrawf]

i am stILL HAVING THE SAME 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 'WHERE ipc = 123 , LIMIT 1' at line 1

[Arch_Angel]

i am stILL HAVING THE SAME 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 'WHERE ipc = 123 , LIMIT 1' at line 1

Looks like you still using the commas. Check my previous post.

[leocrawf]

WHEN I REMOVE THE COMMAS I GET THIS ERROR:
Warning: mysql_num_rows(): supplied argument is not a valid MySQL result resource in /home/www/SOMEDOMAIN.COM 17
success=0