PHPMailer - błąd przy ustaleniu nadawcy Opcje

[Mateushh]

Witam,

wysyłam maila PHPmailerem na gmail. Wszystko pięknie, mail dochodzi, z tym że nie tak jak powinien.
(IMG:http://iv.pl/images/22498508093725775385.png)
Zamiast maila nadawcy, wyświetla się adres moj, czyli odbiorcy.

[PHP] pobierz, plaintext 
$name = strip_tags(trim($_POST["contact_id"]));
        $name = str_replace(array("\r","\n"),array(" "," "),$name);
        $emailsender = filter_var(trim($_POST["contact_email"]), FILTER_SANITIZE_EMAIL);
        $message = trim($_POST["contact_content"]);
 
        $mail = new PHPMailer();
        $mail->CharSet = "UTF-8";
 
        $mail->IsSMTP();
        $mail->Host = "smtp.gmail.com";
        $mail->Port = 465;
        $mail->SMTPAuth = true;
        $mail->Username = "mojadres@gmail.com";
        $mail->Password = "mojehaslo";
        $mail->SMTPSecure = "ssl";
 
        $mail->From = $emailsender; 
        $mail->FromName = $name; 
        $mail->Sender = $emailsender;
        $mail->AddReplyTo($emailsender, $name);
        $mail->AddAddress('mojadres@gmail.com', 'mati'); 
 
        $mail->IsHTML(true); 
        $mail->Subject = 'Nowa wiadomość od: '.$name;
 
        $gridBody = "<h3>Witaj</h3></br>";
        $gridBody .= "<p class='sekcja-wiadomosci'>Otrzymałeś nową wiadomość od: ".$name.".</p>";
        $gridBody .= "Jak najszybciej skontaktuj się z nadawcą!<br/><br/><br/>";
        $gridBody .= "<p class='sekcja-wiadomosci'>".$message."</p>";
 
        $mail->Body = nl2br($gridBody);
        $mail->AltBody = 'Wiadomość została niepoprawnie wysłana. Wystąpił błąd podczas jej wysyłania.';
 
            $mail->Send();
            echo 'Dziękujemy! Twoja wiadomość została wysłana.';
 
            $mail->ClearAddresses();
	    $mail->ClearAttachments();
[PHP] pobierz, plaintext 

Jak temu zaradzić?

[casperii]

[PHP] pobierz, plaintext 
$mail->SetFrom(''.$emailSender.'', ''.$name.'');
$mail->AddReplyTo(''.$emailSender.'', ''.$name.'');
$mail->Sender = $emailSender;
[PHP] pobierz, plaintext 

[nospor]

casperii wyjasnij mi prosze, bo bardzo mnie to ciekawi, co robią '' w tym kodzie:
''.$emailSender.''

[casperii]

casperii wyjasnij mi prosze, bo bardzo mnie to ciekawi, co robią '' w tym kodzie:
''.$emailSender.''

@nospor
trzecia linijka tego kodu:

[PHP] pobierz, plaintext 
$emailsender = filter_var(trim($_POST["contact_email"]), FILTER_SANITIZE_EMAIL);
[PHP] pobierz, plaintext 

A to, że napisałem:
$emailSender zamiast $emailsender to nie duży błąd, zwłaszcza o godz. 3:28.

[nospor]

Teraz jest 11 w dzien i nadal masz problemy z czytaniem. Zadalem zupelnie inne pytanie

[casperii]

$emailsender jest nazwą zmiennej , która otrzymuję wartość $_POST["contact_email"] po wysłaniu forma metodą post.

[nospor]

Nadal nie o to pytam. Wyraźnie napisalem:

co robią '' w tym kodzie

Naprawde nie widzisz ze tam są dwa apostrofy w pytaniu? Dokladnie takie same jakie ty dajesz w swoim kodzie. Pytam się: po co?

[casperii]

@nospor. Nie wiem co to za głupie pytania i nie potrzebna dyskusja nie związana z tematem.

Tak samo dlaczego echo ''.$emailsender.''; a nie echo $emailsender;

Mam po prostu takie przyzwyczajenie i już.

Skoro jesteś taki omnibus to odpowiedz w czym to przeszkadza czy napiszę sobie:

echo ''.$emailsender.''; a nie echo $emailsender;

albo:

$mail->SetFrom('$emailSender', '$name');

czy też , chociaż nie wiem czy parsera nie wywali:

$mail->SetFrom($emailSender, $name);

[nospor]

To nie są głupie pytania. Piszesz źle to sie pytam czemu i tyle.

Nie:
$mail->SetFrom(''.$emailSender.'', ''.$name.'');

a:
$mail->SetFrom($emailSender, $name);

To ze sam masz złe przyzwyczajenia nie oznacza, ze masz nimi zarazac innych

albo:

$mail->SetFrom('$emailSender', '$name');

A to to juz w ogole tragedia... apostrofy nie parsują zmiennych...

[casperii]

Zadałem pytanie:
w czym to przeszkadza czy napiszę sobie:
echo ''.$emailsender.''; a nie echo $emailsender;

[nospor]

A ja ci odpowiedzialem. Ty naprawde masz dzis problemy z czytaniem... dobrze, zacytuje ci jeszcze raz:

Piszesz źle to sie pytam czemu i tyle.
....
To ze sam masz złe przyzwyczajenia nie oznacza, ze masz nimi zarazac innych

[casperii]

Gdyby było źle to by wyświetliło błąd. A zarówno jak i jedna tak i druga forma jest poprawnie wyświetlona.
Dobrze zapytam się w inny sposób:

Podaj różnicę zapisu i dlaczego uważasz, że jest ona zła:

[PHP] pobierz, plaintext 
echo ''.$emailsender.''; a nie echo $emailsender;
[PHP] pobierz, plaintext 

Nie pytam tutaj o to , że druga forma jest krótsza, czy że nie ma ' ' :-)
Tylko dlaczego powinno się pisać echo $cos; a nie echo ''$cos.'';

