onSubmit return flase w FF działa w IE niet :/ Opcje

[TomASS]

Kodziki w JS:

Kod

function validateForm(AForm)
{
    if(!(AForm.oskont_imie_nazwisko.value && AForm.oskont_telefon.value && AForm.mz_nazwa.value && AForm.mz_miasto.value && AForm.mz_kod.value && AForm.mz_ulica.value && AForm.normal_od.value && AForm.normal_do.value && AForm.ID_typ.value && AForm.ilosc.value)){

        if (!(AForm.oskont_imie_nazwisko.value)) AForm.oskont_imie_nazwisko.style.border="1px solid #ff9900;";
        else AForm.oskont_imie_nazwisko.style.border="1px solid #7B9EBD;";
        
        if (!(AForm.oskont_telefon.value)) AForm.oskont_telefon.style.border="1px solid #ff9900;";
        else AForm.oskont_telefon.style.border="1px solid #7B9EBD;";
        
        if (!(AForm.mz_nazwa.value)) AForm.mz_nazwa.style.border="1px solid #ff9900;";
        else AForm.mz_nazwa.style.border="1px solid #7B9EBD;";
        
        if (!(AForm.mz_miasto.value)) AForm.mz_miasto.style.border="1px solid #ff9900;";
        else AForm.mz_miasto.style.border="1px solid #7B9EBD;";
        
        if (!(AForm.mz_kod.value)) AForm.mz_kod.style.border="1px solid #ff9900;";
        else AForm.mz_kod.style.border="1px solid #7B9EBD;";
        
        if (!(AForm.mz_ulica.value)) AForm.mz_ulica.style.border="1px solid #ff9900;";
        else AForm.mz_ulica.style.border="1px solid #7B9EBD;";
        
        if (!(AForm.normal_od.value)) AForm.normal_od.style.border="1px solid #ff9900;";
        else AForm.normal_od.style.border="1px solid #7B9EBD;";
        
        if (!(AForm.normal_do.value)) AForm.normal_do.style.border="1px solid #ff9900;";
        else AForm.normal_do.style.border="1px solid #7B9EBD;";
        
        if (!(AForm.ID_typ.value)) AForm.ID_typ.style.border="1px solid #ff9900;";
        else AForm.ID_typ.style.border="1px solid #7B9EBD;";
        
        if (!(AForm.ilosc.value)) AForm.ilosc.style.border="1px solid #ff9900;";
        else AForm.ilosc.style.border="1px solid #7B9EBD;";
        
        alert('Musisz wypełnić wszystkie pola.');
        return false;
    }
    else{
        return true;
    }
}

I dalej HTML:

[HTML] pobierz, plaintext 
<form method="POST" id="formularz" onsubmit="return validateForm(this);">
[HTML] pobierz, plaintext 

W IE wysyła formularz zawsze :/ W FF wysyła jeśli są wypełnione odpowiednie pola (czyli ok). - co jest?

[devnul]

dla IE powinieneś mieć onclick na submicie z tą samą funckją - ot taki kolejny brak

[TomASS]

No niestety :/ zmieniłem troszkę funkcję na:

Kod

function $(id){
    return document.getElementById(id);
}

function validateForm(AForm)
{
    if(!($(AForm).value && $(AForm).oskont_telefon.value && $(AForm).mz_nazwa.value && $(AForm).mz_miasto.value && $(AForm).mz_kod.value && $(AForm).mz_ulica.value && $(AForm).normal_od.value && $(AForm).normal_do.value && $(AForm).ID_typ.value && $(AForm).ilosc.value)){

        if (!($(AForm).oskont_imie_nazwisko.value)) $(AForm).oskont_imie_nazwisko.style.border="1px solid #ff9900;";
        else $(AForm).oskont_imie_nazwisko.style.border="1px solid #7B9EBD;";
        
        if (!($(AForm).oskont_telefon.value)) $(AForm).oskont_telefon.style.border="1px solid #ff9900;";
        else $(AForm).oskont_telefon.style.border="1px solid #7B9EBD;";
        
        if (!($(AForm).mz_nazwa.value)) $(AForm).mz_nazwa.style.border="1px solid #ff9900;";
        else $(AForm).mz_nazwa.style.border="1px solid #7B9EBD;";
        
        if (!($(AForm).mz_miasto.value)) $(AForm).mz_miasto.style.border="1px solid #ff9900;";
        else $(AForm).mz_miasto.style.border="1px solid #7B9EBD;";
        
        if (!($(AForm).mz_kod.value)) $(AForm).mz_kod.style.border="1px solid #ff9900;";
        else $(AForm).mz_kod.style.border="1px solid #7B9EBD;";
        
        if (!($(AForm).mz_ulica.value)) $(AForm).mz_ulica.style.border="1px solid #ff9900;";
        else $(AForm).mz_ulica.style.border="1px solid #7B9EBD;";
        
        if (!($(AForm).normal_od.value)) $(AForm).normal_od.style.border="1px solid #ff9900;";
        else $(AForm).normal_od.style.border="1px solid #7B9EBD;";
        
        if (!($(AForm).normal_do.value)) $(AForm).normal_do.style.border="1px solid #ff9900;";
        else $(AForm).normal_do.style.border="1px solid #7B9EBD;";
        
        if (!($(AForm).ID_typ.value)) $(AForm).ID_typ.style.border="1px solid #ff9900;";
        else $(AForm).ID_typ.style.border="1px solid #7B9EBD;";
        
        if (!($(AForm).ilosc.value)) $(AForm).ilosc.style.border="1px solid #ff9900;";
        else $(AForm).ilosc.style.border="1px solid #7B9EBD;";
        
        alert('Musisz wypełnić wszystkie pola.');
        return false;
    }
    else{
        return true;
    }
}

