Multiple upload form to database

[Mr_Chew]

I have an upload form with 3 fields (attached screenshot):

• Image upload field,-Events link field, Location field

There's a javascript option to dynamically add additional instances of these fields if a user has multiple files.

The multiple image uploads currently work with the code I have now, but how do I insert multiple image paths, events field info, location field info into my database i have setup depending on the amount of additional fields the user selects?

I feel as though somewhere in my php code I need to have some kind of statements that says

"While there are images to upload, for each image, insert the image path, event link, and location field into the database as a new record until there are no more images to upload"

HTML Form:

<form method="post" action="<?php echo $_SERVER['PHP_SELF'];?>" enctype="multipart/form-data">  

Image:
<input type="file" name="images[]" /> <br />
Events Link:
<input type="text" name="events[]" /><br />
Location:
<textarea name="location[]" cols="20" rows="3" ></textarea> 
 <input type="submit" name="submit" value="Submit" class="button black medium">  

 <input type="reset" name="reset1" class="button black medium">   

<p id="addnew"><a href="javascript:new_link()">Add New </a></p>  

</form>  

PHP Code:

if (isset($_POST['submit'])) {
    $number_of_file_fields = 0;
    $number_of_uploaded_files = 0;
    $number_of_moved_files = 0;
    $uploaded_files = array();
    $upload_directory = dirname(__file__) . '\\images\\locs\\resized\\'; //set upload directory

for ($i = 0; $i < count($_FILES['images']['name']); $i++) {
    $number_of_file_fields++;
	echo $number_of_file_fields;
    if ($_FILES['images']['name'][$i] != '') { //check if file field empty or not
        $number_of_uploaded_files++;
        $uploaded_files[] = $_FILES['images']['name'][$i];
        if (move_uploaded_file($_FILES['images']['tmp_name'][$i], $upload_directory . $_FILES['images']['name'][$i])) {
            $number_of_moved_files++;
        }
    }
}
echo "<p style='background-color:#DDD; border:1px solid #000; padding: 8px;'>
		<strong>Number of File fields created</strong>: ". $number_of_file_fields ."<br/>
		<strong>Number of files submitted</strong>: ". $number_of_uploaded_files . "<br/>
		<strong>Number of successfully moved files</strong>: ". $number_of_moved_files . "<br/>
		<strong>File Names are</strong><br/>" . implode(',', $uploaded_files);
}?>

My database fields are: id, image name, path, events_link, location

THANK YOU SO MUCH FOR HELP IN ADVANCE!

[bpat1434]

The code you posted has some issues with what you're looping over and the storage of your data. In your form you have images[] as the name of the file input. This means in your $_FILES array you will see the following structure when the form gets posted:

array (
    "images" => array (
        0 => array (
            "tmp_name" => "/tmp/15037156asdfadsf",
            "name" => "myPictureFile.jpg",
            "mime_type" => "image/jpeg",
            "filesize" => 135315325,
            "error" => 0
        ),
        1 => array (
            "tmp_name" => "/tmp/asdfasdf984asgd648,
            "name" => "My Custom Image.gif",
            "mime_type" => "image/gif",
            "filesize" => 115181123,
            "error" => 0
        )
    )
)

So you want to loop over $_FILES["images"] and for each item in that array, insert into the database the corresponding link and location. Here's what I would do (modified your code a bit):

<?php
	// Start a transaction if you can, will make things much easier
	mysql_query("BEGIN TRANSACTION");

// $key will be the numeric index of the uploaded file, while $image will be an array of data about the uploaded file
foreach ($_FILES["images"] as $key => $image)
{
	$number_of_file_fields++;
	echo $number_of_file_fields;

	// check for a proper upload (0 === no error, >0 === some error see: http://www.php.net/manual/en/features.file-upload.errors.php )
	if ($image["error"] == 0)
	{
		$number_of_uploaded_files++;

		$uploaded_files[] = $image["name"];

		if (move_uploaded_file($image["tmp_name"], $upload_directory . $image["name"]))
		{
			$number_of_moved_files++;

			$result = mysql_query(
				sprintf(
					"INSERT INTO `tablename` (`image_name`, `path`, `events_link`, `location`) VALUES ('&#37;s', '%s', '%s', '%s')",
					mysql_real_escape_string($image["name"]),
					mysql_real_escape_string($upload_directory),
					mysql_real_escape_string($_POST["events"][$key]),
					mysql_real_escape_string($_POST["location"][$key])
				)
			);

			if (!$result)
			{
				// Rollback the transaction if you started one (undoes all database changes in our transaction
				mysql_query("ROLLBACK");

				// Remove any moved files
				foreach ($uploaded_files as $file)
				{
					@unlink($upload_directory . $file);
				}

				throw new Exception ("Unable to save link.  " . mysql_error());
			}
		}
	}
}

// Commit the transaction if you started one
mysql_query("COMMIT");

Hope that helps.

[bradgrafelman]

@bpat: Actually, the structure of $_FILES is going to be like the following:

Array
(
    [images] => Array
        (
            [name] => Array
                (
                    [0] => myPictureFile.jpg
                    [1] => My Custom Image.gif
                )

        [type] => Array
            (
                [0] => image/jpeg
                [1] => image/gif
            )

        [tmp_name] => Array
            (
                [0] => /tmp/15037156asdfadsf
                [1] => /tmp/asdfasdf984asgd658
            )

        [error] => Array
            (
                [0] => 0
                [1] => 0
            )

        [size] => Array
            (
                [0] => 135315325
                [1] => 115181123
            )

    )

)

EDIT: For an applicable example straight from the manual, see "Example #3 Uploading array of files" on the following man page: [man]features.file-upload.post-method[/man].

[bpat1434]

Yeah... my oversight.... stupid PHP not being consistent (or moreover me for not double-checking before posting)...