[nospor]

Z Krakowa do Warszawy mozesz jechac przez Berlin. Nikt ci nie broni... ale nie pisz na forum, ze to poprawna droga bo nie jest.
Identycznie z kodem ktory wstawiasz... To ze php nie pluje bledem nie oznacza, ze jest to dobry kod. Masz jakies niedobre przyzwyczajenia wynikajacy z braku podstaw i propagujesz je na forum i nie daj boze ktos pomysli ze tak sie wlasnie pisze.... Nie, tak sie nie pisze. Tak piszą osoby, ktore nie ogarniają podstaw. Zwracam ci wiec na to uwage, moze sam sie czegos nauczysz.

[casperii]

@nospor nie twierdzę, że jestem prymusem PHP. Czy też hiper , super mega programistą. "Jakąś" wiedzę posiadam, którą stopniowo poszerzam, lecz daleko mi to perfekcyjności i jestem tego świadom.
Pisząc coś i natrafiając na błąd jak zauważyłeś nie szukam gotowego rozwiązania tylko naprowadzenia. Za to ty jesteś jak już Ci kiedyś pisałem specyficzną osobowością, która w jakiś sposób nie pała chęcią niesienia pomocy, tylko myślenia ironicznego :-) (u know what i mean). Co w moim przypadku jest więcej nerwów niż logicznego myślenia.

Nadal nie otrzymałem jasnych argumentów tylko dziwne porównanie jazdy przez Berlin.
Wydaje mi się, że to ma taki sam sens jak pisząc echo a print. Jedni wolą brunetki inni rude (IMG:style_emoticons/default/smile.gif)

[nospor]

O to:
echo ''.$emailsender.'';
nie ma zadnego sensu. Tu nie ma co wyjasniac.

Co do pomagania to pomagam caly czas. Nie moja wina ze nie dociera do ciebie co się Tobie pisze....

[vokiel]

Co do tego

[PHP] pobierz, plaintext 
echo ''.$emailsender.''; // == pusty ciąg + wartość w $emailsender + pusty ciąg
echo $emailsender; // == wartość w $emailsender
[PHP] pobierz, plaintext 

To może wygląda na mikro-optymalizację, ale łączenie stringów z dwoma pustymi ciągami nic nie daje - wynik jest ten sam (o ile wartość w $emailsender jest typu string), powoduje tylko/aż tyle, że PHP musi wykonać dodatkowo dwie niepotrzebne operacje. To nie to samo co różnica między echo a print (chociaż one też mają kilka różnic).

