[HTML][PHP]Formularz PHP(form2mail) dodanie treści Opcje

[patys]

Siema,

Na swojej stronie mam formularz php wysyłający emaile(form2mail), dodałem różne zmienne i teraz mam problem, aby dodać te zmienne do wysyłanej wiadomości.

[PHP] pobierz, plaintext 
						<table width="239" cellpadding="0" cellspacing="0"><form action="inc/email/Form2Mail1.php">
 
						<input type=hidden name=Subject value="Testing Form2Mail1.php">
<br>
							<tr>
								<td class="first">
									Data przyjazdu<br />
									<select name="FirstName">
										<option>Styczeń</option>
										<option>Luty</option>
										<option>Marzec</option>
										<option>Kwiecień</option>
										<option>Maj</option>
										<option>Czerwiec</option>
										<option>Lipiec</option>
										<option>Sierpień</option>
										<option>Wrzesień</option>
										<option>Październik</option>
										<option>Listopad</option>
										<option>Grudzień</option>
									</select>
								</td>
								<td class="second"><br />
								 Rok<br>
									<select name="LastName" class="year">
										<option>2011</option>
										<option>2012</option>
										<option>2013</option>
									</select></td>
									<td class="second"><br />
									Data<br />
									<select name="From" class="date">
										<option>1</option>
										<option>2</option>
										<option>3</option>
										<option>4</option>
										<option>5</option>
										<option>6</option>
										<option>7</option>
										<option>8</option>
										<option>9</option>
										<option>10</option>
										<option>11</option>
										<option>12</option>
										<option>13</option>
										<option>14</option>
										<option>15</option>
										<option>16</option>
										<option>17</option>
										<option>18</option>
										<option>19</option>
										<option>20</option>
										<option>21</option>
										<option>22</option>
										<option>23</option>
										<option>24</option>
										<option>25</option>
										<option>26</option>
										<option>27</option>
										<option>28</option>
										<option>29</option>
										<option>30</option>
										<option>31</option>
									</select>
								</td>
							</tr>
							<tr>
								<td class="first">
									Ilość nocy<br />
									<select name="noce">
										<option>1</option>
										<option>2</option>
										<option>3</option>
										<option>4</option>
										<option>5</option>
										<option>6</option>
										<option>7</option>
										<option>8</option>
										<option>9</option>
										<option>10</option>
										<option>11</option>
										<option>12</option>
										<option>13</option>
										<option>14</option>
										<option>15</option>
										<option>16</option>
										<option>17</option>
										<option>18</option>
										<option>19</option>
										<option>20</option>
										<option>21</option>
										<option>22</option>
										<option>23</option>
										<option>24</option>
										<option>25</option>
										<option>26</option>
										<option>27</option>
										<option>28</option>
										<option>29</option>
										<option>30</option>
										<option>31</option>
										<option>Więcej</option>
									</select>
								</td>
								<td class="third">
									Ilość pokoi<br />
									<select name="pokoje">
										<option>1</option>
										<option>2</option>
										<option>3</option>
										<option>4</option>
										<option>Więcej</option>
 
									</select>
								</td>
							</tr>
							<tr>
								<td class="first">
									Ilość dorosłych<br />
									<select name="dorosli">
										<option>1</option>
										<option>2</option>
										<option>3</option>
										<option>4</option>
										<option>5</option>
										<option>Więcej</option>
 
									</select>
								</td>
								<td class="third">
									Ilość dzieci<br />
									<select name="dzieci">
										<option>1</option>
										<option>2</option>
										<option>3</option>
										<option>4</option>
										<option>5</option>
										<option>Więcej</option>
									</select>
								</td>
							</tr>
							<tr>
								<td colspan="2" class="reset"><a href="#">Resetuj</a></td>
							</tr>
						</table>
[PHP] pobierz, plaintext 

form2mail.php

