PHP E-mail zenden werkt niet. - Softwareontwikkeling

zondag 6 september 2015 20:30

Acties:

0 Henk 'm!

Bernhard1990

Topicstarter

Beste,

Ik heb onderstaand formulier gemaakt ik wil graag dat ik een mail ontvang en dat er een mail naar degene gaat die het formulier heeft ingevuld. Ik krijg wel een mail maar degene die het invult niet. Iemand een idee wat er fout gaat?

code:

<?php

ini_set('display_errors', 1);
error_reporting(E_ALL);

if((isset($_GET['submit']) && $_GET['submit'] == '$_post')){ 


$onderwerp = 'Bevestigingsmail';
$webmaster = 'info@voorbeeld.nl';


$emailadres = $_POST['email']; 
$naam = $_POST['fullname'];
$telefoonnummer = $_POST['cellphone'];
$stad = $_POST['city'];
$opmerking = $_POST['comment'];


$body = <<<EOD
<br><hr><br>
E-mail: $email <br>
Naam: $fullname <br>
telefoonnummer: $cellphone <br>
Stad: $city <br>
Opmerking: $comment <br>
EOD;

$headers = "from: $email\r\n";
$headers .= "conntent-type: text/html\r\n";
$success =mail ($webmaster, $onderwerp, $body, $headers);

 }

// Set email variables
$email_to = 'info@voorbeeld.nl';
$email_subject = 'onderwerp mail';

// Set required fields
$required_fields = array('fullname','email','cellphone','city');

// set error messages
$error_messages = array(
    'fullname' => 'Vul je naam in.',
    'email' => 'Vul je e-mailadres in.',
    'cellphone' => 'Vul je telefoonnummer in.',
    'city' => 'Kies een stad.',
    
);

// Set form status
$form_complete = FALSE;

// configure validation array
$validation = array();

// check form submittal
if(!empty($_POST)) {
    // Sanitise POST array
    foreach($_POST as $key => $value) $_POST[$key] = remove_email_injection(trim($value));
    
    // Loop into required fields and make sure they match our needs
    foreach($required_fields as $field) {       
        // the field has been submitted?
        if(!array_key_exists($field, $_POST)) array_push($validation, $field);
        
        // check there is information in the field?
        if($_POST[$field] == '') array_push($validation, $field);
        
        // validate the email address supplied
        if($field == 'email') if(!validate_email_address($_POST[$field])) array_push($validation, $field);
    }
    
    // basic validation result
    if(count($validation) == 0) {
        // Prepare our content string
        $email_content = 'New Website Comment: ' . "\n\n";
        
        // simple email content
        foreach($_POST as $key => $value) {
            if($key != 'submit') $email_content .= $key . ': ' . $value . "\n";
        }
        
        // if validation passed ok then send the email
        mail($email_to, $email_subject, $email_content);
        
        // Update form switch
        $form_complete = TRUE;
    }
}

function validate_email_address($email = FALSE) {
    return (preg_match('/^[^@\s]+@([-a-z0-9]+\.)+[a-z]{2,}$/i', $email))? TRUE : FALSE;
}

function remove_email_injection($field = FALSE) {
   return (str_ireplace(array("\r", "\n", "%0a", "%0d", "Content-Type:", "bcc:","to:","cc:"), '', $field));
}

?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en" lang="en">
<head>



    <title>Contact Form</title>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
    
    <link href="contactform.css" rel="stylesheet" type="text/css" />
    
    <script type="text/javascript" src="https://ajax.googleapis.com/ajax/libs/mootools/1.3.0/mootools-yui-compressed.js"></script>
    <script type="text/javascript" src="validation.js"></script>
    
    <script type="text/javascript">
        var nameError = '<?php echo $error_messages['fullname']; ?>';
        var emailError = '<?php echo $error_messages['email']; ?>';
        var cellphoneError = '<?php echo $error_messages['cellphone']; ?>';
        var cityError = '<?php echo $error_messages['city']; ?>';
    </script>

</head>

<body> 


<div id="formWrap">
<div id="form">
</div>

<form action="contact.php" method="post" id="comments_form">
<?php if($form_complete === FALSE): ?>
    <div class="row"> 
    <div class="label"> Voor en achternaam </div>
    <div class="input"> 
    <input type="text" id="fullname" class="detail" name="fullname" value="<?php echo isset($_POST['fullname'])? $_POST['fullname'] : ''; ?>" /><?php if(in_array('fullname', $validation)): ?><span class="error"><?php echo $error_messages['fullname']; ?></span><?php endif; ?>
    <div class="context"></div>
    </div> 
    </div>
    
    
    <div class="row"> 
    <div class="label"> E-mailadres </div>
    <div class="input"> 
    <input type="text" id="e-mail" class="detail" name="email" value="<?php echo isset($_POST['email'])? $_POST['email'] : ''; ?>" /><?php if(in_array('email', $validation)): ?><span class="error"><?php echo $error_messages['email']; ?></span><?php endif; ?>
    <div class="context"></div>
    </div> 
    </div>
    
    <div class="row"> 
    <div class="label"> Telefoonnummer </div>
    <div class="input"> 
    <input type="number" id="cellphone" class="detail" name="cellphone" value="<?php echo isset($_POST['cellphone'])? $_POST['cellphone'] : ''; ?>" /><?php if(in_array('cellphone', $validation)): ?><span class="error"><?php echo $error_messages['cellphone']; ?></span><?php endif; ?>
    <div class="context"></div>
    </div> 
    </div>
    
    <div class="row"> 
    <div class="label"> In welke stad? </div>
    <div class="input"> 
    <select>
  <option value="Utrecht">Utrecht</option>
