The report.html file saves the user's description of the aliens in a form field named "aliendescription". However, the report.php file tries to retrieve that information in variable $alien_description from $_POST['description'] whereas it *should* be $_POST['aliendescription'].

In the examples of the output from the PHP file execution, the description field is of course blank, because the description entered was not found by the PHP application. Oddly, this error has been corrected on page 22, but not pages 16 and 24.

You need to modify pages 16 and 24, substituting "aliendescription" for "description" in the PHP variable definition, and update the image of the PHP-generated confirmation on pages 13, 20 and 23 where the description field is blank.

The paragraph says "If there is any PHP code in a web page, it's a good idea to name the file on the web server with .php, not .html". Actually, it is usually required to name it .php to tell the server to parse a script. .html pages will mostly not get parsed and will return the source code of the php script instead.

The text indicates that we want to require input to both the subject and the text fields. The 2nd IF statement only outputs an error message if both fields are blank. The IF statement should use an OR, not an AND.

$_POST['elvismail'] is assigned to the variable $text, but the validation logic statements use the variable $body.

Continuation of the same problem on page 171. If you use the logic in the book as is, then if subject is blank and text is filled in, no email will be sent and no error message will be given. Same thing if subject is filled in and the text field is blank.

The same logic error as on 171 and 172.

You are missing the echo statement below the following statement:

// We know both $subject AND $text are blank"

The echo statement should be:

echo 'You forgot the email subject and body text.<br />';

"Just make sure the 'submit' matches up with the id attribute of the Submit button in the form code." should say "Just make sure the 'submit' matches up with the name attribute of the Submit button in the form code."

I tried using the id attribute instead of the name attribute, and it did not work.

The first line of the comment is incorrect:

We check the value of $_POST"submit']

The line should be:

We check the value of $_POST['submit']

Sample code's textarea tag is indented <?php echo $text; ?> on next line, but doing so caused whitespace to be inserted in form's textarea on initial page load. If no body text is entered in the form, the spaces in the textarea still cause the $text variable to be "not empty" and the email is able to be sent despite the user not having entered any text there.

Removing the indentation got rid of the spaces and enable the validation to work properly again.

The comment:

After entering a name and score and clicking Add, the new score is confirmed and added to the guitarwars table in the database.

This should be changed to:

After entering a name and score and clicking Add, the new score is added to the guitarwars table in the database.

The new score at this point of the coding only gets added to the database, it DOES NOT GET CONFIRMED AND ADDED. The confirmation is performed by the Administrator at a later time by accessing the Admin page.

sql statement is incorrect uses DESCRIBE email_list; should be DESCRIBE guitarwars;

Before creating the table, you have to either create a new database, or decide to add the table to an existing database. Then you have to issue the USE statement; then you can run the CREATE TABLE and INSERT statements.

"With a new column added to the high score database,"?? The database is named "gwdb" and the table is named "guitarwars." I think it should say "With a new column added to the guitarwars table,"

This line:

.....bytes is 1.22MB, or 1,250KB).

Should be replaced by:

.....bytes is 1.28MB, or 1,280KB).

Looks like we are testing for 4 error conditions, but only displaying a message for 3 of them.
The "if ($_FILES['screenshot']['error'] == 0 {" does not display an error if there was an upload problem, you just fall out and nothing happens.
The code that sounds like the right message for this condition is run if there is an error moving the uploaded file from the temp directory to the target. I think we need an appropriate message for this condition. Then by rearranging the '}' we can have the proper message for the upload error statement.

The following line:

.....(GW_MAXFILESIZE / 1024) . ' KB in size.</p>';

Should be changed to:

(GW_MAXFILESIZE / 32) . ' KB in size.</p>';

Seems to be an issue with SQL statements spanning more than one line.
The code in the text kept failing, giving me an error about the SHA() function being undefined. Apparently, it was ending the SQL statement after the double quote at the end of the first line and not concatenating the rest of the statement. So the second line was executing, but it was only a fragment of a statement -- password = SHA('$user_password')";

I removed the concatenation and just let the statement wrap from one line to the next and it worked.

$query = "SELECT user_id, username FROM mismatch_user WHERE username = '$user_username' AND password = SHA('$user_password')";

When conducting the "Test Drive" and adding a image i was getting an error stating "Notice: Undefined index: file ...editprofile.php line 60"
the error on this line looked something like this:
if ($_FILES['file']['error'] == 0) {

Corrected like this and the error goes away:
if ($_FILES['new_picture']['error'] == 0) {

The $query in the book and in the download code is: "SELECT topic_id FROM mismatch_topic ORDER BY category_id, topic_id"

The $query should actually read: "SELECT topic_id FROM mismatch_topic ORDER BY category, topic_id"

Basically, category_id should be replaced with category. Also shown on pages 448 and 456.

In the magnet exercise solution, it only shows "Human Cannonball" and "Team Mascot" being a match for LIKE '%ma%'; however, since the LIKE term is case insensitive and % matches on anything INCLUDING nothing, it should also list "Matador" as a match.

When I take the test drive, I get an error "Notice: Undefined index: sort in C:\Inetpub\wwwroot\hfphp\ch09\riskyjobs\search.php on line 109.

I changed the code from
"$sort = $_GET['sort'];"

to

"isset($_GET['sort']) ? $sort = $_GET['sort'] : $sort = '';"

I got the same error when I ran the code I downloaded from your site.

Hi, me again.

I noticed this earlier than was likely mentioned in the book, but the same error occurs in the downloaded code.

The line 144 in search.php is:
$sort = $_GET['sort'];

But the first sort will not have this variable sent in the GET so it shows an error:

Notice: Undefined index: sort in D:\wamp\www\search.php on line 141

I fixed it as you did with the page GET variable.

$sort = isset($_GET['sort']) ? $_GET['sort'] : '';

Similar to error on 552. The registration form attempts to pre-populate the input fields with variables based on the $_POST data. The first time the script is run these variables do not exist and each field shows a php "Undefined variable" error.

Each field could use a ternary input such as
echo isset($first_name) ? $first_name : '';

The description of the imagefilledellipse points to parameters 2 and 3 as width and height, and parameters 4 and 5 as x and y coordinates. It seems as though they are reversed. Parameters 2 and 3 should represent the x and y coordinates and parameters 4 and 5 should represent the width and height.

Line 49 of index.php has $row[last_name], should be $row['last_name'].