[PHP/Javascript] waarden worden niet goed doorgegeven

[Verwijderd]

Beste,

ik ben op het moment bezig aan een website, en onder andere hiervoor heb ik een mail class geschreven. Het gaat om de volgende pagina: http://www.aqua-well.be/dev/contact.php

Het gaat me vooral om het onderwerp, en het bericht, vooral na het leeglaten van deze 2 velden. In mijn klasse heb ik het zo gemaakt dat er dan een foutmelding moet komen. Helaas, als ik 1 of beide velden leeglaat, word de mail wel degelijk verzonden. Als ik de waarden invul, worden de waarden ook geechoed, en werkt het zoals het moet zijn. Echter, als ik deze leeglaat, worden de waarden niet geechoed (logisch uiteraard), maar in mijn klasse wordt de controle toch omzeild (zowel empty() als isset() als $waarde == "" geprobeerd). Ik snap gewoon niet waar mijn fout zit, ik kan het niet vinden.

Ik had initieel in mijn klasse de waarde $subject en $msg staan, en gaf deze ook op als parameters in mijn functies (dus Subject($subject) en Body($msg) ). De waarden van mijn klasse heb ik ondertussen al een hoofdletter gegeven, om toch een verschil te krijgen.

Ik ben nu ten einde raad, ik weet gewoon niet waar de fout zit, hoogstwaarschijnlijk gaat het gewoon iets stoms zijn, maar ik staar me er blind op en heb al veel geprobeerd.

De broncodes:

contact.php: hier staat gewoon het contactformulier

HTML:

        <div id="content"><?php require('menu.php'); ?><p id="text">U kan ons bereiken via mail door onderstaand formulier in te vullen, of u kan ook telefonisch contact met ons opnemen. </p>
        <script src="functions/xmlhttp.js"></script>
        <script src="functions/mail.js"></script>


<form id="cssform" name="cssform" action="" method="post">
<label for="name">Naam:</label><input type="text" name="name" id="name" /><hr />
<label for="email">Emailadres:</label><input type="text" name="email" id="email" /><hr />
<label for="subject">Onderwerp:</label><input type="text" name="subject" id="subject" /><hr />
<label for="msg">Bericht:</label><textarea name="msg" id="msg" rows="6" cols="35"></textarea><hr />
<input type="button" name="submit" class="submit" value="Verzend" onclick="sendMail();" /><br /><br  /><input type="reset" name="reset" class="submit" value="Herstel velden" />
</form>

        
        </div>

sendmail.php: hier wordt de mailclass geinclude en uitgevoerd

PHP:

$from = $_POST['name'];
$email = $_POST['email'];
$subject = $_POST['subject'];
$body = $_POST['msg'];
echo "//DEBUG sendmail.php<br />";
echo "From: ".$from."<br />";
echo "Email:".$email."<br />";
echo "Subject: ".$subject."<br />";
echo "Message: ".$body."<br />";
echo "//END DEBUG sendmail.php";

include ('mailclass.php');

$m = new Email;
$m->Sender($from);
$m->From($email);
$m->To("[e-mail]");
$m->Subject($subject);
$m->Body($body);
$m->CC($email);
$m->Send();

mailclass.php: de eigenlijke klasse

PHP:

class Email{

    var $error = array();
    
    var $to = array();
    
    var $xheaders = array();
    
    var $Subject = "";
    
    var $Msg = "";
    
    var $adress = "";
    
    var $headers = "";
    
    var $standardsubject = "Aqua-Well Mailer: vraag/opmerking van bezoeker website";//Dit per project aanpassen
    
    var $contactlink = "<a href='contact.php'>hier</a>";



    //email adres op geldigheid controleren
    function Validate_mail($adress){
    
    
        if (!ereg("^[_a-zA-Z0-9-]+(\.[_a-zA-Z0-9-]+)*@([a-zA-Z0-9-]+\.)+([a-zA-Z]{2,4})$", $adress)){
        
                $this->error[].= "U heeft een <strong>ongeldig email-adres</strong> ingevoerd. Voer een geldig e-mailadres in in de vorm van [e-mail]";
                
                
            }  else {
                
                $this->adress = $adress;
                
            }
                
            
    
    }//einde controle geldigheid mail


