Jump to content

[SOLVED] upload file script $var issues


wmguk

Recommended Posts

Hey.

 

I have a script for changing the uploaded picture.

 

Basically I need it to say, if there is no new image selected then $pic = $_POST['pic']; however if a new image is selected then upload it and replace it.

 

Currently it sets the info in the database correctly, but when i dont select a new image to change it to, i get the error saying "There was an error uploading the file called: , please try again!" but obviously this shouldnt even been trying to run if (empty($_FILES['uploadedfile']))

 

Can anyone see my mistake?

 

if (empty($_FILES['uploadedfile']))
{   
$pic = $_POST['pic']; 
}
else
{

// Where the file is going to be placed 
$target_path = "../../images/tiles/";

// File Upload Script.
$target_path = $target_path . basename( $_FILES['uploadedfile']['name']); 
$_FILES['uploadedfile']['tmp_name']; 

if(move_uploaded_file($_FILES['uploadedfile']['tmp_name'], $target_path)) 
{
    echo "";
$pic = basename( $_FILES['uploadedfile']['name']);

} else {
    echo "There was an error uploading the file called: ".  basename( $_FILES['uploadedfile']['name']). 
    ", please try again!<br><br>";
} 	
$pic = basename( $_FILES['uploadedfile']['name']);

}

Link to comment
https://forums.phpfreaks.com/topic/113733-solved-upload-file-script-var-issues/
Share on other sites

Hey guys

 

Thanks for the help - This has been resolved thank you :)

if ($_FILES['uploadedfile']['name'] == ''){   
$pic = $_POST['pic']; 
} else {

// Where the file is going to be placed 
$target_path = "../../images/tiles/";

// File Upload Script.
$target_path = $target_path . basename( $_FILES['uploadedfile']['name']); 
$_FILES['uploadedfile']['tmp_name']; 

if(move_uploaded_file($_FILES['uploadedfile']['tmp_name'], $target_path)) {
    $pic = basename( $_FILES['uploadedfile']['name']);
} else {
    echo "There was an error uploading the file called: ".  basename( $_FILES['uploadedfile']['name']). 
    ", please try again!<br><br>"; } 	
}

Archived

This topic is now archived and is closed to further replies.

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.