Problema con validazione form

[fermat]

buondì.
ho un problema con un form e la validazione in js.
mi pare di aver fatto tutto corretto, però al submit del form nn viene validato nulla:

        <script type="text/javascript">
            function valida() {
                var titolo = document.articoli.titolo.value;
                var testo = document.articoli.testo.value;
                var sezione = document.articoli.sezione.options[document.articoli.sezione.selectedIndex].value;
                var categoria = document.articoli.categoria.options[document.articoli.categoria.selectedIndex].value;
                if(titolo == "" || testo == "" || sezione == "*" || categoria == "*") {
                    alert("Tutti i campi sono obbligatori");
                    return false;
                }
            }
        </script>

questa la pagina:

<?php
if ($_SERVER['REQUEST_METHOD'] == "POST") {
    saveArticle($_POST['titolo'], $_POST['testo'], $_POST['sezione'], $_POST['categoria'], $_POST['tags'], $_POST['pubblicato']);
    header('location:index.php');
}
?>
    <table>
        <form name="articoli" action="<?php $_SERVER['PHP_SELF']; ?>" onsubmit="valida();" method="post">
            <tr>
                <td>Titolo:</td>
                <td><input type="text" name="titolo" value="" /></td>
            </tr>
            <tr>
                <td>
                </td>
                <td>
                    <textarea cols="" rows="" id="testo_articolo" name="testo">
                    </textarea>
                </td>
            </tr>
            <tr>
                <td>Sezione:</td>
                <td>
                    <select name="sezione" onchange="changeSelect(this.value);" id="sezione">
                        <option value="*">*</option>
                        <?php
                        foreach (selectSection () as $rowSection) {
                            echo "<option value='" . $rowSection['sezione_id'] . "'>" . $rowSection['sezione_nome'] . "</option>";
                        }
                        ?>
                    </select>
                </td>
            </tr>
            <tr>
                <td>Categoria:</td>
                <td>
                    <select name="categoria" id="categoria">
                    </select>
                </td>
            </tr>
            <tr>
                <td>Tags:</td>
                <td><input type="text" name="tags" value="" /></td>
            </tr>
            <tr>
                <td>Pubblicato:</td>
                <td>
                    <select name="pubblicato">
                        <option value="NO">NO</option>
                        <option value="SI">SI</option>
                    </select>
                </td>
            </tr>
            <tr>
                <td><input type="submit" value="Save" name="save" /></td>
            </tr>
        </form>
    </table>