</select>
    <input type="dropdown" id="city" class="" name="city" value="<?php echo isset($_POST['city'])? $_POST['city'] : ''; ?>" /><?php if(in_array('city', $validation)): ?><span class="error"><?php echo $error_messages['city']; ?></span><?php endif; ?>
    </div> 
    </div>
    
    
    <div class="row"> 
    <div class="label"> Opmerkingen </div>
    <div class="input"> 
    <textarea id= "comment" name="comment" class="mess"><?php echo isset($_POST['comment'])? $_POST['comment'] : ''; ?></textarea><?php if(in_array('comment', $validation)): ?><span class="error"><?php echo $error_messages['comment']; ?></span><?php endif; ?>
    </div> 
    </div>
    

    <div class="submit"> 
    <input type= "submit" id="submit" name="submit" value="send message" />
    </div>
    </form> 
<?php else: ?>
<p>Thank you for your Message!</p>

<script type="text-javascript">


window.setTimeout("location=('index.html');",5000)

</script>

<?php endif; ?>

</div> 
</body> 

</html>

zondag 6 september 2015 20:46

Acties:

0 Henk 'm!

azerty

Ja. Als je je logica eens goed bekijkt in de code zou je het ook zelf moeten kunnen zien

zondag 6 september 2015 20:49

Acties:

0 Henk 'm!

CH4OS

It's a kind of magic

Is het niet makkelijker om de PHPMailer class te gebruiken?

Dan kun je heel simpel je headers zetten middels methods, zie ook de simple example.

[ Voor 7% gewijzigd door CH4OS op 06-09-2015 20:50 ]

zondag 6 september 2015 21:07

Acties:

0 Henk 'm!

Bernhard1990

Topicstarter

Dat is juist waarom ik het hier vraag ;-)

zondag 6 september 2015 21:12

Acties:

0 Henk 'm!

CH4OS

It's a kind of magic

Bernhard1990 schreef op zondag 06 september 2015 @ 21:07:
Dat is juist waarom ik het hier vraag ;-)

Er wordt echter wel verwacht, dat je zelf ook wat van je eigen code debugged, als je ervoor kiest om je eigen code te gebruiken en dat is wat azerty zegt.

Je fout is niet moeilijk. Wellicht vergeet je bijvoorbeeld iets.

Overigens heeft PHP tegenwoordig ook zelf al filters, waarmee bijvoorbeeld e-mailadressen worden gevalideerd. Lijkt me een stuk betrouwbaarder dan regular expressions.

[ Voor 32% gewijzigd door CH4OS op 06-09-2015 21:15 ]

zondag 6 september 2015 21:16

Acties:

0 Henk 'm!

Compizfox

Bait for wenchmarks

CptChaos schreef op zondag 06 september 2015 @ 20:49:
Is het niet makkelijker om de PHPMailer class te gebruiken? Dan kun je heel simpel je headers zetten middels methods, zie ook de simple example.

Dit, ja.

Uit bovengelinkte pagina:

Formatting email correctly is surprisingly difficult. There are myriad overlapping RFCs, requiring tight adherence to horribly complicated formatting and encoding rules - the vast majority of code that you'll find online that uses the mail() function directly is just plain wrong! Please don't be tempted to do it yourself.

Zodra je iets gaat doen dat ingewikkelder wordt dan een plain text email verzenden naar één ontvanger, zou ik gewoon naar PHPMailer grijpen.

Gewoon een heel grote verzameling snoertjes

zondag 6 september 2015 22:46

Acties:

0 Henk 'm!

NMe

Quia Ego Sic Dico.

Sorry, maar ik ga ook dit topic weer sluiten, nogmaals wijzend op De Quickstart, net zoals ik dat eerder deed. Ik zie totaal geen zelfredzaamheid of eigen inzet terug in dit topic: je dumpt ~200 regels code, zegt dat het niet werkt en hoopt dat wij wel even voor je gaan debuggen.

Het antwoord op dit topic is hierboven al gegeven en daarmee exact hetzelfde als bij all andere tientallen topics hierover: tenzij je diepgaande kennis hebt over alles rondom e-mail kom je er niet door domweg de mail-functie te gebruiken, en zelfs als je die kennis wél hebt is het duizenden keren makkelijker om PHPMailer of een vergelijkbare tool te implementeren die je al dat werk uit handen neemt.

Aangezien je niet gezocht lijkt te hebben noch eigen inzet toont om je eigen probleem op te lossen doe ik ook dit topic op slot.

'E's fighting in there!' he stuttered, grabbing the captain's arm.
'All by himself?' said the captain.
'No, with everyone!' shouted Nobby, hopping from one foot to the other.