php code resulteert in lege pagina

[cschot]

deze code geeft als resultaat een lege pagina dus geen foutmelding. ziet iemand een fout?

PHP:

 
<?php
ini_set('display_errors', 1);  
error_reporting( E_ALL );
if(isset($_POST['submit'])){

// your email address
$youremail = "[e-mail]";

// field validation
if ($email=="" || $comments=="" || $name=="")

{
print ("All fields are required! Please go back and try again.");
}

else {

// send email
$headers = "From: \"$name\" <$email>\n";
$subject = "Feedback Form"; 
$message = "$comments";

mail ("$youremail", "$subject", $message, $headers);
print ("Thank you $name, your email has been sent.");

} 

else{
?>
<form action="formmail.php" method="post">
Name: <input type="text" name="name"><br>

Email: <input type="text" name="email"><br>
Comments: <textarea name="comments"></textarea><br>
<input type="submit" name="submit">
</form>
<?
}
?>

[ Voor 3% gewijzigd door cschot op 28-05-2009 21:35 ]

[doeternietoe]

Zou je even code-tags aan kunnen zetten?

Overzicht van UBB-codes #tag_code

PHP:

error_reporting "e_all";

is zowiezo al fout, om de "e_all" moeten haakjes, voor de syntax. E_ALL is een constante, dus dat klopt ook niet.

Zet in plaats daarvan eens dit boven in je script:

PHP:

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

[ Voor 58% gewijzigd door doeternietoe op 28-05-2009 21:32 ]

[Megamind]

Gebruik code tags En kijk eens naar je eerste regel. http://www.w3schools.com/PHP/func_error_reporting.asp

[Verwijderd]

shorttags niet toegestaan (einde zit er 1)?
zet display_errors ook even aan mocht dat nog niet zijn gebeurd

Om me maar even niet uit te laten over de verdere scripting...

[Onbekend]

cschot schreef op donderdag 28 mei 2009 @ 21:28:

if(!isset($_POST['submit']))

Als er geen form is verzonden gaat hij het verwerken? Volgens mij moet de uitroepteken weg.

[ID-College]

error_reporting "e_all";

Zou je geen () gebruiken?

Daarnaast is je code nou niet echt om over naar huis te schrijven

[Noork]

Ik zou eens debuggen. Lijkt me niet al te moeilijk. Haal gewoon stukken code weg, en kijk waar het script vastloopt. (volgens mij is dat op de eerste regel al )

[MuisM4t]

Op deze manier kan jouw server gebruikt worden voor het versturen van spam.


Zie ook http://www.byte.nl/docs/Mail-Header-Injectie.html

[ Voor 18% gewijzigd door MuisM4t op 28-05-2009 21:38 ]

[cschot]

bedankt,
ik heb hem aangepast zoals jullie zeiden maar het probleem loopt nog

[DataGhost]

Je bent ergens een speciaal teken vergeten. Indent je code eens netjes (dit ziet er echt niet uit), dan zal je direct zien waar de fout zit.

[cschot]

zodra het script werkt ga ik wat tegen injectie doen. maar het werkt nog niet

[frickY]

Waarschijnlijk staat display_errors uit, en heb je een parse-error.
Bij een parse-error wordt je code niet ge-execute, en dus doet jouw ini_set() niets.

Plaats eens het volgende in een .htaccess

code:

php_flag display_errors On

Maak een nieuwe pagina waarin je phpinfo() aanroept om te controleren of display_errors inderdaad true is.

[cschot]

@ dataghost

dankje ik het na de 2e if een accolade vergeten
@allemaal
bedankt!;-) het werkt weer

[ Voor 8% gewijzigd door cschot op 28-05-2009 21:44 ]

[ID-College]

Op regel 28 hoort nog een } gok ik

[DataGhost]

Trouwens, je moet denk ik display_startup_errors hebben en die kan je voor zover ik weet niet in je script instellen, dat zal via een .htaccess moeten mits toegestaan, anders via de server-configuratie.

[cschot]

ik heb hosting dus dat zal wel aanstaan bij mijn provider

[Verwijderd]

Gebruik eerst eens overal netjes $_POST['name'], $_POST['comments'] en $_POST['email']. Op code uit het jaar 2002 zit niemand te wachten.

