LostMyMarbles Posted November 8, 2012 Share Posted November 8, 2012 Hi All My user name says it all, I have now LostMyMarbles and need help. I am very new to PHP and have been trying to work this out over the last couple of days. I just cannot spot why my sub drop down is not getting populated. Can anyone spot my error... I would be very grateful: <link href="images/favicon.ico" rel="icon" /> <?php require_once('Connections/cnTestingMedia.php'); ?> <form action="" method="post"> <select name="client"> <?php mysql_select_db($database_cnTestingMedia, $cnTestingMedia); $client_query = "SELECT clientid, clientname FROM tblclients"; $client_result = mysql_query($client_query); while($client = mysql_fetch_array($client_result)) { if($_POST['client'] == $client['clientid'] ) { echo '<option selected value="' . $client['clientid'] . '">' . $client['clientname'] . '</option>'; } else { echo '<option value="' . $client['clientid'] . '">' . $client['clientname'] . '</option>'; } } ?> </select> <input type="submit" name="submit" value="GO" /><br /><br /> <select name="promotion"> <?php if($_POST['submit']) { echo $_POST['client']; mysql_select_db($database_cnTestingMedia, $cnTestingMedia); $promotion_query = "SELECT promotion_name FROM tblpromotions WHERE clientid = '{$_POST['client']}'"; while($promotion = mysql_fetch_array($promotion_result)) { echo '<option value="' . $promotion['promotion_name'] . '">' . $promotion['promotion_name'] . '</option>'; } } ?> </select> </form> The Client list populates perfectly but NOTHING happens for the Promotion list. Thank you. Quote Link to comment https://forums.phpfreaks.com/topic/270454-dynamic-cascading-drop-downs/ Share on other sites More sharing options...
Andy123 Posted November 9, 2012 Share Posted November 9, 2012 (edited) I don't know if you made an error when pasting your code here, but you are not executing your second query. $promotion_query = "SELECT promotion_name FROM tblpromotions WHERE clientid = '{$_POST['client']}'"; $promotion_result = mysql_query($promotion_query); I also strongly suggest that you use PDO or MySQLi because the mysql_* extension is getting deprecated. Your code is also vulnerable to SQL injection. Please search Google for more information. Edited November 9, 2012 by Andy123 Quote Link to comment https://forums.phpfreaks.com/topic/270454-dynamic-cascading-drop-downs/#findComment-1391312 Share on other sites More sharing options...
LostMyMarbles Posted November 9, 2012 Author Share Posted November 9, 2012 Thanks, I managed to get it working. And I am watching out for the SQL injection. Quote Link to comment https://forums.phpfreaks.com/topic/270454-dynamic-cascading-drop-downs/#findComment-1391353 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.