fecha por usuario es mayor a la fecha actual

kimberly_1612 · #1 (**permalink**) 21/05/2014, 09:15

hola, tengo un formulario donde no quiero que se guarden los datos si la fecha que el usuario introdujo es mayor a la fecha actual (generada por el sistema obviamente). consegui una funcion, y me muestra el mensaje de error , pero igual me almacena los datos. ayuda!!!!
aqui les dejo la parte del codigo que nesecito modificar, gracias de antemano

Código PHP:

  <?php

//conecto la bd antes q nada, o sea aca la estoy llamando

require('base_datos.php');

require('funciones.php');

 
 
$conexion = conectar(); //ay ya me conecte

 
?>

 
<!doctype html>

<html>

<head>

<meta charset="utf-8">

<script type="text/javascript">

function ValidaFechas(){

var dtFechaActual = new Date();

 
var sAnioFin = document.formulario.anno.value;

var sMesFin = document.formulario.mes.value;

var sDiaFin = document.formulario.dia.value;

var sFechaFin = sMesFin + "/" + sDiaFin + "/" + sAnioFin;

 
if(Date.parse(sFechaFin) > dtFechaActual)

 
alert("La fecha de factura no puede ser mayor a la fecha actual.");

return false;

 
 
 
 
}

</script>

</head>

<title>Compras/Donativos</title>

</head>

 
<body>

<form action="guardar_entradacompra.php" method="post" name="formulario" id="formulario" onSubmit="ValidaFechas()">

<table border="1" align="center" width="600" class="tabla">

<tr>

    <td>Fecha de Factura</td>

    <td>

    

    <?php fechafactura() ?>

    

   </td>

</tr>

 
<tr>

    <td>Fecha Deposito</td>

    <td><input type="date" name="fechadeposito" id="fechadeposito" class="s" readonly value="<?php echo date('d/m/Y');?>"/></td> <!-- la puse Y/d/d xq es asi como lo guarda mysql , entonces para no hacer conversion lo lea y lo guardo de esa forma-->

</tr>

</table>

<table border="1" align="center" width="600" class="tabla">

<tr>

    <td colspan="2" align="center">

    <input type="submit" class="submit" value="enviar"> 

    <input type="reset" class="reset" value="borrar">

    </td>

</tr>

 
</table>

</form>

Alexis88 · #2 (**permalink**) 21/05/2014, 14:00

Cuando deseas ejecutar más de una línea de instrucción dada una condición, debes de colocar llaves de apertura y cerrado.

Código Javascript:

Ver originalif (Date.parse(sFechaFin) > dtFechaActual){
    alert("La fecha de factura no puede ser mayor a la fecha actual."); 
    return false;
}

Por otra parte, Date.parse, devuelve la cantidad de milisegundos transcurrido desde las 00:00:00 del 1 de enero de 1970 hasta la fecha indicada, mientras que Date, sin aplicarle alguno de sus métodos, va a devolver una cadena con esta forma:

Código Date object:

Ver originalWed May 21 2014 14:41:19 GMT-0500 (Hora est. del Pacífico de SA)

Entonces, la condición nunca se cumplirá. Para obtener el valor equivalente al que obtienes con Date.parse, debes utilizar el método getTime().

Código Javascript:

Ver originalif (Date.parse(sFechaFin) > dtFechaActual.getTime()){
    alert("La fecha de factura no puede ser mayor a la fecha actual."); 
    return false;
}

Y en cuanto a la cancelación del envío de datos del formulario, en lugar de usar el return false, preferiría hacerlo con el método preventDefault(), más o menos de esta forma:

Código Javascript:

Ver originalvar formulario = document.getElementById("formulario");
 
formulario.addEventListener("submit", function(event){
    event.preventDefault(); //Cancelo el envío
 
    var dtFechaActual = new Date(),
        sAnioFin = document.formulario.anno.value,
        sMesFin = document.formulario.mes.value,
        sDiaFin = document.formulario.dia.value,
        sFechaFin = sMesFin + "/" + sDiaFin + "/" + sAnioFin; 
 
    if(Date.parse(sFechaFin) > dtFechaActual.getTime())
        alert("La fecha de factura no puede ser mayor a la fecha actual.");
    else
        this.submit(); //Si todo está bien, envío el formulario
}, false);

Quita el atributo onsubmit del formulario, elimina (o comenta) la función que hiciste y prueba con esta solución.

Saludos