Message: Undefined index: image on updating image PHP

Question

I want to update my image but I want if I am not updating any image then the previous image should not be removed But right now if I am not passing "image" parameter then I am getting the following error "Message: Undefined index: image", I just want that if I do not pass "image" parameter then the error should not display, How can I resolve this error? Here is my code

if (!file_exists($_FILES['image']['tmp_name']) || !is_uploaded_file($_FILES['image']['tmp_name'])) {
} else {
    $filename = time() . uniqid(rand()) . $_FILES['image']['name'];
    move_uploaded_file($_FILES["image"]["tmp_name"], "vendorProfile/" . $filename);
    $saveArr['image'] = $filename;
}

If you test that ou have uploaded an image using `isset` you can fork the logic to prevent the error — Professor Abronsius, Nov 25 '19 at 09:03
Possible duplicate of ["Notice: Undefined variable", "Notice: Undefined index", and "Notice: Undefined offset" using PHP](https://stackoverflow.com/questions/4261133/notice-undefined-variable-notice-undefined-index-and-notice-undefined) — Nico Haase, Nov 25 '19 at 10:34

score 0 · Accepted Answer · answered Nov 25 '19 at 09:13

Using file_exists($_FILES['image']['tmp_name'] where there has been no image upload causes the undefined index error because you assume that $_FILES['image'] does exist at that point so you should test initially for whether or not image exists in the $_FILES array - using isset for example. I had not intended this to be an answer as I felt the comment should have been enough but perhaps it will help.

if( isset( $_FILES['image'] ) ){
    /* an image was uploaded */
    $obj=(object)$_FILES['image'];
    $tmp=$obj->tmp_name;
    $name=$obj->name;

    if( is_uploaded_file( $tmp ) ){
        $filename = time() . uniqid( rand() ) . $name;
        move_uploaded_file( $tmp, "vendorProfile/{$filename}" );
        $saveArr['image'] = $filename;  
    }

} else {
    /* no image uploaded - do something else */

}

score 0 · Answer 2 · edited Nov 25 '19 at 10:36

Dear Jigar as per explanation given by you I can understand that you are trying to upload a file somewhere and you are getting an undefined index error message if you are not selecting the file.

As per your code first you are trying to check if the file exist or not with below line of code:

(!file_exists($_FILES['image']['tmp_name']) || !is_uploaded_file($_FILES['image']['tmp_name'])) {
        }

and you are trying to upload with else block

In my opinion you should first

define empty array to save image element in it.
```
$saveArr[] = '';
```

do the uploading in if block instead of else. enter code here

if(!file_exists($_FILES['image']['tmp_name']) || 
 !is_uploaded_file($_FILES['image']['tmp_name'])){ 

 $filename = time() . uniqid(rand()) . $_FILES['image']['name'];
 move_uploaded_file($_FILES["image"]["tmp_name"], "vendorProfile/" . $filename);
 $saveArr['image'] = $filename;}

Message: Undefined index: image on updating image PHP

2 Answers2