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

[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.