[SOLUCIONADO] ¿Qué no funciona en este código, debería subir datos de imagen y actualizar campos?

[AngelMarine]

$reqlen = strlen ($large_image_name);
if ($reqlen > 0) {
require("connect_db.php");

$meter = @mysql_query('INSERT INTO images (img_name, id_user) values ("'.mysql_real_escape_string($large_image_name."." .$file_ext).'", "'.mysql_real_escape_string($id_user).'")');

//Hasta aquí funciona.

Lo siguiente no:

$meter2 = @mysql_query('UPDATE users SET profile_photo="'.mysql_real_escape_string($large_i mage_name.".".$file_ext).'" WHERE email="$email"');
}

$email está definido, ya lo he probado.

[quimfv]

No te da un error? Has mirado como se construye tu query

Código PHP:

echo 'UPDATE users SET  profile_photo="'.mysql_real_escape_string($large_i  mage_name.".".$file_ext).'" WHERE email="$email"';
 

solo cambiando los delimitadores....

Código PHP:

$meter2 = @mysql_query("UPDATE users " .
                 "SET profile_photo='".mysql_real_escape_string($large_i mage_name.'.'.$file_ext)."' " .
                 "WHERE email='".$email."'");
 

[gnzsoloyo]

Consejo, @AngelMarine: Cuando estés en etapa de desarrollo NUNCA uses el "@" para suprimir las salidas. Eso evitará que veas los errores de ejecución, y si no los ves... ¿Como sabras qué está pasando?

Ese es, a mi entender, un vicio de muchos programadores, que parecen suponer que siempre se hacen bien las cosas, y no es así.

[quimfv]

Gran consejo @gnzsoloyo... el único sentido de esa "@" es ocultar al usuario los mensajes de error del sistema, que suelen salir de forma poco elegante. Para evitar el feo de lanzar mensajes no formateados es mucho mejor hacer un buen tratamiento de excepciones try{}catch{} donde se puede formatear el mensaje de error o incluso redirigir el flujo del programa a algo que sortee o corrija el error.....

Pero sobre todo en tiempo de programación NUNCA autoocultarse información.

[gnzsoloyo]

Me gustan las Try{}Catch{}... Permiten hacer códigos controlados, depurables.

[AngelMarine]

Iniciado por quimfv

No te da un error? Has mirado como se construye tu query

Código PHP:

echo 'UPDATE users SET  profile_photo="'.mysql_real_escape_string($large_i  mage_name.".".$file_ext).'" WHERE email="$email"';
 

solo cambiando los delimitadores....

Código PHP:

$meter2 = @mysql_query("UPDATE users " .
                 "SET profile_photo='".mysql_real_escape_string($large_i mage_name.'.'.$file_ext)."' " .
                 "WHERE email='".$email."'");
 

He hecho ambas cosas, quitar @ y cambiando delimitadores, y no, no funciona, gracias de todas formas por la ayuda, dime, ¿Habrá otra manera de actualizar la tabla?

[gnzsoloyo]

Iniciado por AngelMarine

He hecho ambas cosas, quitar @ y cambiando delimitadores, y no, no funciona, gracias de todas formas por la ayuda, dime, ¿Habrá otra manera de actualizar la tabla?

No te salgas del problema. Todavía no has resuelto el tema.
¿Pusiste el echo?
¿Qué te mostró?

Postea el resultado de ese:

Código PHP:

  echo 'UPDATE users SET  profile_photo="'.mysql_real_escape_string($large_image_name.".".$file_ext).'" WHERE email="$email"'; 
   

Desde ya te digo que yo estoy viendo a simple vista un error de lógica y sintaxis.
Mira, supongamos que cargo las variables, y ejecuto solamente la parte bássica sin la funcion que pones:

Código PHP:

  $large_image_name = "abcdefghijklmnopqrstuvwxyz";
$file_ext = "123";
$email = "[email protected]";

echo 'UPDATE users SET  profile_photo="'.$large_image_name.".".$file_ext.'" WHERE email="$email"'; 
   

Eso me da como resultado esto:

Código SQL:

UPDATE users SET  profile_photo="abcdefghijklmnopqrstuvwxyz.123" WHERE email="$email"
 

¿Te parece bien?
Bueno, no, esta mal. Eso te dará un error de ejecución en MySQL, porque debería ser así:

Código SQL:

UPDATE users SET  profile_photo='abcdefghijklmnopqrstuvwxyz.123' WHERE email='[email protected]'
 

Es decir, la construcción de la sentencia está mal hecha.