    //controleren of het emailadres correct is ingevuld.
    function From($from){
    
        if(!empty($from)){
        
            $this->Validate_mail($from);
            $this->xheaders["Reply-To"] = $from;
        
        } else {
        
            $this->error[].= "U dient uw email adres in te vullen";
        
        }
        
    
    }//einde controle mail
    
    function Sender($name){
    
        if(empty($name)){
        
            $this->error[] .= "U dient uw naam in te vullen";
        
        } else {
        
            $this->xheaders["From"] = $name;

        
        }
    
    }
    
    
    
    //te ontvangen emailadres
    function To($to){
    
        if(empty($to)){
        
            $this->error[].= "interne fout: er is geen bestemmingsadres opgegeven...";
            mail("[e-mail]", "Fout in de Aqua-well website...", "Kijk in het sendmail bestand de bestemmeling na...");
        
        } else {
        
            $this->Validate_mail($to);
            
            $this->to = $to;
        
        }
    
    }//einde ontvanger
    
    //onderwerp instellen
    function Subject($subject){
    
    echo "<br /><br />Aan class doorgegeven subject: ". $subject;
    
        if($subject == ""){
        
            $this->error[] .= "<li>".$this->standardsubject."</li>";
            echo "LEEG SUBJECT!!!!!";
        
        } else {
        
            ucfirst(strip_tags($subject));
            $this->Subject = $subject;
        }
        
    
    }//einde onderwerp instellen
    
    
    //Carbon Copy instellen
    
    function CC($cc){   
        
            $this->xheaders["CC"] = strip_tags($cc);
    
    }//einde Carbon Copy
    
    //opstellen van het bericht
    
    function Body($message){
    
        strip_tags($message);
        
        echo "<br /><br />Aan class doorgegeven bericht: ". $message."<br />";
    
        if(empty($message)){
        
            $this->error[] .= "U dient een bericht in te vullen.";
            
        } else {
        
            $this->Msg = $message;
            
        }
        
    
    }//einde opstellen bericht
    
    
    //verstuur de mail
    function Send(){
    
        
        if(empty($this->error)){
        
            
                foreach($this->xheaders as $key => $value){
                
                    $this->headers .=  "$key: $value\n";
            
                }
                
                $this->headers .= "X-Mailer: PHP/".phpversion();
                $this->headers .= "'Content-Type: text/html; charset=\"windows-1251\"'";
                $this->headers .= "'MIME-Version: 1.0 '";
            
            if(!mail($this->to, $this->subject, $this->msg, $this->headers)){
            
                echo "Er is een probleem opgetreden met het verzenden van de mail. De webmaster is hiervan op de hoogte gebracht door het systeem. Wij trachten dit zo snel mogelijk op te lossen.";
                
                mail("[e-mail]", "Systeemfout in Aqua-well website", "Er is een fout opgetreden bij het mailen. Kijk sendmail.php en mailclass.php na in de map functions.");
            
            } else {
            
                echo "<hr />Uw bericht is succesvol verzonden. Wij trachten u zo snel mogelijk van antwoord te dienen.";
                echo "<br />to = ".$this->to;
                echo "<br />subject = ".$this->subject;
                echo "<br />body = ".$this->msg;
                echo "<br />headers = ". $this->headers;

            
            }
        
        } else {
        
            echo "<ul id='error'>Er is een fout opgetreden bij het verzenden van de mail:";
            
            foreach($this->error as $key => $value){
            
                echo "<li>".$value."</li>";
            
            }
            
            echo "<ul>";
            echo "<br />Klik ".$this->contactlink." om terug te keren naar het contactformulier.";
            

        
        }
    
    }//einde mail versturen




}//einde class

