[SOLUCIONADO] condicion if else no se cumple para boton file

[HackID1]

Buenas forosdelweb!!.
Mi problema es el siguiente, pienso que básico pero no se porque no me funciona la condición.

Tengo un formulario en el cual hay varios campos que se pueden modificar para luego actualizar con un

UPDATE

en otro fichero.

Bien el form contiene un campo

type='file'

para permitir seleccionar una imagen nueva, sino se selecciona se deja la anterior.

Código HTML:

<form method='POST' action='trabajos_consultar3.php' enctype = 'multipart/form-data'> 
..... otros campos
(*)Imagen: <input type = 'file' name = 'imagen' />  <br/>
.... otros campos....
<input type='submit' value='Actualizar' name='enviar'>
 

Cuando relleno los campos y selecciono una nueva imagen a subir en el type='file', el código funciona sin mostrar errores.

Pero cuando no selecciono ninguna imagen en el input type='file', el fichero que procesa el UPDATE me muestra fallos en la creación de la imagen.

Estoy haciendo un IF ELSE para conocer si se pasa un valor a esa $VARIABLE, de esta forma, pero no consigo quitar el error Cuando no se elige imagen. aunque el código funciona pero muestra ell error.

El fichero es:

Código PHP:

<?php
             
 
                /**** DATOS DE CONEXION **********/         
                $hotsdb = "localhost";  //valor de nuestra BD
                $basededatos = "control3";  //sera el valor de nuestra BD
 
                $usuariodb = "root";        //sera el valor de nuestra BD
                $clavedb = "";      //sera el valor de nuestra BD
 
 
                $conexion_db = mysql_connect("$hotsdb","$usuariodb","$clavedb")
                or die ("Conexión denegada, el Servidor de BD que solicita
                        NO EXISTE");
                $db = mysql_select_db("$basededatos", $conexion_db)
                or die ("La Base de Datos <b>$basededatos</b> NO EXISTE");
 
                
                $cadena = "";
                
                $codtrabajo=$_POST["codtrabajo"]; 
                $estado=$_POST["estado"]; 
                $pagado=$_POST["pagado"]; 
                $observacion=$_POST["observacion"]; 
                $descripcion=$_POST["descripcion"]; 
                $precio=$_POST["precio"]; 
                $fecha_entrada=$_POST["fecha_entrada"]; 
                $fecha_entrega=$_POST["fecha_entrega"]; 
                $ac=$_POST["ac"]; 
                
                
                
                //IF para nombre de la imagen, si se escribe se realiza este UPDATE sino el de ELSE
                if (isset($_FILES['imagen']) ) {
 
                //$imagen = $FILES['imagen'];
                
                /************ imagen visualizacion y creacion ***********/
                $nombre = "camisa-" . $codtrabajo ; 
                
                $im     = imagecreatefrompng($_FILES["imagen"]["tmp_name"]);
                $color = imagecolorallocate($im, 220, 210, 60);
                $px     = (imagesx($im) - 20 * strlen($cadena)) / 2 ;
                imagestring($im, 7, $px, 70, $cadena, $color);
                //imagepng($im);        //visualiza en naveg
                imagepng($im, "img/".$nombre.".png");   //guarda imagen en img/esenombre.
                imagedestroy($im);
 
                $query = "
                        UPDATE trabajos tr
                        SET     tr.codtrabajo= '".$codtrabajo."',
                                tr.estado= '".$estado."',
                                tr.pagado= '".$pagado."',
                                tr.observacion= '".$observacion."',
                                tr.descripcion= '".$descripcion."',
                                tr.precio= '".$precio."',
                                tr.tipo_imagen = '".$nombre."',
                                tr.fecha_entrada= '".$fecha_entrada."',
                                tr.fecha_entrega= '".$fecha_entrega."',
                                tr.ac= '".$ac."'
                                
                        WHERE tr.codtrabajo = '".$codtrabajo."'
                        ";
                        
                $result = mysql_query($query); 
                
                if(!mysql_query($query)) echo mysql_error();
                
                            
                $result = mysql_query($query); 
    
                
                
                include('cierra_conexion.php');   
            
                }
                else if (!isset($_FILES['imagen']) ) {
                
                //$imagen = $FILES['imagen'];
            
                $query = "
                        UPDATE trabajos tr
                        SET     tr.codtrabajo= '".$codtrabajo."',
                                tr.estado= '".$estado."',
                                tr.pagado= '".$pagado."',
                                tr.observacion= '".$observacion."',
                                tr.descripcion= '".$descripcion."',
                                tr.precio= '".$precio."',
                                tr.fecha_entrada= '".$fecha_entrada."',
                                tr.fecha_entrega= '".$fecha_entrega."',
                                tr.ac= '".$ac."'
                                
                        WHERE tr.codtrabajo = '".$codtrabajo."'
                        ";
                        
                $result = mysql_query($query); 
                
                if(!mysql_query($query)) echo mysql_error();
                
                            
                $result = mysql_query($query); 
    
                
                
                include('cierra_conexion.php');   
            
                }
                else {
                
                    echo "No hay cambios.";
                }
?>
 

Los errores que muestra en pantalla cuando no selecciono ninguna imagen son estos:

Código PHP:

Warning: imagecreatefrompng(): Filename cannot be empty in C:\xampp\htdocs\controltrabajos\trabajos_consultar3.php on line 161
 
Warning: imagecolorallocate() expects parameter 1 to be resource, boolean given in C:\xampp\htdocs\controltrabajos\trabajos_consultar3.php on line 162
 
Warning: imagesx() expects parameter 1 to be resource, boolean given in C:\xampp\htdocs\controltrabajos\trabajos_consultar3.php on line 163
 
Warning: imagestring() expects parameter 1 to be resource, boolean given in C:\xampp\htdocs\controltrabajos\trabajos_consultar3.php on line 164
 
Warning: imagepng() expects parameter 1 to be resource, boolean given in C:\xampp\htdocs\controltrabajos\trabajos_consultar3.php on line 166
 
Warning: imagedestroy() expects parameter 1 to be resource, boolean given in C:\xampp\htdocs\controltrabajos\trabajos_consultar3.php on line 167
 

Y no debería ya que para eso esta la condición IF que si no selecciono imagen en el input file se salte ese trozo de código.

¿¿ Que puede ser.??

Un saludoo y gracias !!

[quimfv]

Código PHP:

if (!empty($_FILES)) {
 

Intenta esto

[pateketrueke]

De hecho el array $_FILES nunca vendrá vacío, sin embargo contiene un índice de error que sirve precisamente para determinar si se eligió o no un archivo.

Hay que revisar el manual: http://php.net/manual/es/features.file-upload.php

[HackID1]

He intentado colocando el

empty

en el código pero tampoco me funciona, sigue mostrando los fallos en pantalla.

Saben como puedo por lo menos ocultarlos o otra forma de comprobar si se sube un fichero o no cuando se rellena el formulario y se actualiza con el

UPDATE.

Pateketrueke he mirado el link pero no encuentro info sobre el índice de error que me dices que podría valer para comprobar si hay archivo para subir o no en mi formulario

Gracias!! Aún no he podido solucionar este fallo. :(

Saludos

[Malenko]

Iniciado por HackID1

Pateketrueke he mirado el link pero no encuentro info sobre el índice de error que me dices que podría valer para comprobar si hay archivo para subir o no en mi formulario

Gracias!! Aún no he podido solucionar este fallo. :(

Saludos

Yo no tengo mucha idea de php, pero vamos, diria que en el enlace que te han puesto viene el código para comprobar (entre otras) la situación que tu comentas.

[HackID1]

He seguido leyendo información y al fin he dado con la solución para conocer como saber si hay fichero seleccionado o no.

He cambiado la línea 34 del código anterior:

Código PHP:

if (isset($_FILES['imagen']) ) {
 

Por esta condición:

Código PHP:

if( $_FILES['imagen']['type'] == '' ) {
    $error = true;  //si error es TRUE es Q no se selecciono fichero. pasa a ELSE
    echo 'No modificada la Imagen!';
                
}
                
if(empty($error))
{
//aqui el resto de código de creacion de imagen...
//desde la LINEA 38 hasta la 74
 

Con esto compruebo que el tipo de archivo no sea '' (vacio). Y esta forma me funciono.

Y por ultimo he quitado el else if y tan solo he puesto un else.
Con lo que hay solo 2 condiciones una efectua la consulta si se sube la imagen y otra si no se sube imagen.

Gracias por vuestras sugerencias me vinieron muy bien, veo que este código lo puedo mejorar mucho, hasta para permitir tamaño máx o varios formatos de imagen.

:)