Forum PHP.pl > [PHP]Formularz nie działa po przejsciu z 5.2 na 5.3

Pomoc - Szukaj - Użytkownicy - Kalendarz

Pełna wersja: [PHP]Formularz nie działa po przejsciu z 5.2 na 5.3

struna12

16.09.2013, 18:14:12

Witam
Jestem kompletnie zielony w temacie PHP. Mam strone internetową na której jest formularz kontaktowy. Po przełączeniu na serwerze wersji php z 5,2 na 5.3 maile przychodzą bez treści. (nie wyświetla sie tekst który wpisuje osoba wysyłająca wiadomość)

Proszę powiedzcie czy można coś pozmieniać w pliku php zeby zaczeło działać? jakaś normalizacja po przejściu na php5.3? czy może któreś wpisy się zdeprecjionowały?

POMOCY!:)

fragment kodu formularza:

CODE

<form action="email.php" method="post" onsubmit="return check_form(this)">
Imię <span style="color:red;font-size:11px;">*</span><br><input type="text" name="imie" class="form"><br>
Numer Telefonu<br><input type="text" name="numer" class="form"><br>
Twój e-mail <span style="color:red;font-size:11px;">*</span><br><input type="text" name="email" class="form"><br>
Treść wiadomości <span style="color:red;font-size:11px;">*</span><br><textarea name="wiadomosc" class="text"></textarea><br>
<input type="hidden" name="wyslij" value="1">
<input type="submit" value="" class="button">
<br><br><span style="color:red;font-size:11px;">* Pole wymagane</span>
</form><br><br>

A tu zawartość pliku email.php

CODE

<?php
//sprawdzenie czy zał±cznik pochodzi z formularza
if ($_POST["wyslij"]==1)
{
// dane o odbiorcy, nadawcy
$odbiorca="sempretaniec@sempretaniec.pl";
$tytul="Wiadomosc ze strony SEMPRE";
$nadawca = "admin";
$nadawca_mail="user";

// tre¶ć listu
$tresclistu = "<table border=\"0\" width=\"100%\" border=\"0\" cellpadding=\"1\" cellspacing=\"2\" align=center style=\"font-family: Verdana; font-size: 12px;\">
<tr>
<td>Imię i Nazwisko:</td>
<td><b>".$imie."</b></td>
</tr>
<tr>
<td>Numer telefonu:</td>
<td><b>".$numer."</b></td>
</tr>
<tr>
<td>Adres Email do korespondencji:</td>
<td><b>".$email."</b></td>
</tr>
<tr>
<td>Mieszkam w:</td>
<td><b>".$miejscowosc."</b></td>
</tr>
<tr>
<td>Tre¶ć wiadomo¶ci:</td>
<td><b>".$wiadomosc."</b></td>
<tr>
<td colspan=\"2\"><br>Proszę nie odpowiadać na tę wiadomo¶ć jest wygenerowana automatycznie<br></td>
</tr>
</table>";

// definicja nagłówków
$naglowki = "From: $nadawca \n";
$naglowki .= "MIME-Version: 1.0\n";
$naglowki .= "Content-Type: multipart/mixed;\n";
$naglowki .= "\tboundary=\"___$znacznik==\"";

// nagłówki listu
$tresc="--___$znacznik==\n";
$tresc .="Content-Type: text/html; charset=\"iso-8859-2\"\n";
$tresc .="Content-Transfer-Encoding: 8bit\n";
$tresc .="\n$tresclistu\n";

// wysłanie listu
if (mail($odbiorca,$tytul,$tresc,$naglowki))
{
print "Wiadomo¶ć wysłana dziękujemy";
}
else
{
print "wiadomosc nie wyslana, wystapil blad";
}
}
?>

Turson

16.09.2013, 18:32:50

Zobacz, masz taki kod

[PHP] pobierz, plaintext 
<td>Imię i Nazwisko:</td>
<td><b>".$imie."</b></td> 
[PHP] pobierz, plaintext

Skąd skrypt ma wiedzieć co oznacza $imie?
Zdefiniuj wcześniej wysłane zmienne

