nike_guy_man — Mon, 20 Jan 2003 19:55:02 +0000

<?php
  function validate_form()
  {
    global $subject, $body;

    $errors=0;
    if (!trim($subject))
    {
    echo \"<BR><font color=\\"red\\"><B>Subject</B> is required.</font>\";
    $errors++;
    }

    if (!trim($body))
    {
    echo \"<BR><font color=\\"red\\"><B>Message Body</B> is required.</font>\";
    }

    switch ($errors)
    {
    case 0:
        return TRUE;

        case 1:
        echo \"<BR><BR><font color=\\"red\\">Please use your browser's back button to return to the form.\";
        return FALSE;

        default:
        echo \"<BR><BR><font color=\\"red\\">Please use your browser's back button to return to the form.\";
        return FALSE;
        }
    }

$ok = validate_form();
function update_database()
{
          echo \"<br>Updating database.\";

$fp = fopen (\"/home/westernc/public_html/baseball/data.txt\", \"w\");
  fwrite($fp, \"$subject\");
  fwrite($fp, \"$body\");
  fclose ($fp);
}

if ($ok) {
       update_database();
}


</BODY>
</HTML>
?>

You were missing a semi at the end of 37 (Which I moved anyways)
Also, you didn't have the { and } for the if ($ok) update_Database()

I moved $ok = validate_form() because if you keep it in the function for update_database() it won't run until update_database() is run too
I ran it through the Zend Development Environment Personal Studio and it turned up OK...

brady.k — Mon, 20 Jan 2003 18:22:19 +0000

ok heres the acutal code:

<?php

  function validate_form()
  {
    global $subject, $body;

    $errors=0;
    if (!trim($subject))
    {
    echo "<BR><font color=\"red\"><B>Subject</B> is required.</font>";
    $errors++;
    }

    if (!trim($body))
    {
    echo "<BR><font color=\"red\"><B>Message Body</B> is required.</font>";
    }

    switch ($errors)
    {
    case 0:
        return TRUE;

        case 1:
        echo "<BR><BR><font color=\"red\">Please use your browser's back button to return to the form.";
        return FALSE;

        default:
        echo "<BR><BR><font color=\"red\">Please use your browser's back button to return to the form.";
        return FALSE;
        }
    }

function update_database()
{
          echo "<br>Updating database.";
$ok = validate_form()
$fp = fopen ("/home/westernc/public_html/baseball/data.txt", "w");
  fwrite($fp, "$subject");
  fwrite($fp, "$body");
  fclose ($fp);
}

if ($ok)
       update_database();
?>

</BODY>
</HTML>

i'm getting an error on line 38...anybody see anything...cuz thats the path and everything...? thanks

brady.k — Mon, 20 Jan 2003 18:09:23 +0000

i also just realized that it only validates it...i'm retarded...i'm updating the code...lets see what happens

nike_guy_man — Mon, 20 Jan 2003 02:36:15 +0000

fucntion validate_form() {

should be
function validate_form() {

If you don't fix the double quotes, once it gets past the first error it'll die there too... you'll get an line 10 too if you don't put the ; after $errors = 0

Also, on the site, you must not have an open tag for php

<?php
 </strong> or <strong> <? </strong>depending on what you are allowed to do
<?php
    function update_database()
    {
    echo \"<BR>Updating the database...\";
    }
?>

Does that actually update the DB or just print that?

Busy — Sun, 19 Jan 2003 09:24:35 +0000

you have no ";" after $errors=0
also you are doubling quotes in your echo statments
echo "Subject is required.";
should be
echo "Subject is required.";
or use single quotes around it, there is a couple of others there too

not sure about the rest