controllo isset files non funzionante

[spawn88]

buongiorno a tutti,

eseguo un controllo se è stata inserita un'immagine da caricare altrimenti carica solo gli altri dati senza dare errore ma sembra che il controllo nen venga mai eseguito e non riesco a capire perchè.
questo è il codice di controllo:

<?php
        session_start();
        
        //connessione db
        $conn= mysql_connect("localhost", "root", "") or die ("impossibile connettersi");
        //selezione del DB
        mysql_select_db("archiviottico", $conn) or die ("impossibile selezionare il db");
        
        if(!isset($_FILES["files"]["tmp_name"])){
                $cev=$_SESSION['cev'];
                $utente=$_SESSION['username'];            
                $id=$_SESSION['idpolizza'];
                $idpolizza= $_POST['idpolizza'];        
                $numeropolizza= $_POST['numeropolizza'];
                $compagnia= $_POST['compagnia'];
                $note= $_POST['note'];
                $idcliente= $_POST['idcliente'];
                $cev= $_POST['cev'];
            
                $query = "UPDATE polizza SET idpolizza='$idpolizza', numeropolizza='$numeropolizza', compagnia='$compagnia', idcliente='$idcliente', note='$note', cev='$cev', utente='$utente' WHERE idpolizza='$id'";
                mysql_query($query) or die (mysql_error(""));
}//fine if
        else{
            
        
                // controllo che non ci siano stati errori durante l'upload
                if (!is_uploaded_file($_FILES['files']['tmp_name']))
                    die ("Errore di caricamento: ".$_FILES['files']['error']);
    
                    $nome = $_FILES["files"]["name"];
                    $dimensione = $_FILES["files"]["size"];
                    $cartella = "polizze/"; // cartella in cui vengono salvati file pdf delle polizze

                    if($dimensione < 10240000000) {

                        if(is_file($cartella.$nome)){
    
                            echo "Esiste già una polizza con questo nome";
        
                        }
                        else {
                            $cev=$_SESSION['cev'];
                            $utente=$_SESSION['username'];    
                            $id=$_SESSION['idpolizza'];
                            $idpolizza= $_POST['idpolizza'];        
                            $numeropolizza= $_POST['numeropolizza'];
                            $compagnia= $_POST['compagnia'];
                            $note= $_POST['note'];
                            $idcliente= $_POST['idcliente'];
                            $cev= $_POST['cev'];
        
                            // sposto il file caricato nella cartella di destinazione
                            $polizza= move_uploaded_file($_FILES['files']['tmp_name'], $cartella.$nome ) or die("Errore di upload del file.");
              
                            $query = "UPDATE polizza SET idpolizza='$idpolizza', nomepolizza='$nome', numeropolizza='$numeropolizza', dimensione='$dimensione', compagnia='$compagnia', idcliente='$idcliente', note='$note', cev='$cev', utente='$utente' WHERE idpolizza='$id'";
                            mysql_query($query) or die (mysql_error(""));
}//fine else caricamento file
            
        }//fine if $dimensione
    
        }//fine else isset FILES
?>

ho controllato passo passo ogni istruzione ma nulla.
quando modifico i dati compreso il file pdf tutto ok ma quando modifico tutti i dati tranne il file pdf mi restituisce errore di carimento 4 che è un controllo che viene eseguito solo se il file pdf è stato caricato.
dove sbaglio?
grazie a tutti in anticipo

[eiyen]

dov'è questo errore? in che riga? stampati a video il contenuto delle variabili del form per vedere come sono valorizzate

[Ranma2]

In attesa di più spiegazioni ruguardo al tuo problema ti consiglio di rivedere le tue query perché sono ad alto rischio di SQL Injection.

Ti consiglio di utilizzare une stensione di PHP, ovvero PDO

http://php.net/manual/en/book.pdo.php

o in alternativa mysql_real_escape_string in accoppiata con sprintf

http://php.net/manual/en/function.my...ape-string.php

[spawn88]

in realtà non c'è in vero e proprio errore, è che non effettua il controllo if...
se nel form cambio tutti i dati compreso il file la modifica avviene con successo, ma se modifico tutti i campi tranne il file, mi da:

Errore di caricamento:4

e non capisco dove si trovi l'errore

[spawn88]

ho provato a risolvere questo problema ma non capisco perchè sembra che non effettui il controllo

if(!isset($_FILES["files"]["tmp_name"])){ 

qualcuno può illuminarmi?

[satifal]

Ti è già stato suggerito di stampare il contenuto delle variabili!

print_r($_FILES); 

oppure:

var_dump($_FILES); 

[spawn88]

Mi stampa questo messaggio:

Array ( [files] => Array ([name] => [type] => [tmp_name] =>[error]=> 4 [size] => 0 ) )

ed è giusto che il nome, tipo e il tmp_name siano vuoti ma non dovrebbe fare questo perchè se non metto il file da modificare non dovrebbe dare Errore di caricamento 4 vorrebbe dire che non ha eseguito il controllo if

[eiyen]

ma "isset" verifica che esista la variabile... da quel che posti la variabile c'è, ma vuota, quindi controlla (anche) questo aspetto, es.:

if (!isset($_FILES["files"]["tmp_name"])) || ($_FILES["files"]["tmp_name"]=='') { 

...in realtà nel tuo caso dovrebbe bastare il secondo pezzo, ma tant'è...

[satifal]

Ma a questo punto "tmp_name" è settato, ma vuoto per cui è giusto che isset() ti ritorni true. Prova ad aggiungere il controllo sull'empty:

...
if (!isset($_FILES["files"]["tmp_name"]) || empty($_FILES["files"]["tmp_name"])) { 
... 

[spawn88]

fantastico satifal/eyen, ora funziona perfettamente...vorrei chiedervi un ultimo favore:

1. che differenza c'è tra:

empty($_FILES["files"]["tmp_name"])

e

($_FILES["files"]["tmp_name"]=='')

2. che controllo eseguono queste due "diverse" istruzioni nell'if?

grazie molte a entrambi