?>

mail.js: de javascriptfunctie waarin het hele gebeuren met httprequests plaatsvindt.

JavaScript:

function sendMail(){

    var XmlHTTP = createREQ();
    var div = document.getElementById('content');
    var poststr = "name=" + document.getElementById('name').value + "&email=" + document.getElementById('email').value + "&subject= " +document.getElementById('subject').value + "&msg= " + document.getElementById('msg').value;
    
    XmlHTTP.onreadystatechange = function(){
        
        if(XmlHTTP.readyState > 4){
            
            div.innerHTML = " <p id='text'>verwerken...</p>";
            
        }  else {
            
            div.innerHTML = XmlHTTP.responseText;
            
        }   
        
        
    }
    
    XmlHTTP.open("POST", "functions/sendmail.php");
    XmlHTTP.setRequestHeader('Content-Type', 'application/x-www-form-urlencoded');
    XmlHTTP.send(poststr);


}

[Verwijderd]

Ik zelf gebruik eigenlijk altijd strlen() om te controleren of een veld een waarde heeft.

[eek]

PHP:

var poststr = "name=" + document.getElementById('name').value + 
              "&email=" + document.getElementById('email').value + 
              "&subject= " +document.getElementById('subject').value + 
              "&msg= " + document.getElementById('msg').value;

Bij zowel subject als msg heb je een extra spatie. Hierdoor werkt je empty() controle niet meer. Want " " is niet empty en "" wel. Kan je trouwens niet $this->error[]= doen in plaats van $this->error[].=?

[Verwijderd]

bedankt, de spaties waren mijn probleem dus. Dom dom dom, maar aan die steen stoot deze ezel zich niet meer.

@Dominique: ik denk niet dat het veel uitmaakt of je empty(), isset() of strlen() gebruikt, tenzij je wilt dat je waarde een bepaald aantal chars heeft, dan is strlen() idd wel het meest aangewezen.

Nogmaals bedankt voor de moeite both

[T-MOB]

Verwijderd schreef op woensdag 04 juli 2007 @ 22:24:
bedankt, de spaties waren mijn probleem dus. Dom dom dom, maar aan die steen stoot deze ezel zich niet meer.

@Dominique: ik denk niet dat het veel uitmaakt of je empty(), isset() of strlen() gebruikt, tenzij je wilt dat je waarde een bepaald aantal chars heeft, dan is strlen() idd wel het meest aangewezen.

Nogmaals bedankt voor de moeite both

Het is totaal offtopic, maar het maakt wel degelijk uit of je empty, isset of strlen gebruikt. Empty geeft ook true als de variabele de string "0" is. Wanneer je empty gebruikt kan iemand dus geen mail versturen met subject "0"...
Strlen kun je verder eigenlijk alleen (netjes) gebruiken in combinatie met isset(). Wanneer de waarde niet verstuurd wordt bestaat $_POST['var'] niet en zal strlen($_POST['var']) je een warning of notice opleveren.

Verder kan het geen kwaad om te checken op arrays. Door "subject[]" naar je script te posten ipv "subject" krijg je een array binnen ipv een string. Je huidige checks zijn daar niet echt tegen opgewassen...

[killercow]

Wat me ook opvalt:

Je concat de error string aan een array,, maar omdat je key meegeeft wordt er voglgens mij altijd een nieuw array element aangemaakt, wat het concatten ongedaan maakt.

$this->error[] .= "U dient een bericht in te vullen.";

Geen idee of het uitmaakt, maar je weet het maar nooit met oog op de toekomst:
$this->error[] = "U dient een bericht in te vullen.";

[Verwijderd]

Met die error array heb ik al aangepast, het werkt op beide manieren zoals het moet, ik heb de punt gewoon weggelaten en het op de manier gedaan waarop je het zegt.

bedankt voor de toch wel relevante info T-MOB

[crisp]

vergeet url-encoding niet: in javascript kan je dat doen mbv de encodeURIComponent() functie