[AngelMarine]

Iniciado por gnzsoloyo

No te salgas del problema. Todavía no has resuelto el tema.
¿Pusiste el echo?
¿Qué te mostró?

Postea el resultado de ese:

Código PHP:

  echo 'UPDATE users SET  profile_photo="'.mysql_real_escape_string($large_image_name.".".$file_ext).'" WHERE email="$email"'; 
   

Desde ya te digo que yo estoy viendo a simple vista un error de lógica y sintaxis.
Mira, supongamos que cargo las variables, y ejecuto solamente la parte bássica sin la funcion que pones:

Código PHP:

  $large_image_name = "abcdefghijklmnopqrstuvwxyz";
$file_ext = "123";
$email = "[email protected]";

echo 'UPDATE users SET  profile_photo="'.$large_image_name.".".$file_ext.'" WHERE email="$email"'; 
   

Eso me da como resultado esto:

Código SQL:

UPDATE users SET  profile_photo="abcdefghijklmnopqrstuvwxyz.123" WHERE email="$email"
 

¿Te parece bien?
Bueno, no, esta mal. Eso te dará un error de ejecución en MySQL, porque debería ser así:

Código SQL:

UPDATE users SET  profile_photo='abcdefghijklmnopqrstuvwxyz.123' WHERE email='[email protected]'
 

Es decir, la construcción de la sentencia está mal hecha.

Si, da esto:

UPDATE users SET profile_photo="resize_1397.jpg" WHERE email="$email"


He quitado el WHERE (ha conciencia, sabía que modificaría el campo profile_photo para todos), y efectivamente ha funcionado, ha puesto el nombre de la imagen en todos los campos;

[gnzsoloyo]

Ten mucho cuidado con el uso de comillas (") en MySQL, porque sólo si el servidor está configurado para ello, tomará el contenido como cadenas de texto. de lo contrario las interpretará como nombres de objetos de base.
La recomendación es porque por defecto se usan para nombres de objetos.
Es preferible que uses apóstrofos para las cadenas de texto. Es más seguro.

Yo te sugiero, por ejemplo:

Código PHP:

$large_image_name = "abcdefghijklmnopqrstuvwxyz";
$file_ext = "123";
$email = "[email protected]";
 
echo "UPDATE users SET profile_photo='".addslashes("$large_image_name".".$file_ext")."' WHERE email='$email'";
 

Lo que te dará como resultado:

Código SQL:

UPDATE users SET profile_photo='abcdefghijklmnopqrstuvwxyz.123' WHERE email='[email protected]'
 

que sí es correcto.

[AngelMarine]

Iniciado por gnzsoloyo

Ten mucho cuidado con el uso de comillas (") en MySQL, porque sólo si el servidor está configurado para ello, tomará el contenido como cadenas de texto. de lo contrario las interpretará como nombres de objetos de base.
La recomendación es porque por defecto se usan para nombres de objetos.
Es preferible que uses apóstrofos para las cadenas de texto. Es más seguro.

Yo te sugiero, por ejemplo:

Código PHP:

$large_image_name = "abcdefghijklmnopqrstuvwxyz";
$file_ext = "123";
$email = "[email protected]";
 
echo "UPDATE users SET profile_photo='".addslashes("$large_image_name".".$file_ext")."' WHERE email='$email'";
 

Lo que te dará como resultado:

Código SQL:

UPDATE users SET profile_photo='abcdefghijklmnopqrstuvwxyz.123' WHERE email='[email protected]'
 

que sí es correcto.

Gracias, ya funciona. También he hecho una modificación en el email, ahora compara los id´s, y he puesto el nombre de la tabla delante de user.

$meter2 = mysql_query("UPDATE users SET profile_photo='".addslashes("$large_image_name".". $file_ext")."' WHERE users.user='$id_user'");

[quimfv]

La solución de los delimitadores es la que te di yo en primera instancia....

Fijate que tu mismo nos dices porque no funcionaba

Si, da esto:

UPDATE users SET profile_photo="resize_1397.jpg" WHERE email="$email"

No esta traduciendo la variable $email a su contenido....no?


Esta seguro que asi no funcionava

Código PHP:

$meter2 = @mysql_query("UPDATE users " .
                 "SET profile_photo='".mysql_real_escape_string($large_i mage_name.'.'.$file_ext)."' " .
                 "WHERE email='".$email."'");
 

UPDATE users SET profile_photo='resize_1397.jpg' WHERE email='<aqui el valor de $email>'