[PHP] pobierz, plaintext 
<?
/*
 
   General Form2Mail script that will send an email using PHP's standard
   "mail()" function. The script will take all the form variables
   and stuff them in an email. You can define the following special
   variables in the form:
 
       From      The emailaddress to send the email from.
       Subject   The subject of the email.
 
   WARNING: do not change this script to accept a "to" address from the
   form. It could then be abused by spammers. Keep the "to" address
   hardcoded in this script, see below.
 
*/
 
 
/* Change these settings. */
$To = '...';          /* To send the email to. */
$SmtpServer = 'smtp.yourdomain.com';     /* The mailserver to use. */
$NextPage = 'success.html';              /* To jump to after sending. */
$tresc = include('tresc.html';
 
 
 
 
 
/* Compose the email. */
if (isset($HTTP_GET_VARS)) {
  while ($a = each($HTTP_GET_VARS)) {
    if ($a[0] == 'From') continue;
    if ($a[0] == 'Subject') continue;
    if ($a[0] == 'Submit') continue;
    $Body .= $a[0] . "\t= " . $a[1] . "\r\n";
    }
  }
if (isset($HTTP_POST_VARS)) {
  while ($a = each($HTTP_POST_VARS)) {
    if ($a[0] == 'From') continue;
    if ($a[0] == 'Subject') continue;
    if ($a[0] == 'Submit') continue;
    $Body .= $a[0] . "\t= " . $a[1] . "\r\n";
    }
  }
 
 
 
/* Remove dangerous characters. */
$Subject = ereg_replace("[^-[:alnum:]_.!@=* ]","",$Subject);
$From = ereg_replace("[^-[:alnum:]_.!@=* ]","",$From);
 
 
 
/* Send the email. */
ini_set("SMTP",$SmtpServer);
ini_set("sendmail_from",$From);
$Status = mail($To,$Subject,$Body+$tresc);
if ($Status != TRUE) {
  echo '<html><body>' . "\n";
  echo '<b>An error has occurred while sending the email.' . "\n";
  echo '</body></html>' . "\n";
  exit;
  }
 
 
 
 
/* Jump to the NextPage. */
header('location: ' . $NextPage);
?>
[PHP] pobierz, plaintext 

Jest zmienna $body, ale nie wiem skąd brane są zmienne /t i inne. Zmienne "firstname" i "lastname" są normalnie pobierane z selectów. Chciałbym aby reszta tez była pobierana.

Co i jak musiałbym zrobić?

Z góry dziękuję i pozdrawiam (IMG:style_emoticons/default/smile.gif)

[morbic]

Ten Form2Mail jest ubogo napisany... Bardzo ubogo...

Generalnie dodane do HTMLa zmienne, jeśli są w obrębie <form></form>, powinny się same dopisać do maila. Ale i tak jest tu masa rzeczy do korekty.

Ok, zacznijmy od HTMLa:

Kod

<select name="From" class="date">

Zmień "From" na obojętnie co, na przykład "Od" - w skrypcie pole "From" jest wywalane z formularza. Ewentualnie możesz w form2mail.php wywalić linijki:

[PHP] pobierz, plaintext 
if ($a[0] == 'From') continue;
[PHP] pobierz, plaintext 

Obydwie.

Sama klasa musi mieć niezłą ilość lat, jest przestarzała. Zamień wszystkie $HTTP_GET_VARS na $_GET , a $HTTP_POST_VARS na $_POST.

Przydałoby się też wywalić ereg_replace i zamienić na:

[PHP] pobierz, plaintext 
$Subject = preg_replace("/[^-a-z0-9_.!@=* ]/i","",$Subject);
$From = preg_replace("/[^-a-z0-9_.!@=* ]/mi","",$From);
[PHP] pobierz, plaintext 

Można by tez rozwinąć skrzydła i posprawdzać, czy dane pola mają odpowiednie wartości. A nawet należałoby.

I inny szczegół:

[PHP] pobierz, plaintext 
$Status = mail($To,$Subject,$Body+$tresc);
[PHP] pobierz, plaintext 

Powinno być:

[PHP] pobierz, plaintext 
$Status = mail($To,$Subject,$Body.$tresc);
[PHP] pobierz, plaintext 

Widzę też literówkę tu:

[PHP] pobierz, plaintext 
$tresc = include('tresc.html';
[PHP] pobierz, plaintext 

Wywal nawias albo domknij go.

Być może przeoczyłem coś jeszcze, ale to takie najbardziej rzucające się w oczy. I tak przepisałbym cały skrypt.

Ten post edytował morbic 2.07.2011, 21:32:29