[PHP] pobierz, plaintext 
$imie=$_POST["imie"];
$numer=$_POST["numer"];
// i tak dalej...
[PHP] pobierz, plaintext

I to nie ma związku z przejściem na nową wersję PHP, bo jeśli wcześniej był ten sam kod to również nie miał prawa działać.

A sam skrypt jest kiepski. Nie sprawdzasz nawet czy wszystkie pola zostały wypełnione, czy e-mail jest poprawnego formatu.

struna12

16.09.2013, 18:43:56

Dziękuję za szybką odpowiedź.

Niestety jak już wcześniej pisałem nie mam pojęcia o php i nie mam pojęcia gdzie co mam wpisac.

Dziwne jest to, że do tej pory działało wszystko poprawnie.
jesli chodzi o sprawdzanie poprawnosci wpisu to mam jeszcze taki plik js

CODE

Nie nie zmieniałem w tych plikach a od dwóch dni przestały działać. Czy jeśli to nie wina skryptu to moze to być wina serwera?.
Instalowałem joomle na tym serwrze i przy instalacji oprocz wymogu php5.3 było jeszcze coś o "Wyłączone Magic Quotes GPC"
może to to?

timon27

16.09.2013, 19:06:57

Cytat(Turson @ 16.09.2013, 19:32:50 )

...to również nie miał prawa działać.

Miał prawo działać jeśli php został odpowiednio skompilowany (np. u mnie działa).
Najwyraźniej administrator przy nowej wersji zrobił to inaczej.
Tak czy siak faktycznie takie pisanie kodu nie jest dobrym pomysłem.

Turson

16.09.2013, 19:15:24

Wystarczy, że ktoś wyłączy js w przeglądarce i formularz przejdzie bez wpisywania czegokolwiek

Spróbuj tego

[PHP] pobierz, plaintext 
<?php
//sprawdzenie czy załącznik pochodzi z formularza
if ($_POST["wyslij"]==1)
{
// dane o odbiorcy, nadawcy
$odbiorca="sempretaniec@sempretaniec.pl";
$tytul="Wiadomosc ze strony SEMPRE";
$nadawca = "admin";
$nadawca_mail="user";
 
// treść listu
$tresclistu = "<table border=\"0\" width=\"100%\" border=\"0\" cellpadding=\"1\" cellspacing=\"2\" align=center style=\"font-family: Verdana; font-size: 12px;\">
<tr>
<td>Imię i Nazwisko:</td>
<td><b>".$_POST['imie']."</b></td>
</tr>
<tr>
<td>Numer telefonu:</td>
<td><b>".$_POST['numer']."</b></td>
</tr>
<tr>
<td>Adres Email do korespondencji:</td>
<td><b>".$_POST['email']."</b></td>
</tr>
<tr>
<td>Mieszkam w:</td>
<td><b>".$_POST['miejscowosc']."</b></td>
</tr>
<tr>
<td>Treść wiadomości:</td>
<td><b>".$_POST['wiadomosc']."</b></td>
<tr>
<td colspan=\"2\"><br>Proszę nie odpowiadać na tę wiadomość jest wygenerowana automatycznie<br></td>
</tr>
</table>";
 
// definicja nagłówków
$naglowki = "From: $nadawca \n";
$naglowki .= "MIME-Version: 1.0\n";
$naglowki .= "Content-Type: multipart/mixed;\n";
$naglowki .= "\tboundary=\"___$znacznik==\"";
 
// nagłówki listu
$tresc="--___$znacznik==\n";
$tresc .="Content-Type: text/html; charset=\"iso-8859-2\"\n";
$tresc .="Content-Transfer-Encoding: 8bit\n";
$tresc .="\n$tresclistu\n";
 
// wysłanie listu
if (mail($odbiorca,$tytul,$tresc,$naglowki))
{
print "Wiadomość wysłana dziękujemy";
}
else
{
print "wiadomosc nie wyslana, wystapil blad";
}
}
?>
[PHP] pobierz, plaintext

struna12

16.09.2013, 19:31:41

Działa!!!
Dzięki Turson.
Nie mam pojecia jak mogło działać wcześniej.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.