W tym przypadku nie ma różnicy, bo SetFrom przyjmuje jako parametry tylko ciągi znaków. Ale wyobraź sobie, że ta funkcja mogłaby mieć inne działanie w zależności od typu zmiennej - jeśli to tablica to pobiera pierwszy element jako email a drugi jako nazwę, jeśli string to pierwszy parametr jako email, drugi parametr jako nazwę. Dodając cudzysłów wymuszasz niejawnie typ, a w przypadku łączenia ciągów z tablicą będziesz miał błąd:

Kod

PHP Notice:  Array to string conversion in ...

Wracając do tematu.
Nie musisz więc nawet ręcznie ustawiać pola Sender

[PHP] pobierz, plaintext 
$mail->SetFrom($emailSender, $name);
$mail->AddReplyTo($emailSender.'', $name);
// Sender jest ustawiany automatycznie
$mail->Sender = $emailSender;
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
/**
 * Set the From and FromName properties
 * @param string $address
 * @param string $name
 * @param boolean $auto Whether to also set the Sender address, defaults to true
 * @throws phpmailerException
 * @return boolean
 */
public function SetFrom($address, $name = '', $auto = true) {
[PHP] pobierz, plaintext 

[Pyton_000]

@casperii jest nie poprawnie bo:

1. Jak @vokiel powiedział, jest to łączenie stringów (w tym przypadku min. 2) ze sobą.
2. Całość jest rzutowana do string (co nie zawsze może być dobre:
-

[PHP] pobierz, plaintext 
$aa = 11;
 
var_dump(''.$aa.'');
// string(2) "11"
 
var_dump($aa);
// int(11)
 
$bb = array(1,2);
 
var_dump(''.$bb.'');
// PHP Notice:  Array to string conversion 
 
var_dump($bb);
/*array(2) {
  [0] =>
  int(1)
  [1] =>
  int(2)
}
*/
[PHP] pobierz, plaintext 

3. Zbędne znaki, zbędne stracone miejsce, zbędnie klepanie
4. mikro-wydajność - Może i mikro, ale od najprostszych rzeczy trzeba zaczynać. Bo potem będziesz pisał w stylu:

[PHP] pobierz, plaintext 
for($i=0; $i < count($atr); $i++)
[PHP] pobierz, plaintext 

5. Dobre nawyki programistyczne (to samo będzie z formatowaniem)

Wiele wiele więcej można wymieniać. Ale po co...

[Mateushh]

Co do tego

[PHP] pobierz, plaintext 
echo ''.$emailsender.''; // == pusty ciąg + wartość w $emailsender + pusty ciąg
echo $emailsender; // == wartość w $emailsender
[PHP] pobierz, plaintext 

To może wygląda na mikro-optymalizację, ale łączenie stringów z dwoma pustymi ciągami nic nie daje - wynik jest ten sam (o ile wartość w $emailsender jest typu string), powoduje tylko/aż tyle, że PHP musi wykonać dodatkowo dwie niepotrzebne operacje. To nie to samo co różnica między echo a print (chociaż one też mają kilka różnic).

W tym przypadku nie ma różnicy, bo SetFrom przyjmuje jako parametry tylko ciągi znaków. Ale wyobraź sobie, że ta funkcja mogłaby mieć inne działanie w zależności od typu zmiennej - jeśli to tablica to pobiera pierwszy element jako email a drugi jako nazwę, jeśli string to pierwszy parametr jako email, drugi parametr jako nazwę. Dodając cudzysłów wymuszasz niejawnie typ, a w przypadku łączenia ciągów z tablicą będziesz miał błąd:

Kod

PHP Notice:  Array to string conversion in ...

Wracając do tematu.
Nie musisz więc nawet ręcznie ustawiać pola Sender

[PHP] pobierz, plaintext 
$mail->SetFrom($emailSender, $name);
$mail->AddReplyTo($emailSender.'', $name);
// Sender jest ustawiany automatycznie
$mail->Sender = $emailSender;
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
/**
 * Set the From and FromName properties
 * @param string $address
 * @param string $name
 * @param boolean $auto Whether to also set the Sender address, defaults to true
 * @throws phpmailerException
 * @return boolean
 */
public function SetFrom($address, $name = '', $auto = true) {
[PHP] pobierz, plaintext 

niestety to nic nie dało :<