Po wyslaniu maila z formularza ucina jego znaki

pytanie zadane 23 lipca 2017 w PHP przez Szczur Nowicjusz (190 p.)

Witajcie, mam problem z wysyłaniem emaila, który zawiera imie i nazwisko, date, email lub telefon. Wszystko działa jak należy poza jednym - gdy skrypt PHP wyśle już maila ucina mi po prostu znaki wpisane w inputy. Wygląda to mniej-więcej tak

wizyta@**********.pl
	
18:36 (9 minut temu)
	
do mnie


Osoba o imieniu i nazwisku: K
Kontaktujaca sie pod numerem lub mailem: w
Chcialaby zarezerwowac wizyte dnia:2

--
Wyslano z formularza kontaktowego.

a powinno być wpisane np. Kacper i nazwisko. Próbowałem przekleić kod do index.php lecz jest tak jak było. Kod skryptu jest z jakiegoś forum php a stylizacja formularza od Pana Mirosława.

index.php - https://pastebin.com/fZUfrRaT

skrypt.php - https://pastebin.com/Zv2zfgGJ

1 odpowiedź

odpowiedź 23 lipca 2017 przez Arkadiusz Waluk Ekspert (287,950 p.)
wybrane 23 lipca 2017 przez Szczur

Najlepsza

@$name = $_POST['your-name'];

Tutaj odbierasz wartość z tego pola i zapisujesz ją do zmiennej $name.

$content = 'Osoba o imieniu i nazwisku: ' .$name['your-name']

Po co więc za moment, przy dodaniu do treści wiadomości odwołujesz się znów do indeksu "your-name"? $name to nie tablica a zwykły string, wystarczy że wyświetlisz samo $name.

PS mógłbyś wywalić @ i zrobić to tak, jak być powinno.

komentarz 23 lipca 2017 przez Szczur Nowicjusz (190 p.)

Znak @ wywaliłem, fakt. Działa, wielkie dzięki. Zupełnie o tym zapomniałem.

komentarz 23 lipca 2017 przez Arkadiusz Waluk Ekspert (287,950 p.)

Samo wywalenie @ to mało. Wypadałoby dla porządku przed odczytaniem z $_POST sprawdzić czy taki element istnieje, bo gdy ktoś wywoła skrypt bez podania tych danych to rzuci ostrzeżenia (co prawda na produkcji i tak nie powinno to być widoczne). Może sprawdzić chociażby przy użyciu isset / ??. Albo jeszcze lepiej użyć filter_input.