Que puedo tener mal en este codigo

Zarkiria · #1 (**permalink**) 26/05/2009, 10:16

Hola este sodigo me inserta datos en la base de datos, el caso esque inserto los campo todos bien incluida la imagen (elegida con un file) y me dice que la imagen no esta insertada, me he revisado el codigo pero no veo nada asi que sea anormal, me lo podeis hechar un vistazo?? Gracias.

<?php
include_once("conexion.php");
include_once("imagenes.php" );

$ruta = $_FILES['imagen']['name'];
$rutatemp = $_FILES['imagen']['tmp_name'];
$ruta= str_replace(" ", "", $ruta);
$tipo= $_FILES['imagen']['type'];
$tamano = $_FILES['imagen']['size'];
if(!((strpos($tipo, "gif") || strpos($tipo, "jpeg")) && ($tamano < 100000))) {
echo 'La extensión o el tamaño de los archivos no es correcta';
}
else
{
$destino = "upload/{$ruta}";
move_uploaded_file($rutatemp,$destino);
crearThumb( $destino, $destino, 120, true );
}
?>


<?php
if (isset($_POST[enviar])) {
if(!$_POST[titulo]){
$error .='- El campo titulo es obligatorio ';
}
if(!$_POST[descri]){
$error .='- El campo descripción es obligatorio. ';
}
if (!$_POST[fecha]){
$error .='- El campo fecha es obligatorio. ';

}
if(!$_POST[textolargo]){
$error .='- El campo descripción larga es obligatorio. ';
}
if (!$_POST[imagen]){
$error .='-El campos imagen es obligatorio. ';
}
if(count($error)==0) {
$error.='La noticia ha sido insertada correctamente';
$cnn=mysql_connect($hostname,$username,$password) or die (mysql_error());
mysql_select_db($database,$cnn);
$insertSQL="insert into noticias(id_noticia,titulo,descripcion,texto_largo ,fecha,hora,imagen,quien)VALUES('".
$_REQUEST['idnoticia']."','".
$_REQUEST['titulo']."','".
$_REQUEST['descri']."','".
$_REQUEST['textolargo']."','".
$_REQUEST['fecha']."','".
$_REQUEST['hora']."','".
$ruta."','".
$_REQUEST['quien']."')";
$Result1=mysql_query($insertSQL,$cnn) or die (mysql_error());
mysql_close($cnn);
}
}

eulerss · #2 (**permalink**) 26/05/2009, 10:40

ya trataste de imprimir el valor que trae tu varible ruta para probar??

Zarkiria · #3 (**permalink**) 26/05/2009, 10:49

Si y nada, me sigue saliendo q la imagen se inserto correctamente, cuando esta insertada, y encima no me la inserta en la bd, puede ser por el campo de imagen al ser un file??

Ryo · #4 (**permalink**) 26/05/2009, 11:35

El problema es que te salta este error, ¿no?

Código PHP:

  if (!$_POST[imagen]){
$error .='-El campos imagen es obligatorio.<br><br>';
}

Prueba a quitar esa condición y a usar esta otra:

Código PHP:

  if ( !is_uploaded_file( $_FILE['imagen']['tmp_name'] ) )

De hecho, yo comprobaría que la imagen está subida antes de tratarla.

Veo por tu código que estás usando la función crearThumb que publiqué en otro tema. Me alegro de que te esté sirviendo ;)

MegaZord · #5 (**permalink**) 26/05/2009, 11:40

los permisos de la carpeta estan bien??

que error especifico lanza?

aunque me preocuparia mas de la seguridad en ese codigo en todo caso.

salud2

Zarkiria · #6 (**permalink**) 26/05/2009, 11:51

Con esa condicion tampoco, errores no me salen, solo que me dice que hay que insertar una imagen cuando ya esta insertada, y claro no me lo sube al servidor, sin embargo si kito la condicion va perfecto!!!!

Ryo · #7 (**permalink**) 26/05/2009, 11:55

La condición que te he dicho has de ponerla antes de tratar la imagen. Es decir, antes de esto:

Código PHP:

  $ruta = $_FILES['imagen']['name'];

Si is_uploaded_file devuelve true entonces tratas la imagen, si no das el error.

MegaZord · #8 (**permalink**) 26/05/2009, 11:57

pero no veo un echo diciendo "que hay que insertar una imagen "

podrias especificar la condicion

gracias

iandrade · #9 (**permalink**) 26/05/2009, 12:33

NO se si sea esto
if(!((strpos($tipo, "gif") || strpos($tipo, "jpeg")) && ($tamano < 100000)))

Deberia ser algo asi como
if(!((strpos($tipo, "gif")!==false || strpos($tipo, "jpeg")!==false) && ($tamano < 100000)))

Ya que puede regresar la posicion 0 y puede ser tratada como false

MegaZord · #10 (**permalink**) 26/05/2009, 12:35

dificilmente un archivo se pueda llamar jpg sin nada mas por eso noc reo que lanze possicion cero

al<menos q se llame jpg.jgg :S aunke no falta ujau

strpos($tipo, "gif",1) ahi te aseguras que no sea 0 ni q empieze con jpg.jpg

iandrade · #11 (**permalink**) 26/05/2009, 12:40

Cita:

Iniciado por MegaZord

dificilmente un archivo se pueda llamar jpg sin nada mas por eso noc reo que lanze possicion cero

