[javascript] Sprawdzanie formularza Opcje

[ddzkm]

Witam!

Mam formularz i skrypt JS, który sprawdza, czy wymagane pola są wypełnione czy nie (w tym drugim przypadku blokuje wysyłanie formularza do serwera). Niestety zupełnie nie wiedzieć dlaczego po wpisaniu dowolnej wartości w polu "imię" skrypt kończy pracę i wysyła formularz, nawet w przypadku gdy pozostałe pola nie są wypełnione, w innym przypadku zaś o dziwo działa poprawnie. Co tu może być nie tak? Dodam, ze próbowałem kombinować z jednym zbiorowym warunkiem ("||") ale efekt też był ten sam...

Skrypt:

[HTML] pobierz, plaintext 
<script TYPE="text/javascript">
<!--  
function empty_check()
    {
    var form_data = document.forms["contact"];
 
    if (form_data.username.value == "")
        {
        alert("Aby wysłać wiadomość musisz wypełnić wszystkie pola formularza!");
        return false;
        }
 
    if (form_data.e-mail.value == "")
        {
        alert("Aby wysłać wiadomość musisz wypełnić wszystkie pola formularza!");
        return false;
        }
 
    if (form_data.subject.value == "")
        {
        alert("Aby wysłać wiadomość musisz wypełnić wszystkie pola formularza!");
        return false;
        }
 
    if (form_data.text.value == "")
        {
        alert("Aby wysłać wiadomość musisz wypełnić wszystkie pola formularza!");
        return false;
        }
 
 
    alert("Aby wysłać wiadomość musisz wypełnić wszystkie pola formularza!");
    return false;
 
    return true;
    }
//-->
</SCRIPT>
[HTML] pobierz, plaintext 

Formularz:

[HTML] pobierz, plaintext 
<FORM NAME="contact" ACTION="send.php" METHOD="post">
 
<TABLE CELLSPACING="5" CELLPADDING="0" STYLE="border: 1px solid #AEAEAE;">
<TR>
<TD ALIGN="left">
Twoje imię, nazwisko lub pseudomin:<FONT COLOR="#FF0000">*</FONT>
</TD>
<TD ALIGN="left">
 
<INPUT TYPE="text" NAME="username" VALUE="">
 
</TD>
</TR>
<TR>
<TD ALIGN="left">
Twój adres e-mail:<FONT COLOR="#FF0000">*</FONT>
</TD>
<TD ALIGN="left">
 
<INPUT TYPE="text" NAME="e-mail" VALUE="">
 
</TD>
</TR>
<TR>
<TD ALIGN="left">
Temat wiadomości:<FONT COLOR="#FF0000">*</FONT>
</TD>
<TD ALIGN="left">
 
<SELECT NAME="subject"><OPTION VALUE="" SELECTED>[ wybierz temat ]</OPTION>
<OPTION VALUE="1">Temat 1</OPTION>
<OPTION VALUE="2">Temat 2</OPTION>
</SELECT>
 
</TD>
</TR>
<TR>
<TD ALIGN="left">
Treść wiadomości:<FONT COLOR="#FF0000">*</FONT>
</TD>
<TD ALIGN="left">
<TEXTAREA NAME="text" COLS="39" ROWS="10"></TEXTAREA>
</TD>
</TR>
<TR>
<TD COLSPAN="2"><HR>
</TD>
</TR>
<TR>
<TD COLSPAN="2" ALIGN="center">
 
<INPUT TYPE="submit" NAME="button" VALUE="Wyślij wiadomość" onclick="return empty_check()">
 
</TD>
</TR>
</TABLE>
</FORM> 
[HTML] pobierz, plaintext 

[ddzkm]

Po małej modyfikacji działa (nie mogłem niestety zmienić przycisku na "button", bo przestawała działać walidacja w PHP (IMG:style_emoticons/default/biggrin.gif) ). A oto wersja działająca:

[HTML] pobierz, plaintext 
function empty_check()
    {
    pustych=0;
 
    fd=document.forms["contact"];
 
    if (fd.username.value=="") pustych++;
 
    else if (fd.email.value=="") pustych++;
 
    else if (fd.subject.value =="") pustych++;
 
    else if (fd.text.value=="") pustych++;
 
    if (pustych==0) return true;
 
    else
        {
        alert("Aby wysłać wiadomość musisz wypełnić wszystkie pola formularza!");
        return false;
        }
    }    
[HTML] pobierz, plaintext 

Zaś w formularzu zgodnie z podpowiedzią zmieniłem jedynie "e-mail" na "email".

Wielkie dzięki za pomoc (IMG:style_emoticons/default/tongue.gif)