I just wanted to alert everyone that useraccount.php has a problem. The code forgets to include the email field in the form when you are trying to create a new account which will prevent you from creating an account which leads me to another problem/suggestion in the code in this case on the user-transact.php. When creating an account it just verifies it has all the necessary information to do it, if it doesn't without any error message alerting you of missing information(in this case the email since there was no field to type it in) it redirects to the index.php.

I have fixed the code of useraccount.php to include the email when creating an account. These are the changes you need to make:

right before the line where it says: if ($mode == "Edit"){ add (a closing brace) }.
Then right before what it should now read:
"?php
    }
  if ($mode == "Edit") { " add
"?php
 if ($mode == "Create") {
?>
<p>
  Email Address:<br />
  <input type="text" class="txtinput" name="email" maxlength="255"
    value="" />
</p>"
After this changes I could create accounts and thefore continue reading the book.
One more thing, I think Buzzly was the one who wrote this chapter which by the way buzzly I think is the best chapter so far, if you have the time could you confirm that my changes are correct otherwise what are the corrections that need to be made.

Thanks
Christian

__________________
Christian

You are half correct. First, there is no missing bracket. You'll find the correct closing bracket on line 25 of the code. However, you are correct about missing the email during creation. I had actually corrected it, but for some reason the fixed code didn't make it into the book. I apologize for the error.

Modify line 37: to read: and it should work like a charm.

Yes, I did write Chapter 15, and thank you. Let me know if this change helps you.

Michael K. Glass
Author, Beginning PHP, Apache, MySQL Web Development

On the "transact-user.php" page, in the "Modify Account" case (lines 68-79), no "email" data is being sent from the previous page. As a result, the commented lines below are not necessary and must be removed from the code to be able to perform the sql update.

    case 'Modify Account':
      if (isset($_POST['name'])

          //and isset($_POST['email'])

          and isset($_POST['accesslvl'])
          and isset($_POST['userid']))
      {
        $sql = "UPDATE forum_users " .
               "SET

               // email='" . $_POST['email'] . "',

               name='" . $_POST['name'] .
               "', access_lvl=" . $_POST['accesslvl'] .
               ", signature='" . $_POST['signature'] . "' " .
               " WHERE id=" . $_POST['userid'];

Hope this is clear. Let me know if there are any questions.


Bruce

Bruce,
You might want to make sure that the variables in your code are not named $e-mail, or you're right- they won't work. This was a global problem that occurred in the publishing process (gotta love word processor's auto-correct feature, right?) :)

Elizabeth,
Fortunately, I was able to catch the "$e-mail" thing early on, but thanks! However, when I go through "useraccount.php", if $mode == "Modify", the code doesn't send a value for 'email'. Therefore, when "transact-user.php" sees that $_POST['email'] is empty in the 'Modify Account' case, it won't perform the UPDATE and simply redirects back to 'admin.php'. Don't mean to argue here. Us Cincinnati-folk gotta stick together :] Just want to make sure I'm reading the code correctly. Again, thanks. Nice to have the author(s) looking after us.
Bruce

Bruce

You are right, another solution is to leave transact-user.php as it is and add the line
on useraccount.php after the line that says: Good catch. Thanks for posting it.

Christian