[RISOLTO][JQuery] Validazione form

Xeyos · Messaggio da **Xeyos** » mercoledì 25 giugno 2014, 12:02

Ciao a tutti,
probabilmente sto facendo un errore grossolano che non trovo nella validazione di un form, perchè dovrebbe funzionare.

Codice: Seleziona tutto

<script>
 function checkForm() {
     var xml1 = document.getElementsByName('xml1').val();
     var xml2 = document.getElementsByName('xml2').val();
     var xml3 = document.getElementsByName('xml3').val();
     if(xml1 == '' || xml2 == ''  || xml3 == ''){
         alert("The inputs can not be empty ");
         return false;
     }
     else return false;
 };  

</script>
<form id="formxml" method="post" action="<?php echo URL;?>controller/testXML" enctype="multipart/form-data">
    <label for="xml1">XML1:</label>
    <input type="file" name="xml1" id="xml1"><br>
    <label for="xml2">XML2:</label>
    <input type="file" name="xml2" id="xml2"><br>
    <label for="xml3">XML3:</label>
    <input type="file" name="xml3" id="xml3"><br>
    <input type="submit" name="submit" value="submit" onclick="return checkForm()">
</form>

Questo non sembra funzionare, il form viene comunque spedito.
Io devo controllare che siano stati inseriti 3 file nei 3 input e poi inviati.

ienaplinsky

Io utilizzo l'attributo onsubit della form, non l'attributo onclick del button submit

Messaggio da **Zoff** » mercoledì 25 giugno 2014, 21:50

Il problema di fondo è che c'è un errore nel codice javascript, usi il metodo val() di jquery sui normali oggetti del DOM di javascript, per cui viene generatore un errore che impedisce alla funzione di restituire false, quindi il submit procede.
Tra l'altro stai anche usando il metodo sbagliato, dovresti usare getElementById, non getElementsByName.

Così funziona:

Codice: Seleziona tutto

<!DOCTYPE html>
<html>
    <head>
        <meta charset="utf-8">
        <title>OK</title>
    </head>
    <body>
        <script>
            function checkForm() {
                var xml1 = document.getElementById('xml1').value;
                var xml2 = document.getElementById('xml2').value;
                var xml3 = document.getElementById('xml3').value;
                if (xml1 == '' || xml2 == '' || xml3 == '') {
                    alert("The inputs can not be empty ");
                    return false;
                }
                else
                    return false;
            }
            ;

        </script>
        <form id="formxml" method="post" action="<?php echo $_SERVER['PHP_SELF'];?>/controller/testXML" enctype="multipart/form-data">
            <label for="xml1">XML1:</label>
            <input type="file" name="xml1" id="xml1"><br>
            <label for="xml2">XML2:</label>
            <input type="file" name="xml2" id="xml2"><br>
            <label for="xml3">XML3:</label>
            <input type="file" name="xml3" id="xml3"><br>
            <input type="submit" name="submit" value="submit" onclick="return checkForm();">
        </form>
    </body>
</html>

Xeyos · Messaggio da **Xeyos** » mercoledì 25 giugno 2014, 23:02

Beh direi che troppe ore di programmazione continuata fanno male, vado comunque a nascondermi per l'errore stupido...
Grazie mille Zoff.

Forum Ubuntu-it