i mam tak:

[HTML] pobierz, plaintext 
<form method="POST" id="formularz" onsubmit="return validateForm('formularz');">
<input onClick="return validateForm('formularz');" type="submit" name="action" value="Zgłoś">
[HTML] pobierz, plaintext 

dalej nie działa pod IE (pod FF działa)

Ten post edytował TomASS 6.04.2007, 13:23:15

[akubiczek]

Na pewno działa (pod IE6 i IE7). Sprawdź co ci zwraca validateForm('formularz').

Poza tym ma być onclick, a nie onClick

[devnul]

@akubiczek: to czy z duzej czy z małej to akurat pikuś (tak wiem ze powinno być małymi) - przeglądarki to olewają sikiem prostym jak narazie więc to błędu nie czyni.

a ja chyba wiem gdzie błąd leży:
masz tak

Kod

if (!($(AForm).mz_kod.value)) $(AForm).mz_kod.style.border="1px solid #ff9900;";
        else $(AForm).mz_kod.style.border="1px solid #7B9EBD;";

a powinno być tak

Kod

if (!($(AForm).mz_kod.value)) $(AForm).mz_kod.style.border="1px solid #ff9900;"
        else $(AForm).mz_kod.style.border="1px solid #7B9EBD;";

lub tak

Kod

if (!($(AForm).mz_kod.value)) {$(AForm).mz_kod.style.border="1px solid #ff9900;";}
        else {$(AForm).mz_kod.style.border="1px solid #7B9EBD;";}

widzisz tą subtelną różnice?

[TomASS]

@Na pewno działa (pod IE6 i IE7). Sprawdź co ci zwraca validateForm('formularz').

No u mnie jakoś na pewno nie

Poza tym ma być onclick, a nie onClick

A pomyśl czy mi to pomoże i czy ma to tutaj jakieś znaczenie

widzisz tą subtelną różnice?

Różnicę widzę, ale wszystko to co podałeś sprowadza się do jednego, bez różnicy na nawiasy i średnik na końcu linii. Tak czy siak nie widzę najsubtelniejszej różnicy - nie działa nadal - tak, siak czy owak :/

Dzięki

Ten post edytował TomASS 6.04.2007, 18:48:14

[JaRoPHP]

Kiedyś miałem podobny problem i rozwiązałem to w sposób:

Zamiast

[HTML] pobierz, plaintext 
if (!(AForm.oskont_imie_nazwisko.value))
[HTML] pobierz, plaintext 

dałem

[HTML] pobierz, plaintext 
if ((AForm.oskont_imie_nazwisko.value == ""))
[HTML] pobierz, plaintext 

[Zajec]

Może testujesz na IE7? Przeczytaj tekst porneL'a o onclick i pochodnych: http://pornel.net/onclick

Jest tam wzmianka o bugu zaktualizowanego IE7.

[Norbas]

Nie piszesz zbyt optymalnego kodu JS i HTML Moja propozycja poniżej.

HTML:

Kod

<form method="post" action="" onsubmit="return validateForm(this)">
<!-- wstawić pola formularza -->
<input type="submit" value="Wyślij" />
</form>

JS:

Kod

function validateForm(f){
var i,e=f.elements,puste=false
var pola=['oskont_telefon','mz_nazwa','mz_miasto','mz_kod','mz_ulica','normal_od','normal_do','ID_typ','ilosc']
for(i=pola.length-1; i>=0; --i)
    if(e[pola[i]].value!='') e[pola[i]].style.border='1px solid #7B9EBD'
    else{
        puste=true
        e[pola[i]].style.border='1px solid #ff9900'
    }
if(puste) alert('Musisz wypełnić wszystkie pola')
return !puste}

Do funkcji jako parametr przekazywany jest obiekt, a nie nazwa formularza

[TomASS]

Może testujesz na IE7? Przeczytaj tekst porneL'a o onclick i pochodnych: http://pornel.net/onclick

Testowałem na IE6 A tekst już dawno mam w ulubionych

Nie piszesz zbyt optymalnego kodu JS i HTML

Z JS się zgodzę, że nie jest optymalny HTML jest

Dzięki - Twój kod działa pod IE i FF
Thx

[akubiczek]

No u mnie jakoś na pewno nie

Chodziło mi o samego HTMLa, bo to jego zmieniałeś w kolejnym poście.

A pomyśl czy mi to pomoże i czy ma to tutaj jakieś znaczenie

Pomoże ci się stać lepszym w tym co robisz. Ale nie chcesz, nie słuchaj

[Maksymus007]

[HTML] pobierz, plaintext 
onClick="return event.returnValue=costam()"
[HTML] pobierz, plaintext 

to u mnie dziala, na wszystkich przegladarkach, wlacznie z IE7 i 6

Ten post edytował Maksymus007 9.07.2007, 19:30:53