Problem. Chapter 6 Page 158(157,158,159,160)

aformoftruth · January 24th, 2010, 05:22 AM

I have a problem for some reason the drop down boxes in movie.php are doubling up the actor and director options? Does anyone know what is wrong?

Here is the code for the three pages used...but only the first two are relevent...

Click on ADD next to movies on the first page and then you see in the drop down boxes they are doubling up.

for admin.php

PHP Code:


			
<?PHP 



$db = mysql_connect('localhost', 'bp6am', 'bp6ampass') or 

    die('Unable to Connect. Please check connection settings.');



mysql_select_db('moviesite', $db) or die(mysql_error($db));



?>



<html>

<head>

<title>Movie Database</title>



<style type="text/css">



th {background-color:#999;}

.odd_row {background-color:#EEE;}

.even_row {background-color:#FFF;}



</style>

</head>

<body>

<table style="width:100%;">

<tr>

    <th colspan="2">Movies <a href="movie.php?action=add">[ADD]</a></th>

</tr>

<?php

$query = 'SELECT * FROM movie';

$result = mysql_query($query, $db) or die(mysql_error($db));



$odd = true;

while ($row = mysql_fetch_assoc($result)){

    echo ($odd == true) ? '<tr class="odd_row">' : '<tr class="even_row">';

    $odd = !$odd;

    echo '<td style="width:75%";>';

    echo $row['movie_name'];

    echo '</td><td>';

    echo '<a href="movie.php?action=edit&id='.$row['movie_id'].'">[EDIT]</a>';

    echo '<a href="delete.php?type=movie&id='.$row['movie_id'].'">[DELETE]</a>';

    echo '</td></tr>';

}

?>

<tr>

    <th colspan="2">People <a href="people.php?action=add">[ADD]</a></th>

</tr>



<?php 

$query = 'SELECT * FROM people';

$result = mysql_query($query, $db) or die(mysql_error($db));

$odd = true;

while ($row = mysql_fetch_assoc($result)){

    echo ($odd == true) ? '<tr class="odd_row">' : '<tr class="even_row">';

    $odd = !$odd;

    echo '<td style="width:25%";>';

    echo $row['people_fullname'];

    echo '</td><td>';

    echo '<a href="people.php?action=edit&id='.$row['people_id'].'">[EDIT]</a>';

    echo '<a href="delete.php?type=people&id='.$row['people_id'].'">[DELETE]</a>';

    echo '</td></tr>';

}

?>

</table>

</body>

</html>

for movie.php

PHP Code:


			
<?php

$db = mysql_connect('localhost', 'bp6am','bp6ampass') or 

    die('Unable to Connect. Plerase Check your conneciton parameters.');



mysql_select_db('moviesite', $db) or die(mysql_error($db));

?>

<html>

<head>

<title>Add Movie</title>

</head>

<body>

<form action="commit.php?action=add&type=movie" method="post">

<table>

<tr>

    <td>Movie Name</td>

    <td><input type="text" name="movie_name"/></td>

</tr>

<tr>

    <td>Movie Type</td>

    <td>

        <select name="movie_type">

<?php

//select the movie type 

$query = 'SELECT 

            movietype_id, movietype_label

        FROM

            movietype

        ORDER BY

            movietype_label';



$result = mysql_query($query, $db) or die(mysql_error($db));



//populate the select options with the results

while ($row = mysql_fetch_assoc($result)){

    foreach ($row as $value){

        echo '<option value="'.$row['movietype_id'].'">';

        echo $row['movietype_label'].'</option>';

    }

}

?>

        </select>

    </td>

</tr>

<tr>

    <td>Movie Year</td>

    <td>

        <select name="movie_year">

        <?php

        //populate the select options with years

        for ($yr = date("Y"); $yr >=1970; $yr--){

            echo '<option value"'.$yr.'">'.$yr.'</option>';

        }

        ?>

        </select>

    </td>

</tr>

<tr>

    <td>Lead Actor</td>

    <td><select name="movie_leadactor">

    <?php

    //select actor records

    $query= 'SELECT

                people_id, people_fullname

            FROM

                people

            WHERE

                people_isactor=1

            ORDER BY

                people_fullname';

    $result = mysql_query($query, $db) or die(mysql_error($db));

     

    //populate the select options with the results

    while($row = mysql_fetch_assoc($result)){

        foreach($row as $value){

            echo '<option value="'.$row['people_id'].'">'.$row['people_fullname'].'</option>';

        }

    }

    ?>

    </select>

    </td>

</tr>

<tr>

    <td>Director</td>

    <td>

    <select name="movie_director">

    <?php

    //select director records

    $query ='SELECT 

                people_id, people_fullname

            FROM

                people

            WHERE

                people_isdirector = 1

            ORDER BY 

                people_fullname';



    $result = mysql_query($query, $db) or die(mysql_error($db));

    

    while ($row = mysql_fetch_assoc($result)){

        foreach ($row as $value){

            echo '<option value="'.$row['people_id'].'">'.$row['people_fullname'].'</option>'; 

        }

    }

    ?>

    </select>

    </td>

</tr>

<tr>

    <td colspan="2" style="text-align:center;">

        <input type="submit" name="submit" value="Add"/>

    </td>

</tr>

</table>

</table>

</form>

</body>

</html>

for commit.php

PHP Code:


			
<?php

$db = mysql_connect('localhost', 'bp6am','bp6ampass') or

die ('Unable to connect. Please check connection parameters');



mysql_select_db('moviesite',$db) or die(mysql_error($db));

?>



<html>

<head>

<title>Commit</title>

</head>

<body>

<?php

switch ($_GET['action']){

    case 'add':

        switch($_GET['type']){

            case'movie':

                $query = 'INSERT INTO 

                            movie(movie_name, movie_year, movie_type, movie_leadactor, movie_director)

                        VALUES

                            ("'.$_POST['movie_name'].'",

                            '.$_POST['movie_year'].',

                            '.$_POST['movie_type'].',

                            '.$_POST['movie_leadactor'].',

                            '.$_POST['movie_director'].')';

                break;

        }

        break;

}



if (isset($query)){

    $result = mysql_query($query, $db) or die(mysql_error($db));

}

                

?>

<p>Done!</p>

</body>

</html>

aformoftruth · January 24th, 2010, 05:27 AM

I downloaded the does from the wrox site and its doing the same thing?

Whats wrong with the code?

oh and movie type is doing it aswell!

Thanks.

Truth

aformoftruth · January 24th, 2010, 05:42 AM

Ok if you take out the foreach statements then it works!

while ($row = mysql_fetch_assoc($result)) {
foreach ($row as $value) {
echo '<option value"' . $row['movietype_id'] . '">';
echo $row['movietype_label'] . '</option>';
}
}