0

Buen día,

Tengo un script PHP el cual envía un correo con ciertas variables que vienen recibidas por medio del metodo POST y a su vez guarda en una base de datos MySql mencionada info.

Tuve un problema y es que se enviaban correos por si solos como si algo en el servidor ejecutara el script, llegando vacios aun así cuando en el .js (por medio de Ajax) no permitía que se enviaran los datos al .php hasta que el usuario llenara todos los datos por lo cual me imaginé que se ejecutaba solo el PHP y no que quizá el .js enviaba datos "burlando" esa validación de no enviar datos vacíos.

ese problema del correo lo corregí validando en el .php que las variables no estuvieran vacías (lo cual es obvio no deben estar vacías por la validación en el .js), aquí la validación

if (!empty(@$_POST['nuevo_id'] && @$_POST['nombre'] && @$_POST['celular'] && @$_POST['direccion'] && @$_POST['correo'])){

//aquí envío el correo y solucioné que no se enviarán correos solos sin información
}

resulta y pasa que el código que guarda la info en la base de datos (que son exactamente las mismas variables usadas para enviar el correo), el cual quedó por fuera del if, se ejecuta solo así como pasaba con el correo, esto me genera que hayan registros vacíos en la base de datos, en 3 días ya tenia 250 registros nuevos todos en blanco, imagino que la solución sera igual a la del correo y es lo que probaré, poner el código que guarda en la base de datos dentro del if que me valida que las variables no estén vacías mi pregunta es:

Eso a que se debe? por que se ejecuta el código solo?, es necesario que siempre que haga consultas MySql oi enviar correos tenga que validar los campos aunque sea obvio que no deben estar vacíos? o debo agregar algo adicional, les dejo la consulta por si algo:

$agregar_info_usuario = mysqli_query($conexion,"UPDATE `gabriel_SoftwareTecflucol`.`solicitudes`
SET `nombre`='$nombre',
`celular`='$celular',
`email`='$correo',
`ruta_foto1`='$ruta_final_foto1',
`ruta_foto2`='$ruta_final_foto2',
`direccion`='$direccion',
`comentarios`='$comentarios'
WHERE `solicitudes`.`id`='$nuevo_id';");
10
  • ¿Cual es el error?
    – Oswuell
    Commented el 4 may. 2018 a las 20:32
  • si es posible muestra todo el codigo php de tu archivo. la información que has proporcionado no hace denotar errores.
    – user47993
    Commented el 4 may. 2018 a las 20:35
  • Si se esta enviando el correo por si solo, no es magia, solo no se esta controlando. Te aconsejo validar primero el insert en tu base de datos si y solo si los registros se insertan con éxito, pues envíes el correo @Gabriel Uribe Gomez Commented el 4 may. 2018 a las 20:48
  • Si por ejemplo tú haces submit de un formulario y el mismo llama a un PHP, debes evitar que el PHP se ejecute usando preventDefault u otro método parecido, o de lo contrario, se ejecutará el PHP que tenga el formulario en la etiqueta action. Eso por un lado. Por otro, por lo que veo, te faltan muchas validaciones y controles por establecer, no solamente en el código, el cual no debe hacer lo que le plazca, tú eres el programador y eres quien debes decirle al código lo que debe hacer. Por otra parte, la base de datos tampoco debería admitir valores en blanco o duplicados.
    – A. Cedano
    Commented el 4 may. 2018 a las 20:48
  • @A.Cedano como podría hacer lo que mencionas de que la base de datos no reciba datos en blanco? Commented el 4 may. 2018 a las 20:54

1 respuesta 1

0

Codigo PHP se ejecuta por si solo?

PHP evalúa (ejecuta) TODO el código cuando mandas llamar algún script. A menos que separes el código en funciones, y mandes llamar esas funciones de forma condicional. Entonces, las funciones solo se ejecutarán cuando las condiciones adecuadas se cumplan.

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.