doesnt it do you head in when u cant see such a minor error

[newbreed65]

i know this is a very simply error were im just missing or got to many " or ' some were but im tried adding and removin ones and had no luck

 

heres a lil bit of were im getting a error in my code

 

        case "movietype":
          $sql = "UPDATE movietype SET
                    movietype_name = '" . $_POST['movie_name'] . "',
                  WHERE movietype_id = '" . $_GET['id'] . "'";
          break;

 

and this is the error im getting...

 

Invalid query: 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 movietype_id = '4'' at line 3

[laffin]

 movietype_name = '" . $_POST['movie_name'] . "',

 

did u try removing that comma at the end?

 

 

[Mchl]

$sql = "UPDATE movietype SET
movietype_name = '{$_POST['movie_name']}'
WHERE movietype_id = '{$_GET['id']}'";

 

Also note, that you're opening yourself for sql injection this way.

 

Safer way

$movieName = mysql_real_escape_string($_POST['movie_name']);
$id = (int)$_GET['id'];   //I'm assuming that id is numeric here
$sql = "UPDATE movietype SET
movietype_name = 'movieName'
WHERE movietype_id = '$id'";

 

 

[newbreed65]

thank Mchl it updating now just got to make a few changes so it inserting the right data