Error in uploading multiple images

Question

I am trying to upload multiple images at once here is what i've got so far:

if(isset($_POST['submit']))
{

        $file_name=$_FILES["image"]["name"];

            foreach($file_name as $files)
            {
                $target_path = "Sub_uploads/".$files;

                if(move_uploaded_file($files["image"]["tmp_name"],$target_path)) 
                {   

                    $target_path="Sub_uploads/".$files;

                    $sql   = "INSERT INTO product_images (image) VALUES ('$target_path')"; 
                    $query = mysql_query($sql); 
                }
            }
             echo "<script>alert('data inserted');document.location='Sub_CateGory_image.php'</script>";
}
?>

It seems so that the error occurs at this line: if(move_uploaded_file($files["image"]["tmp_name"],$target_path))

Just replace $_FILES["image"]["tmp_name"] with $files["image"]["tmp_name"] — Vishal S. Pawar
– Vishal S. Pawar, Commented Mar 14, 2018 at 6:13

Mayank Majithia · Accepted Answer · 2018-03-14 06:42:03Z

1

You need to use variable of ForEach $files

if(isset($_POST['submit']))
{

        $file_name=$_FILES;

            foreach($file_name as $files)
            {
                $target_path = "Sub_uploads/".$files["image"]["name"];

                if(move_uploaded_file($files["image"]["tmp_name"],$target_path)) 
                {   

                    $target_path="Sub_uploads/".$files["image"]["name"];

                    $sql   = "INSERT INTO product_images (image) VALUES ('$target_path')"; 
                    $query = mysql_query($sql); 
                }
            }
             echo "<script>alert('data inserted');document.location='Sub_CateGory_image.php'</script>";
}

edited Mar 14, 2018 at 6:42

answered Mar 14, 2018 at 6:07

Mayank Majithia

1,99618 silver badges21 bronze badges

Sign up to request clarification or add additional context in comments.

2 Comments

u_mulder Over a year ago

$files is a string.

u_mulder Over a year ago

Please, if you don't know the structure of $_FILES array - don't answer. Your edited answer is still incorrect.

u_mulder · Accepted Answer · 2018-03-14 06:24:30Z

0

As you iterate over array of name, the connected tmp_name property will have the same key as current iterated name. So, add a key to your foreach and get a tmp_name under this key:

$file_name = $_FILES["image"]["name"];
foreach($file_name as $key => $files)   // add `$key` here
{
    $target_path = "Sub_uploads/".$files;
    // use `$key` to get connected `tmp_name`
    if(move_uploaded_file($_FILES["image"]["tmp_name"][$key], $target_path)) 
    {   
        $target_path="Sub_uploads/".$files;
        $sql   = "INSERT INTO product_images (image) VALUES ('$target_path')"; 
        $query = mysql_query($sql); 
    }
}

answered Mar 14, 2018 at 6:24

u_mulder

54.8k5 gold badges54 silver badges72 bronze badges

Collectives™ on Stack Overflow

Error in uploading multiple images

2 Answers 2

2 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

2 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related