Validacion funciona pero igual pasa con errores

[Fleon]

Holas a todos, tengo un sencillissimo script de validacion en JavaScript:

Código Javascript:

function validar() {
            if (document.save_cliente.cliente_nombre.value.length == 0) {
                alert("Debes Ingresar el nombre del Cliente!")
                document.save_cliente.cliente_nombre.focus()
                return 0;
            }
        }
 

Ese script funciona bien, pero aun tener el campo vacio me guarda los datos en PHP, o sea, no evita que pase la moificacion, este es el script de guardar:

Código PHP:

elseif (isset($_REQUEST['do']) && $_REQUEST['do'] == 'save') {      
            $smcFunc['db_query']('', '
                UPDATE {db_prefix}da_anuncios_clientes
                SET cliente_nombre = {string:name}, cliente_link = {string:link}, activo = {int:active}
                WHERE id = {int:cliente_id}',
                array(
                    'cliente_id' => $_POST['cliid'],
                    'name' => $_POST['cliente_nombre'],
                    'link' => $_POST['cliente_url'],
                    'active' => $_POST['cliente_activo'],
                )
            );
            
            redirectexit('action=admin;area=manageanuncios;sa=clientes');   
       }
 

Este es el formulario:

Código PHP:

echo '
    <div id="manage_boards">
        <form action="', $scripturl, '?action=admin;area=manageanuncios;sa=editcliente;do=save" name="save_cliente" method="post" accept-charset="', $context['character_set'], '" onSubmit="javascript:return validar()">
            
            <div class="cat_bar">
                <h3 class="catbg">
                    ', isset($_GET['s']) ? $txt['manage_anuncios_edit_cliente'] : $txt['manage_anuncios_add_cliente'], '
                </h3>
            </div>';
            
        foreach ($context['clientes'] as $cliente) {
            
        echo'
        <input type="hidden" name="cliid" value="', $cliente['cliente']['id'], '" />
        <div class="windowbg">
                <span class="topslice"><span></span></span>
                <div class="content">
                    <dl class="settings">
 
                        <dt>
                            <strong>', $txt['manage_anuncios_cliente_nombre'], ':</strong>
 
                        </dt>
                        <dd>
                            <input type="text" name="cliente_nombre" id="cliente_nombre" value="', $cliente['cliente']['name'], '" size="30" class="input_text" />
                        </dd>
 
                        <dt>
                            <strong>', $txt['manage_anuncios_cliente_url'], ':</strong>
                        </dt>
                        <dd>
                            <input type="text" name="cliente_url" value="', $cliente['cliente']['url'], '" size="30" class="input_text" />
                        </dd>
                        
                        <dt>
                            <strong>', $txt['manage_anuncios_cliente_activo'], ':</strong><br />
                        </dt>
                        <dd>';
                        if ($cliente['cliente']['active'] == 1)
                            $checked = 'checked="checked" value="'. $cliente['cliente']['active']. '"';
                        else 
                            $checked = 'value="0"';
                            
                            echo '<input type="checkbox" name="cliente_activo" ' . $checked . ' id="cliente_activo" />
                        </dd>
 
                    </dl>';
        }
        
        if (!isset($_GET['s']))
            echo' <input type="submit" name="add" value="', $txt['manage_anuncios_cliente_save'], '" class="button_submit" />';
        else
            echo '<input type="submit" name="edit" value="', $txt['manage_anuncios_cliente_modify'], '" class="button_submit" />';
 
 
    echo'   </div>
            <span class="botslice"><span></span></span>
        </div>
        </form>
    </div>
    <script type="text/javascript"><!-- // --><![CDATA[
        function validar() {
            if (document.save_cliente.cliente_nombre.value.length == 0) {
                alert("Debes Ingresar el nombre del Cliente!")
                document.save_cliente.cliente_nombre.focus()
                return 0;
            }
        }
    // ]]></script>';
 

Asi que no se que podria estar pasando aqui.

Fleon xD

[Triby]

Necesariamente debes validar en PHP, porque si el navegador del usuario no tiene Javascript activo entonces esa validación no se realiza.

[Fleon]

Bueno, yo hice una chapuza, pero me funciona de momento:

Código PHP:

if (isset($_REQUEST['do']) && $_REQUEST['do'] == 'edit') {  
        if ($_POST['cliente_nombre'] == '') {
            echo '<script type="text/javascript">
                    alert(\'Debes ingresar el nombre del cliente!.\');
                    history.back();
                 </script>';
                 exit;
        } else {
            $smcFunc['db_query']('', '
                UPDATE {db_prefix}da_anuncios_clientes
                SET cliente_nombre = {string:name}, cliente_link = {string:link}, activo = {int:active}
                WHERE id = {int:cliente_id}',
                array(
                    'cliente_id' => $_POST['cliid'],
                    'name' => $_POST['cliente_nombre'],
                    'link' => $_POST['cliente_url'],
                    'active' => $_POST['cliente_activo'],
                )
            );
            
            redirectexit('action=admin;area=manageanuncios;sa=clientes');   
       }
     }
 

Saludos.

[Triby]

Y si el usuario no tiene javascript habilitado?... no verá esa alerta, sólo una página en blanco.