al<menos q se llame jpg.jgg :S aunke no falta ujau

strpos($tipo, "gif",1) ahi te aseguras que no sea 0 ni q empieze con jpg.jpg

No esta en el nombre esta en $tipo. que es el tipo mime del archivo(si lo tiene)

MegaZord · #12 (**permalink**) 26/05/2009, 12:42

a ok ok

mmm entonces lo mas seguro que pda regresar un cero o no?

aunque creo que el tipo depende mucho del navegador, y es vulnerable creo yo.

Zarkiria · #13 (**permalink**) 27/05/2009, 03:51

He intentado hacer esto, q me lo he encontrado por ahi, pero al no insertar una imagen no me muestra el mensaje, me muestran todos los mensajes menos el de la imagen, asi no me podria funcionar?

if (isset($_POST[enviar])) {
if(!$_POST[titulo]){
$error .='- El campo titulo es obligatorio ';
}
if(!$_POST[descri]){
$error .='- El campo descripción es obligatorio. ';
}
if (!$_POST[fecha]){
$error .='- El campo fecha es obligatorio. ';

}
if(!$_POST[textolargo]){
$error .='- El campo descripción larga es obligatorio. ';
}
if (empty($file['imagen']['name'])) {
$error .'- La imagen es obligatoria. ';
}

Ryo · #14 (**permalink**) 27/05/2009, 04:44

Cita:

Iniciado por Zarkiria

if (empty($file['imagen']['name'])) {
$error .'- La imagen es obligatoria. ';
} [/B]

Sería con $_FILE y, además, la línea está mal. Tienes esto:

Código PHP:

  $error .'- La imagen es obligatoria.<br><br>';

Y sería esto:

Código PHP:

  $error .='- La imagen es obligatoria.<br><br>';

De todas formas, te complicas demasiado. Lo que deberías hacer es usar el is_uploaded_file al tratar la imagen de esta forma:

Código PHP:

  if ( is_uploaded_file( $_FILE['imagen']['tmp_name'] ) ) {

//Aquí tratas la imagen, la subes, haces el thumbnail...

} else {

$error .='- La imagen es obligatoria.<br><br>';

}

Zarkiria · #15 (**permalink**) 27/05/2009, 04:58

Asi?

if (isset($_POST[enviar])) {
if(!$_POST[titulo]){
$error .'- El campo titulo es obligatorio ';
}
if(!$_POST[descri]){
$error .'- El campo descripción es obligatorio. ';
}
if (!$_POST[fecha]){
$error .'- El campo fecha es obligatorio. ';
}
if(!$_POST[textolargo]){
$error .'- El campo descripción larga es obligatorio. ';
}
if ( is_uploaded_file( $_FILE['imagen']['tmp_name'] ) ) {
$destino = "upload/{$ruta}";
move_uploaded_file($rutatemp,$destino);
crearThumb( $destino, $destino, 120, true );
}
else
{
$error .'- La imagen es obligatoria. ';
}

Ryo · #16 (**permalink**) 27/05/2009, 05:10

Sigues teniendo esto:

Código PHP:

  $error .'- La imagen es obligatoria.<br><br>';

Te falta la asignación en sí (=). Es decir, así:

Código PHP:

  $error .= '- La imagen es obligatoria.<br><br>';

Además, en esta versión no tienes definidas las variables $ruta ni $rutatemp: ten eso también en cuenta.

Zarkiria · #17 (**permalink**) 28/05/2009, 02:32

Nose que es lo que estoy haciendo, lo tengo tal y cm me has dicho (las variables $ruta y $rutatemp las tengo definidas mas arriba) y me sigue mostrando q inserte una imagen cnd ya esta insertada, q leches pasa???

if (isset($_POST[enviar])) {
if(!$_POST[titulo]){
$error .='- El campo titulo es obligatorio ';
}
if(!$_POST[descri]){
$error .='- El campo descripción es obligatorio. ';
}
if (!$_POST[fecha]){
$error .='- El campo fecha es obligatorio. ';
}
if(!$_POST[textolargo]){
$error .='- El campo descripción larga es obligatorio. ';
}
if ( is_uploaded_file( $_FILES['imagen']['tmp_name'] ) ) {
$destino = "upload/{$ruta}";
move_uploaded_file($rutatemp,$destino);
crearThumb( $destino, $destino, 120, true );
}
else
{
$error .='- La imagen es obligatoria. ';
}
if(count($error)==0) {
$error.='La noticia ha sido insertada correctamente';
$cnn=mysql_connect($hostname,$username,$password) or die (mysql_error());
mysql_select_db($database,$cnn);
$insertSQL="insert into noticias(id_noticia,titulo,descripcion,texto_largo ,fecha,hora,imagen,quien)VALUES('".
$_REQUEST['idnoticia']."','".
$_REQUEST['titulo']."','".
$_REQUEST['descri']."','".
$_REQUEST['textolargo']."','".
$_REQUEST['fecha']."','".
$_REQUEST['hora']."','".
$ruta."','".
$_REQUEST['quien']."')";
$Result1=mysql_query($insertSQL,$cnn) or die (mysql_error());
mysql_close($cnn);
}
}

Ryo · #18 (**permalink**) 28/05/2009, 03:40

Prueba a sustituir todos los $_FILES por $_FILE.

Zarkiria · #19 (**permalink**) 28/05/2009, 03:45

Nada, tampoco, esto es muy mosqueante...mmmm