Question

I would like to ask, is there a better way to run this code. I have a form with a select of #BA_customer which when selected populates a second menu with the address of the selected client. I also need to display a dept dropdown based on the customer selecetion. I think the code I have is causing a conflict where the customer is being passed twice in 2 seperate statements. What is the correct way to do this? Many thanks

    <script language="javascript" type="text/javascript">
          $(function() {
                $("#BA_customer").live('change', function() { if ($(this).val()!="")
                $.get("../../getOptions.php?BA_customer=" + $(this).val(), function(data) {
                $("#BA_address").html(data); });
            }); 
          });
    </script>
    <script language="javascript" type="text/javascript">
          $(function() {
                $("#BA_customer").live('change', function() { if ($(this).val()!="")
                $.get("../../getDept.php?BA_customer=" + $(this).val(), function(data) {
                $("#BA_dept").html(data); });
            }); 
          });
    </script>

+++++++ dept.php Code +++++++++++++++++++++

$customer = mysql_real_escape_string( $_GET["BA_customer"] ); // not used here, it's the customer choosen

    $con = mysql_connect("localhost","root","");
    $db = "sample";
      if (!$con)
        {
        die('Could not connect: ' . mysql_error());
        }

        mysql_select_db($db, $con);
        $query_rs_select_dept = sprintf("SELECT * FROM departments where code = '$customer'");
        $rs_select_dept = mysql_query($query_rs_select_dept, $con) or die(mysql_error());
        $row_rs_select_dept = mysql_fetch_assoc($rs_select_dept);
        $totalRows_rs_select_dept = mysql_num_rows($rs_select_dept);



          echo '<label for="dept">Select a Department</label>'.'<select name="customerdept">';
          echo '<option value="">Select a Department</option>';
          while ($row_rs_select_dept = mysql_fetch_assoc($rs_select_dept))
          {
          $dept=$row_rs_select_dept['name'];
          echo '<option value="dept">'.$dept.'</option>';
          }
          echo '</select>';

++++ SOLUTION +++++++++++++

mysql_select_db($db, $con);
$query_rs_dept = sprintf("SELECT * FROM departments where code = '$customer'");
$rs_dept = mysql_query($query_rs_dept, $con) or die(mysql_error());
$totalRows_rs_dept = mysql_num_rows($rs_dept);

echo '<label for="dept">Select a Department</label>'.'<select name="customerdept">';
echo '<option value="">Select a Department</option>';
while ($row_rs_dept = mysql_fetch_assoc($rs_dept))
{
       $dept=$row_rs_dept['name'];
       echo '<option value="dept">'.$dept.'</option>';
}
echo '</select>';

Remove the first $row_rs_select_dept = mysql_fetch_assoc($rs_select_dept); from the script and it works. Just posted solution in case some other soul needs help with problem like this.

Answer 1

Maybe you could send the two requests together?

$(function() {
    $("#BA_customer").live('change', function() { 
        if($(this).val()!="")
            $.get("../../getOptions.php?BA_customer=" + $(this).val(), function(data) {
                $("#BA_address").html(data); 
            });
            $.get("../../getDept.php?BA_customer=" + $(this).val(), function(data) {
                $("#BA_dept").html(data); 
            });
        }); 
});