Discussion :

[cUs3Mp]

Bonjour

je parcourais les sources de mon site cherchant a comprendre pourquoi les mails ne partaient pas et en trouvant cette fonction dans la class j'ai vu un truc choquant mais je ne suis pas sur, si vous aussi le return true vous perturbe donnez votre avis.

je crois que méme si la fonction s est mal déroulée elle retournera true toujours !

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
    function Send() {
        $header = "";
        $body = "";
 
        if((count($this->to) + count($this->cc) + count($this->bcc)) < 1)
        {
            $this->SetError($this->Lang("provide_address"));
            return false;
        }
 
        // Set whether the message is multipart/alternative
        if(!empty($this->AltBody))
            $this->ContentType = "multipart/alternative";
 
        $this->SetMessageType();
        $header .= $this->CreateHeader();
        $body = $this->CreateBody();
 
        if($body == "") { return false; }
 
        // Choose the mailer
        if($this->Mailer == "sendmail")
        {
          if(!$this->SendmailSend($header, $body))
              return false;
        }
        elseif($this->Mailer == "mail")
        {
          if(!$this->MailSend($header, $body))
              return false;
        }
        elseif($this->Mailer == "smtp")
        {
          if(!$this->SmtpSend($header, $body))
              return false;
        }
        else
        {
            $this->SetError($this->Mailer . $this->Lang("mailer_not_supported"));
            return false;
        }
 
        return true;
    }

[darkstar123456]

c'est juste une fonction hyper degeu mais non, le return true me choque pas.


PS : si je dis qu'elle est degeu c'est que je me souviens mon premier cours de C++ :

Ce n'est pas toujours possible mais dans une fonction, on essaye de ne mettre qu'un seul return

Et ici, y'a vachement moyen ! Les valeurs se perdent...

[cUs3Mp]

je déclar sent_mail a true en debut de ma classe

la classe c est class.phpmailer.php

ca me semble mieux comme ca il retourne false si erreur sinon true

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
    function Send() {
        $header = "";
        $body = "";
 
        if((count($this->to) + count($this->cc) + count($this->bcc)) < 1)
        {
            $this->SetError($this->Lang("provide_address"));
            sent_mail = false;
        }
 
        // Set whether the message is multipart/alternative
        if(!empty($this->AltBody))
            $this->ContentType = "multipart/alternative";
 
        $this->SetMessageType();
        $header .= $this->CreateHeader();
        $body = $this->CreateBody();
 
        if($body == "") { sent_mail = false; }
 
        // Choose the mailer
        if($this->Mailer == "sendmail")
        {
          if(!$this->SendmailSend($header, $body))
              sent_mail = false;
        }
        elseif($this->Mailer == "mail")
        {
          if(!$this->MailSend($header, $body))
              sent_mail =  false;
        }
        elseif($this->Mailer == "smtp")
        {
          if(!$this->SmtpSend($header, $body))
              sent_mail =  false;
        }
        else
        {
            $this->SetError($this->Mailer . $this->Lang("mailer_not_supported"));
            sent_mail =  false;
        }
 
        return sent_mail ;
 
 }

[cUs3Mp]

j'ai oublié les $ devant tout les variables sent_mail

m'engueulez pas svp j'ai la téte en l air today
:-p