Problema con fopen()

[rikardinho90]

Ciao a tutti,

in un form php ho 2 campi in cui l'utente può inserire un'immagine:

<input name="immagine1" id="immagine1" type="file">
<input name="immagine2" id="immagine2" type="file">

Nello script recupera_file.php ho questo:

//recupero info su immagine1 
$immagine1 = $_FILES['immagine1']['tmp_name']; 
$immagine_name1 = $_FILES['immagine1']['name']; 
$immagine_size1 = $_FILES['immagine1']['size']; 
$immagine_type1 = $_FILES['immagine1']['type']; 

//recupero info su immagine2 
$immagine2 = $_FILES['immagine2']['tmp_name']; 
$immagine_name2 = $_FILES['immagine2']['name']; 
$immagine_size2 = $_FILES['immagine2']['size']; 
$immagine_type2 = $_FILES['immagine2']['type']; 

poi dopo i vari controlli su tipo di file e dimensione passo a questo che invia le immagini alla directory indicata

//Invia immagini alla directory 
if (!move_uploaded_file($_FILES['immagine1']['tmp_name'], 'img/'.$immagine_name1) || !move_uploaded_file($_FILES['immagine2']['tmp_name'], 'img/'$immagine_name2.)) 
{ $msg = 'Errore' 
} 

Il problema che ho è questo: se inserisco tutte e due le immagini nei campi immagine 1 e immagine2 tutto funziona correttamente, se invece voglio inserire solo una immagine lasciando vuoto il campo immagine2 mi dice che l'argomento di fopen() non può essere vuoto. Come posso fare per ovviare a questo problema, cioè avere le possibilità di scegliere se inserire 1 o 2 immagini?

[homerbit]

ti basta fare un controllo a monte dell'upload sul valore di immagine1\2

[rikardinho90]

in che modo?

[clasku]

a me il tuo codice funziona correggendo questa riga

//Invia immagini alla directory  
if (!move_uploaded_file($_FILES['immagine1']['tmp_name'], 'img/'.$immagine_name1) || !move_uploaded_file($_FILES['immagine2']['tmp_name'], 'img/'$immagine_name2.))  
{ $msg = 'Errore'  
} 

così

//Invia immagini alla directory  
if (!move_uploaded_file($_FILES['immagine1']['tmp_name'], 'img/'.$immagine_name1) || !move_uploaded_file($_FILES['immagine2']['tmp_name'], 'img/'.$immagine_name2))  
{ $msg = 'Errore'  
} 

[rikardinho90]

ho sbagliato trascrivendo il punto... il problema non è l'upload delle immagini...è che se io il campo lo voglio lasciare vuoto lui mi da errore xkè non è stato scelto nessun file

[clasku]

io ho fatto queste prove:
1. due campi compilati => upload ok;
2. primo campo compilato e secondo vuoto => upload ok;
3. primo campo vuoto e secondo compilato => upload ok;
4. tutti e due vuoti => nessun upload (e direi...)

Non ho mai ricevuto errori, né a schermo né nel log di apache

[rikardinho90]

ok allora forse l' "errore" è questo controllo che faccio sul tipo di immagine che viene caricata:

if ($immagine_type1 != "image/jpeg" || 
    $immagine_type2 != "image/jpeg")      
{ echo 'Il formato consentito è solo jpeg';
} 

Se lascio un campo vuoto ovviamente non riconosce il formato come jpeg. Come posso risolvere? E' necessario che le immagini caricate siano solo jpeg.

[clasku]

eh, dovresti verificare a monte di tutto che il dato non sia vuoto e poi valorizzare le tue variabili...
non so, mi viene in mente che forse potresti
1. verificare che FILES non sia vuoto;
2. inserire i dati diversi da vuoto in un array e poi caricarli usando ad esempio un foreach.

Avresti anche un altro vantaggio in questo modo: inserire più campi FILES nel form senza dover modificare il codice per l'upload