pjc2003 Posted January 5, 2007 Share Posted January 5, 2007 Hi ive created this page stock_control but i cant seem to get the SQL to pick up the value of a variable part_number that I have used the GET function to bring in from previous page.. the error im getting is: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 '( location , valid) VALUES ('G', '0' ) WHERE part_number='0000below is my code:<?phprequire('./header.php');require('./dbconn.php');$part_number = $_GET['part_number'];$action = 'show_form';if(!empty($_POST)) { $action= 'process_data'; }if($action== 'process_data') {$part_number = $_POST['part_number'];$location_name = trim($_POST['location']);$valid = intval(trim($_POST['valid']));$sql= "UPDATE stockdata ( location , valid) " ."VALUES ('$location_name', '$valid' )WHERE part_number='$part_number' ";$ok= mysql_query($sql) or die(mysql_error()); if($ok){ echo "<p class='ErrorText'>Update OK!</p>"; } else { echo "<p class ='ErrorText'>Update Failed!</p>"; } } else { $action = 'show_form'; } echo $sql; //end running database queryif($action == 'show_form') {?><form method="post" action="stock_control.php"><input type="hidden" name="part_number" value="<?=$part_number?>" /> <p class ='MainText'> Change Location - <select name="location"> <option =1 <?php if ($location_name ="A") echo ' selected '?> >A</option> <option =2 <?php if ($location_name ="B") echo ' selected '?> >B</option> <option =3 <?php if ($location_name ="C") echo ' selected '?> >C</option> <option =4 <?php if ($location_name ="D") echo ' selected '?> >D</option> <option =5 <?php if ($location_name ="E") echo ' selected '?> >E</option> <option =6 <?php if ($location_name ="F") echo ' selected '?> >F</option> <option =7 <?php if ($location_name ="G") echo ' selected '?> >G</option> <option =8 <?php if ($location_name ="H") echo ' selected '?> >H</option> <option =9 <?php if ($location_name ="I") echo ' selected '?> >I</option> <option =10 <?php if ($location_name ="J") echo ' selected '?> >J</option> <option =11 <?php if ($location_name ="K") echo ' selected '?> >K</option> <option =12 <?php if ($location_name ="L") echo ' selected '?> >L</option> <option =13 <?php if ($location_name ="M") echo ' selected '?> >M</option> <option =14 <?php if ($location_name ="N") echo ' selected '?> >N</option> <option =15 <?php if ($location_name ="O") echo ' selected '?> >O</option> <option =16 <?php if ($location_name ="P") echo ' selected '?> >P</option> <option =17 <?php if ($location_name ="Q") echo ' selected '?> >Q</option> <option =18 <?php if ($location_name ="R") echo ' selected '?> >R</option> <option =19 <?php if ($location_name ="S") echo ' selected '?> >S</option> <option =20 <?php if ($location_name ="T") echo ' selected '?> >T</option> <option =21 <?php if ($location_name ="U") echo ' selected '?> >U</option> <option =22 <?php if ($location_name ="V") echo ' selected '?> >V</option> <option =23 <?php if ($location_name ="W") echo ' selected '?> >W</option> <option =24 <?php if ($location_name ="X") echo ' selected '?> >X</option> <option =25 <?php if ($location_name ="Y") echo ' selected '?> >Y</option> <option =26 <?php if ($location_name ="Z") echo ' selected '?> >Z</option> </select></p> <p class ='MainText'> Change Validity- <select name="valid"> <option =1 <?php if ($valid =0) echo ' selected '?> > 0</option> <option =2 <?php if ($valid =1) echo ' selected '?> > 1</option> </select></p> <input type="submit" value="save"/> </p></form> <?php }require('./footer.php');?>Any ideas would be great.. Quote Link to comment Share on other sites More sharing options...
kenrbnsn Posted January 5, 2007 Share Posted January 5, 2007 Why are you creating another post on the same topic? Ken Quote Link to comment Share on other sites More sharing options...
Carterhost Posted January 5, 2007 Share Posted January 5, 2007 Try Changing your SQL to:[code]$sql= "UPDATE stockdata ( location = '$location_name', valid = '$valid' WHERE part_number= '$part_number' ";[/code]That's the only thing I can suggest. Quote Link to comment Share on other sites More sharing options...
kenrbnsn Posted January 5, 2007 Share Posted January 5, 2007 That's invalid, this would be the valid statement:[code]<?php$sql= "UPDATE stockdata set location = '$location_name', valid = '$valid' WHERE part_number= '$part_number' ";?>[/code]Ken Quote Link to comment Share on other sites More sharing options...
Carterhost Posted January 5, 2007 Share Posted January 5, 2007 Whoops! yeah, that's what I meant! Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.