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