Verder moet je op je ontwikkelmachine error_reporting globaal op zijn minst op E_ALL zetten, en display_errors aan. In het script zelf is het leuk, maar bij een parse error* zie je dus niks.

En als het niet in php.ini gaat, dan gaat dat wel met een .htaccess bestand.

[edit]
* En 2 keer else gebruiken voor dezelfde conditional levert natuurlijk die parse error op. Accollade } voor regel 28.
Leer goed indenten, leer goed debuggen. Dit is gewoon aanklooien.

[ Voor 26% gewijzigd door Verwijderd op 28-05-2009 21:50 ]

[doeternietoe]

cschot schreef op donderdag 28 mei 2009 @ 21:44:
ik heb hosting dus dat zal wel aanstaan bij mijn provider

Misschien een idee om dat toch even te controleren? Ik zou nooit ergens van uit gaan voordat ik het zeker wist.

[Delusion]

PHP:

<?php
ini_set('display_errors', 1);  
error_reporting( E_ALL );

if( $_SERVER['REQUEST_METHOD'] == 'POST' && isset($_POST['submit']) )
{

    // your email address
    $youremail = "[e-mail]";
    
    // field validation
    if( empty($email) || empty($comments) || empty($name) )
    {
        echo "All fields are required! Please go back and try again.";
    }
    else
    {
    
        // send email
        $headers = 'From: ' . $name ' <' . $email . '>' . "\n";
        $subject = "Feedback Form"; 
        $message = $comments;
        
        mail($youremail, $subject, $message, $headers);
        echo 'Thank you' . $name . ', your email has been sent.';
    
    } 

else
{
    ?>
    <form action="formmail.php" method="post">
    Name: <input type="text" name="name"><br>
    
    Email: <input type="text" name="email"><br>
    Comments: <textarea name="comments"></textarea><br>
    <input type="submit" name="submit">
    </form>
    <?
}
?>

Heb ik even snel gedaan

Probeer dat eens, ik vind je coding echt een rommel, en vars zoals $name zou ik gewoon $_POST laten, want ik zie hier nergens hoe je dat opvangt.

[edit]
Nvm, zie dat het al opgelost is.

[ Voor 3% gewijzigd door Delusion op 28-05-2009 22:02 ]

[cschot]

thanx delusion ik ga dit als basis gebruiken

[cschot]

dit script stuur geen email.

als ik de gegevens heb ingevuld gaat hij door maar hij blijft wit. er komt niet te staan dat de mail verzonden is. in mijn inbox komt de mail ook niet. ik gebruik nu het script wat delusion gemaakt heeft. de error reporting werkt wel, maar er komt nu geen melding.

[ Voor 11% gewijzigd door cschot op 28-05-2009 22:11 ]

[Verwijderd]

Sorry, maar je basis is nog steeds ruk. Je gaat uit van register_globals, je gebruikt een submitknop met name="submit", je gaat ervan uit dat de POST variabelen "email", "comments" en "name" altijd zijn ingevuld, je flikkert zomaar een variabele uit de request in de mail headers...

Allemaal een recept voor een hoop ellende. Ik veeg wekelijks de rommel bij elkaar die zulke scripts veroorzaken. Gebruik sowieso een degelijke mailer class die elke mogelijkheid tot MIME injection afvangt.

Wat gebeurt er volgens jou als ik als naam iets als het volgende invul?

" <[e-mail]>
X-Dit-Hoort-Hier-Niet: foo
X-From: "

Delusion heeft je probleem niet opgelost, hij heeft je code iets verduidelijkt, maar het script bevat nog steeds een parse error.

[ Voor 9% gewijzigd door Verwijderd op 28-05-2009 22:12 ]

[cschot]

ik denk dat bill gates niet blij is:P

het gaat er mij om dat dit script werkt. als het werkt dan ga ik al die anti-spam dingen erin gooien. maar als het nu al niet werkt heeft dat nog geen zin.

hoe moet ik die submit anders noemen dan?
wat moet ik in die .htaccess zetten om die parse error zichtbaar te maken?

[ Voor 99% gewijzigd door cschot op 28-05-2009 22:18 ]

[BasieP]

verder nog iets met error handling:

code:

mail($youremail, $subject, $message, $headers);
echo 'Thank you' . $name . ', your email has been sent.';

dus zelf al faalt de mail functie, je krijgt gewoon een 'goed' melding.

mail() heeft een return value, het is wijs die te gebruiken.

[Raynman]

Misschien gewoon eerst eens een basiscursus PHP zoeken? Of op zijn minst wat doen met alle antwoorden die je ondanks de zeer geringe eigen inzet toch al gekregen hebt...

Heb je dit überhaupt zelf gemaakt? Met dingen als "your email address" lijkt het nogal op een scriptje dat je van een of andere site kunt plukken en dan een variabele veranderen en hopen dat het werkt.

[Mike2k]

@delusion:

PHP:

if( $_SERVER['REQUEST_METHOD'] == 'POST' && isset($_POST['submit']) )

hoeft niet perse, tenslotte, een simpele

PHP:

if (isset($_POST['submit'])) {

zou al volstaan.

@cschot:
That script is bound to get your server blacklisted!
Je zou bijv eens kunnen kijken of het email adres wel een echt email adres kan zijn...
Split het email adres op het "@" en haal het 2de deel (het domein dus) eens door deze functie: http://nl3.php.net/getmxrr
Als die "TRUE" is, dan bestaat er een MX record voor het domein, anders niet...

[Delusion]

@Fastex:
Dat weet ik, maar dat gebruik ik wel als extra check.

[cschot]

het script is idd van internet. ik had eerst een heel uitgebreid script en dat werkte. maar het gaat meer is voor een intern netwerk. waarschijnlijk komt het niet open op internet. ik heb het wel nagekeken voordat ik jullie om raad vroeg maar ik ben beginner dus zie je snel dingen over het hoofd en omdat die error reporting niet werkte(heb ik er zelf ingezet) kon ik dus ook niet op die manier de fout ontdekken. Ik heb wel cursussen voor php gevolgd op internet.

@fastex

ik ga die emailcheck functie er later inzetten. het gaat mij er eerst om of ik vanuit het simpelste kan mailen.

[ Voor 11% gewijzigd door cschot op 28-05-2009 22:36 ]

[cschot]

ik ga hem gewoon helemaal zelf typen en na elk accoladetje testen.

[Cartman!]

Ik gebruik overigens juist alleen

PHP:

if('POST' == $_SERVER['REQUEST_METHOD'])
{
}

want als je een image gebruikt als submit (type="image" name="submit") stuurt de ene browser wel submit en de ander submit_x en submit_y (IE6 geloof ik sowieso). En als jij ineens je name anders noemt dan submit blijft t nu ook nog werken.

[Spinal]

Waarom zou je überhaupt op $_POST['submit'] controleren? Je kunt toch net zo goed gewoon controleren of de POST-variabelen die je wilt hebben zijn ingevuld? (Dus email, comments en name) Met de $_POST['submit'] doe je toch niks uiteindelijk...

[Woy]

cschot schreef op donderdag 28 mei 2009 @ 22:34:
@fastex

ik ga die emailcheck functie er later inzetten. het gaat mij er eerst om of ik vanuit het simpelste kan mailen.

Ik wil niet lullig doen, maar als je met zulke eenvoudige problemen al telkens vast loopt, is het misschien te hoog gegrepen om ervanuit te gaan dat je de emailcheck functie straks wel goed maakt. Er zijn namenlijk enorm veel dingen die je af moet vangen, en dat los je niet op door maar een beetje te proberen totdat het goed lijkt te werken.

Tuurlijk moet iedereen een keer leren, maar je kan dan beter vanaf het begin ook de punten die, onder andere in dit topic, aangedragen worden gebruiken.

Zo is het handig om een fatsoenlijke mailer te gebruiken die al een hoop injectie tegen kan gaan en niet de mail functie. De register-globals die je gebruikt kun je ook beter niet gebruiken. Als laatste zul je inderdaad beter aan identing moeten doen, want dat voorkomt al een hoop problemen met bijvoorbeeld vergeten accolades.

[Cartman!]

ZanderZ schreef op vrijdag 29 mei 2009 @ 11:45:
Waarom zou je überhaupt op $_POST['submit'] controleren? Je kunt toch net zo goed gewoon controleren of de POST-variabelen die je wilt hebben zijn ingevuld? (Dus email, comments en name) Met de $_POST['submit'] doe je toch niks uiteindelijk...

Precies, daarom