Statement running regardless of If Statement - php

Question

The 'unlink()' method is executed even though no images have been uploaded, why is this and how can I fix it?
If (isset($_FILES["image"]["name"])) {
move_uploaded_file($_FILES["imageUp"]["tmp_name"],
"C:/xampp/htdocs/" . $_FILES["image"]["name"]);
//if (file_exists($delete)) {
unlink($delete);
}
else {
$img = $data['row']->image_name;
}
//}

Answer 1

isset($_FILES["image"]["name"]
will always be set even if you upload a file or not. isset checks whether the thing is set or not. So it will always be true.
So you need to change your code :
if (isset($_FILES["image"]["name"]) && !empty($_FILES["image"]["name"])) {
move_uploaded_file($_FILES["imageUp"]["tmp_name"],
"C:/xampp/htdocs/" . $_FILES["image"]["name"]);
//if (file_exists($delete)) {
unlink($delete);
} else {
$img = $data['row']->image_name;
}
Please try with this. Hope this helps :)

Answer 2

According to the php documentation move_uploaded_file returns TRUE if everything went OK and FALSE if there was a problem.
So as you say, the image is not moving because move_uploaded_file is returning FALSE.
You can prove changing your code to this:
var_dump(move_uploaded_file($_FILES["imageUp"]["tmp_name"],
"C:/xampp/htdocs/" . $_FILES["image"]["name"]));
And seeing what is returning.
move_uploaded_file will return FALSE when:
The destination doesnt exists
The server doesnt have enough permissions to move the file to the destination
The file did not upload because it is bigger than the php.ini variable upload_max_filesize
The file did not upload because it is bigger than the php.ini variable post_max_size
Check and correct all those points and it should be working.

Answer 3

The real reason why unlink() is executed is that you've disabled the if-condition
if (file_exists($delete)) by using //.
here is the code, you may want to write:
if (isset($_FILES["image"]["name"]) && is_uploaded_file($_FILES['image']['tmp_name'])) {
move_uploaded_file($_FILES["image"]["tmp_name"],
"C:/xampp/htdocs/" . $_FILES["image"]["name"]);
if (file_exists($delete)) {
unlink($delete);
} else {
$img = $data['row']->image_name;
}
}
and make sure $delete is save! Imagine if $delete is equal to 'C:\users\your-name\some-private-file'.
If you need more help, please give us more code sourrounding this snippet.