found solution on this forum now have new error!

xkatx21x · #1 Oct 24th, 2007, 13:25

I used a tutorial to make an email form using php.

First I was only recieving 1 piece of info from the form in the email, then i was recieving an error message but I thought I had found the solution here http://www.webforumz.com/php-forum/8...il-expects.htm

But now I recieve this error message

"Parse error: syntax error, unexpected T_ELSE in D:\vhosts\solidmasonry.co.nz\httpdocs\sendmail.php on line 34"

the code i'm using is

PHP: Select all


<?php
  $title = $_REQUEST['title'] ;
  $firstname = $_REQUEST['firstname'] ;
  $lastname = $_REQUEST['lastname'] ;
  $email = $_REQUEST['email'] ;
  $location = $_REQUEST['location'] ;
  $locother  = $_REQUEST['locother'] ;
  $message = $_REQUEST['message'] ;
  $findus = $_REQUEST['findus'] ;

  if (!isset($_REQUEST['email'])) {
    header( "Location: 

http://www.solidmasonry.co.nz/quote.html" );
  }
  elseif (empty($email) || empty($message)) {
    ?>

    <html>
    <head><title>Error</title></head>
    <body>
    <h1>Error</h1>
    <p>
    Oops, it appears you forgot to enter either your
    email address or your message. Please press the BACK
    button in your browser and try again.
    </p>
    </body>
    </html>

    <?
  }
  
$embody = "$title $firstname $lastname $email $location 

 $locother $message $findus";

else {
    mail( "Myemail@gmail.com", "Quote Form",
 $message $embody, "From: $email" );
    header( "Location: 

http://www.solidmasonry.co.nz/index.html" );
  }
?>

Can any one see the problem?
I'm a complete virgin to php!

Rakuli · #2 Oct 24th, 2007, 13:33

} <--- (Remove this curly brace

$embody = "$title $firstname $lastname $email $location $locother $message $findus";

xkatx21x · #3 Oct 24th, 2007, 13:45

tried removing the curly bracket but still getting the same error message!

Any other suggestions??

Rakuli · #4 Oct 24th, 2007, 13:49

Add curly brace there --> }else {

mail( "Myemail@gmail.com", "Quote Form", $message $embody, "From

xkatx21x · #5 Oct 24th, 2007, 13:53

now it says the error is on line 36!!
AAAAAAARRRRRRRGGGGGGGGGHHHHHHH!!!!!!!!

Rakuli · #6 Oct 24th, 2007, 14:07

Few deep breaths -- everyone gets an annoying but or 30

PHP: Select all


<?php

  $title = $_REQUEST['title'] ;

  $firstname = $_REQUEST['firstname'] ;

  $lastname = $_REQUEST['lastname'] ;

  $email = $_REQUEST['email'] ;

  $location = $_REQUEST['location'] ;

  $locother  = $_REQUEST['locother'] ;

  $message = $_REQUEST['message'] ;

  $findus = $_REQUEST['findus'] ;



if (!isset($_REQUEST['email'])) {

    header( "Location: http://www.solidmasonry.co.nz/quote.html" );



} else if (empty($email) || empty($message)) {



?>



    <html>

    <head><title>Error</title></head>

    <body>

    <h1>Error</h1>

    <p>

    Oops, it appears you forgot to enter either your

    email address or your message. Please press the BACK

    button in your browser and try again.

    </p>

    </body>

    </html>

<?

} else {



    $embody = "$title $firstname $lastname $email $location $locother $message $findus";



    mail( "Myemail@gmail.com", "Quote Form",

     $message $embody, "From: $email" );

    header( "Location: http://www.solidmasonry.co.nz/index.html" );

  }

?>

xkatx21x · #7 Oct 24th, 2007, 14:16

Parse error: syntax error, unexpected T_VARIABLE in D:\vhosts\solidmasonry.co.nz\httpdocs\sendmail.php on line 70

Line 70 is
mail( "Myemail@gmail.com", "Quote Form",

Rakuli · #8 Oct 24th, 2007, 14:21

add a fullstop between $message and $embody

xkatx21x · #9 Oct 24th, 2007, 14:50

New error message

Warning: Cannot modify header information - headers already sent by (output started at D:\vhosts\solidmasonry.co.nz\httpdocs\sendmail.php :2) in D:\vhosts\solidmasonry.co.nz\httpdocs\sendmail.php on line 71

alexgeek · #10 Oct 24th, 2007, 15:04

put ob_start(); at the top of your code and ob_end_flush just after the header.

xkatx21x · #11 Oct 24th, 2007, 15:05

This is doing my head in now!

this is the code for the form

Code: Select all

<form method="post" action="sendmail.php">
<select name="title">
<option value="mr">Mr</option>
<option value="mrs">Mrs</option>
<option value="miss">Miss</option>
<option value="ms">Ms</option>
<option value="other">Other</option>
</select><br/>
  First Name:<input name="firstname" type="text" /><br/>
  Last Name:<input name="lastname" type="text" /><br/>
  Email: <input name="email" type="text" /><br />
  Location: <select name="location">
<option value="city">Auckland - Auckland City</option>
<option value="franklin">Auckland - Franklin</option>
<option value="gbisland">Auckland - Great Barrier Island</option>
<option value="helensville">Auckland - Helensville</option>
<option value="hibcoast">Auckland - Hibiscus Coast</option>
<option value="manukau">Auckland - Manukau City</option>
<option value="northshore">Auckland - North Shore</option>
<option value="papakure">Auckland - Papakura City</option>
<option value="waitakere">Auckland - Waitakere City</option>
<option value="warkworth">Auckland - Warkworth</option>
<option value="wellsford">Auckland - Wellsford</option>
<option value="other">Other</option>
</select></br>
If other location please tell us: <input name="locother" type="text" /><br/>
  What work do you require?:<br />
  <textarea name="message" rows="15" cols="40">
  </textarea><br />
Where did you hear about us?<input name="findus" 

type="text" /><br/>
  <input type="submit"  value="Request Quote"/>
</form>

all I want is for the information selected by the client to be sent to my email.

Somebody HELP please *flutters eyelashes*

alexgeek · #12 Oct 24th, 2007, 15:10

Did you do what I just said? Any errors?

xkatx21x · #13 Oct 24th, 2007, 15:23

I think I did what you said

PHP: Select all



    <?php

      ob-start();

      $title = $_REQUEST['title'] ;

      $firstname = $_REQUEST['firstname'] ;

      $lastname = $_REQUEST['lastname'] ;

      $email = $_REQUEST['email'] ;

      $location = $_REQUEST['location'] ;

      $locother  = $_REQUEST['locother'] ;

      $message = $_REQUEST['message'] ;

      $findus = $_REQUEST['findus'] ;



    if (!isset($_REQUEST['email'])) {

        header( "Location: 

http://www.solidmasonry.co.nz/quote.html" );



    } else if (empty($email) || empty($message)) {



    ?>



        <html>

        <head><title>Error</title></head>

        <body>

        <h1>Error</h1>

        <p>

        Oops, it appears you forgot to enter either your

        email address or your message. Please press the BACK

        button in your browser and try again.

        </p>

        </body>

        </html>

    <?

    } else {



        $embody = "$title $firstname $lastname $email $location 

$locother $message $findus";


        mail( "katrina.lemon@gmail.com", "Quote Form",

          $message. $embody, "From: $email" );

        header( "Location: 

http://www.solidmasonry.co.nz/index.html" );

      }

ob_end_flush

    ?>

this is is error code

Fatal error: Call to undefined function: start() in D:\vhosts\solidmasonry.co.nz\httpdocs\sendmail.php on line 4

maybe i misunderstood your instructions

alexgeek · #14 Oct 24th, 2007, 15:24

Use:

PHP: Select all


ob_start()

not:

PHP: Select all


ob-start()

xkatx21x · #15 Oct 24th, 2007, 15:28

changed that sorry

Error message

Warning: Cannot modify header information - headers already sent by (output started at D:\vhosts\solidmasonry.co.nz\httpdocs\sendmail.php :2) in D:\vhosts\solidmasonry.co.nz\httpdocs\sendmail.php on line 73

		Webforumz.com > Main Forums > Program Your Website > PHP Forum
found solution on this forum now have new error!