Save image upload path to server folder in database

here's a script to upload multiple images to the server folder and store their path in the database, however the path to the images is only stored in one column.

For example: I am uploading 3 images, image1.jpg, image2.jpg, image3.jpg. These images are supposed to be stored in 3 columns ie offimage1, offimage2, offimage3.

Now the problem is that the path for all three images is only stored in offimage1. the path stored in offimage1 looks something like this:

uploads/image1.jpg*uploads/image1.jpgimage2.jpg*uploads/image1.jpgimage2.jpgimage3.jpg

      

I want images to be saved this way:

uploads/image1.jpg in colum offimage1 
uploads/image1.jpgimage2.jpg in colum offimage2 
uploads/image1.jpgimage2.jpgimage3.jpg in colum offimage3

      

html form

 <form enctype="multipart/form-data" action="insert_image.php?id=<?php echo $_GET['id']; ?>" method="post">
        <div id="filediv"><input name="file[]" type="file" id="file"/></div><br/>
        <input type="button" id="add_more" class="upload" value="Add More Files"/>
        <input type="submit" value="Upload File" name="submit" id="upload" class="upload"/>
    </form>

      

insert_image.php

<?php
ob_start();
require 'connection.php';
if (isset($_POST['submit'])) {
    $j = 0; //Variable for indexing uploaded image 

    $target_path = "uploads/"; //Declaring Path for uploaded images
    for ($i = 0; $i < count($_FILES['file']['name']); $i++) {//loop to get individual element from the array

        $validextensions = array("jpeg", "jpg", "png");  //Extensions which are allowed
        $ext = explode('.', basename($_FILES['file']['name'][$i]));//explode file name from dot(.) 
        $file_extension = end($ext); //store extensions in the variable

        $target_path = $target_path . md5(uniqid()) . "." . $ext[count($ext) - 1];//set the target path with a new name of image
        $j = $j + 1;//increment the number of uploaded images according to the files in array       

      if (($_FILES["file"]["size"][$i] < 100000) //Approx. 100kb files can be uploaded.
                && in_array($file_extension, $validextensions)) {
            if (move_uploaded_file($_FILES['file']['tmp_name'][$i], $target_path)) {//if file moved to uploads folder
                //echo $j. ').<span id="noerror">Image uploaded successfully!.</span><br/><br/>';

                $file_name_all.=$target_path."*";
                $filepath = rtrim($file_name_all, '*'); 
                //echo $filepath;
                $officeid = $_GET['id'];
                $sql = "UPDATE register_office SET offimage='$filepath' WHERE id='$officeid' ";
                            if (!mysqli_query($con,$sql)) 
                                {
                                    die('Error: ' . mysqli_error($con));
                                }

            } else {//if file was not moved.
                echo $j. ').<span id="error">please try again!.</span><br/><br/>';
            }
        } else {//if file size and file type was incorrect.
            echo $j. ').<span id="error">***Invalid file Size or Type***</span><br/><br/>';
        }
    }
    header("Location: co_request_sent.php ");
}
mysqli_close($con); 
?>

      

would be grateful if someone could help me

+3


source to share


1 answer


The reason your path information for all three images looks like one big line:

uploads/image1.jpg*uploads/image1.jpgimage2.jpg*uploads/image1.jpgimage2.jpgimage3.jpg

      

is that you iterate over the FILES array and concatenate $ target_path on each iteration:

$target_path = $target_path . md5(uniqid()) . "." . $ext[count($ext) - 1];

      

if you want $ target_path to only store information for the current image, reset the variable to move first:

$target_path = "uploads/";

      



inside a for loop (so move it two lines down).

This is the first part of your problem. However, storing image information in columns named offimage1, offimage2, etc. This is a very bad idea .

This thing should be stored in a row, not a column.

So you might be better off making a new caption table with columns "id", "officeid" and "path".

Now just insert a row for each image (it doesn't matter how many you get now) and make sure you bind it to the register_office table with officeid.

0


source







All Articles