Simpel PHP mail form werkt wel, stuurt geen gegevens.

[MrVegeta]

Ik moet even een snel een mail formpje opzetten en google was me vriend en gaf me al snel een simpel formpje, maar ik krijg het niet 100% werkend. Ik krijg wel een mailtje maar er staan verder geen gegevens in, geen afzender, onderwerp, e-mail adres etc.

Er zijn 2 bestanden waar het om gaat, de contact.php, (allemaal opmaak meuk etc er uit gehaald)

HTML:

<form name="form1" method="post" action="includes/send_contact.php">
<textarea name="detail" cols="50" rows="4" id="detail"></textarea><br />
<<input name="name" type="text" id="name" size="50"><br />
<input name="customer_mail" type="text" id="customer_mail" size="50"><br />
<input type="submit" name="Submit" value="Submit"> <input type="reset" name="Submit2" value="Reset"><br />
</form>

En het PHP bestand, send_contact.php,

PHP:

<?php
// Contact subject
$subject ="$subject";
// Details
$message="$detail";

// Mail of sender
$mail_from="$customer_mail";
// From
$header="from: $name <$mail_from>";

// Enter your email address
$to ='[e-mail]';

$send_contact=mail($to,$subject,$message,$header);

// Check, if message sent to your email
// display message "We've recived your information"
if($send_contact){
echo "We've recived your contact information";
}
else {
echo "ERROR";
}
?>

Nou ben ik zelf niet echt bedreven met PHP maar ik zie hier niet waarom die geen gegevens mee stuurt. Hij krijgt de gegevens zoals subject, message, etc uit contact.php en moet dat toch gewoon meesturen met het mailtje? Maar ipv daarvan krijg ik steeds een leeg mailtje.

[harrald]

edit: never mind (nieuwe forum regels)

[ Voor 84% gewijzigd door harrald op 28-03-2009 18:05 ]

[Bergen]

Staat register_globals aan? Anders werkt dat uiteraard niet met die variabelen zo kopieren uit het formulier... ($subject ="$subject" enzo) En als register_globals niet aan staat, mooi zo laten. Dit is een vreselijk achterhaalde manier om variabelen uit een formulier te halen. Echo eens wat er precies wordt doorgegeven aan het mailcommando. Even debuggen dus.

[ Voor 29% gewijzigd door Bergen op 28-03-2009 18:08 ]

[Onbekend]

De verstuurde variablen uit het form zijn geen normale php-variablen, maar POST-variablen.
Je moet dit bijvoorbeeld gebruiken:
$subject =$_POST["subject"];

Zie hier voor meer info:
http://www.w3schools.com/php/php_post.asp

[kaesve]

hmm. je stuurt de data mee met een POST. is het dan niet de bedoeling dat je ze uit een $_POST[] array haalt?

dus

code:

$subject ="$_POST['subject']";

ipv

code:

$subject ="$subject";

[MrVegeta]

PHP:

<?php
// Contact subject
$subject= $_POST["$subject"];
// Details
$message= $_POST["$detail"];
// Mail of sender
$mail_from= $_POST["$customer_mail"];
// From
$header="from: $name <$mail_from>";

// Enter your email address
$to ='[e-mail]';

$send_contact=mail($to,$_POST["$subject"],$_POST["$message"],$_POST["$header"]);

// Check, if message sent to your email
// display message "We've recived your information"
if($send_contact){
echo "We've recived your contact information";
}
else {
echo "ERROR";}
?>

Zoiets? Wil ook nog niet werken. Ik begrijp wel dat "jullie" geen zin hebben om een help-desk te zijn maar ik wil dit gewoon ff klaar hebben in het weekend en ik heb het gevoel dat het al zo goed als klaar is aangezien de mail al gewoon wordt gestuurd...

Hoe kan ik bekijken of de global_register aanstaat op mijn server?

[TvdW]

nvm

[ Voor 97% gewijzigd door TvdW op 28-03-2009 18:29 . Reden: was al gezegd ]

[Spinal]

Scriptje maken met phpinfo(); erin, daar kun je vinden of register_globals aan of uit staat

[itarix]

Je hebt de variablen iets te goed overgenomen vanuit orginele script

probeer eens bij alle $_POST[""] commando's de dollar tekens weg te halen
$_POST["$subject"]; moet $_POST["subject"]; zijn
Of zou je echt persee enkele haakjes moeten gebruiken?
In dat geval kun je ook $_POST['subject']; proberen.

[harrald]

in je mailform verstuur je iets met bijv de naam: detail
vervolgens wil je de inhoud van detail uit POST pakken met $detail?
waarom?

probeer dit: $detail = $_POST['detail'];

ik raad je echt aan om eens in de basis stof van php te duiken want je gaat op deze manier gevaarlijke scriptjes schrijven.

edit: @hierboven 'string' ondersteund geen variabelen in de string waar "string" dit wel doet.
$foo= 'bla' echo '$foo' //print uit: $foo
$foo= "bla" echo '$foo' //print uit: bla

[ Voor 23% gewijzigd door harrald op 28-03-2009 18:40 ]

[HuHu]

Register_globals is echt heel fout om te gebruiken en moet je dus echt niet aan zetten!

[HuHu]

Ow... daarnaast is het misschien makkelijker om van iets als PHPmailer gebruik te maken. Dat zorgt er voor dat al je dingen gewoon veilig afgehandeld worden en vereenvoudigd het sturen van mails via PHP.

[MrVegeta]

register_globals / off / off. Okay vandaar dat het dus niet werkt. Dus als er iemand nog tips heeft om een simpel formuliertje in elkaar te zetten hoor ik het graag...

[harrald]

MrVegeta schreef op zaterdag 28 maart 2009 @ 18:40:
register_globals / off / off. Okay vandaar dat het dus niet werkt. Dus als er iemand nog tips heeft om een simpel formuliertje in elkaar te zetten hoor ik het graag...

Je hebt nu verscheidene tips gekregen, echter doe je er nog niet veel mee.
Zoek eens uit wat $_POST doet, en bekijk hoe de mail() functie werkt. Dan moet je toch een heel stuk verder komen.

[Verwijderd]

Check de mail functie tutorials op W3Schools eens;
http://www.w3schools.com/PHP/php_mail.asp

Je hebt niet echt veel research gedaan he ? Zoals harrald al zei,
ga je eens verdiepen in de Mail functie. Deze is vrij simpel, en je hebt
de boel zo onder de knie

Denk wel dat je eruit komt met het linkje...

[Bergen]

MrVegeta schreef op zaterdag 28 maart 2009 @ 18:40:
register_globals / off / off. Okay vandaar dat het dus niet werkt.

Nogmaals, NIET AANZETTEN. Het is rete onveilig en staat niet voor niets standaard uitgeschakeld sinds PHP 4.2.0. Zie ook http://nl2.php.net/register_globals.

Dus als er iemand nog tips heeft om een simpel formuliertje in elkaar te zetten hoor ik het graag...

Je hebt er toch al een? Je moet alleen nog even de foutjes eruit halen.

[Janoz]

Wat Harrald zegt.

Verder zijn we hier inderdaad geen afhaal balie, en dat is niet omdat 'we' daar geen zin in hebben, maar een policy om het hier een beetje werkbaar te houden.