Jump to content

want to keep this script but just add for extra image upload


justAnoob

Recommended Posts

this script works great for uploading 1 picture into my server directory and the image path to mysql. As you can see below my filefield name is "image" for the one picture that can be uploaded. If I added 4 more fields....

"image2" "image3" "image4" "image5"  to my form... Would it be complicated to just add the upload of these fields into the script????  The first image is required when you upload,,,, but the other 4 will not be required.  I'm kinda confused,, other people tell me to just start with a new script... But I like the one I have.

<?php
session_start();
include "connection.php";
mysql_connect("$host", "$username", "$password") or die("Could not connect.");
mysql_select_db("$db_name") or die("Could not find database");
$item_name = mysql_real_escape_string($_POST['item_name']);
$description = mysql_real_escape_string($_POST['description']);
$in_return = mysql_real_escape_string($_POST['in_return']);

define ("MAX_SIZE","1000");
function getExtension($str)
{
$i = strrpos($str,".");
if (!$i)
{
	return "";
}
$l = strlen($str) - $i;
$ext = substr($str,$i+1,$l);
return $ext;
}
$errors=0;
if(isset($_POST['submit']))
{
$image=$_FILES['image']['name'];
if ($image)
{
	$filename = stripslashes($_FILES['image']['name']);
	$extension = getExtension($filename);
	$extension = strtolower($extension);
	if (($extension != "jpg") && ($extension != "jpeg") && ($extension != "gif") && ($extension != "png"))
	{
		echo '<h1>Picture #1 is not in correct format.</h1>';
		$errors=1;
	}
	else
	{
		$size=filesize($_FILES['image']['tmp_name']);
		$size2=filesize($_FILES['image2']['tmp_name2']);
		if ($size || $size2 > MAX_SIZE*1024)
		{
			echo '<h1>You have exceeded the size limit on one of your pictures.</h1>';	$errors=1;
		}

		$category = $_POST['listmenu'];
		$image_name=time().'.'.$extension;
		$newname="userimages/$category/".$image_name;
		$copied = copy($_FILES['image']['tmp_name'], $newname);
		if (!$copied)
		{
			echo '<h1>File #1 unsuccessfull!</h1>';
			$errors=1;
		}
	}
}
}
$mysqlcategory = $category;
$imgpath = $newname;
$findit = $_SESSION['id'];
$result=mysql_query("SELECT id FROM members WHERE username = '$findit'");
$row=mysql_fetch_assoc($result);
$user_id = $row['id'];
$sql = "INSERT INTO member_trades(item_name, description, in_return, imgpath, category, user_id)VALUES('$item_name','$description','$in_return', '$imgpath', '$mysqlcategory', '$user_id')";
mysql_query($sql) or die(mysql_error());

// go to confirmation page if upload is completed.
if(isset($_POST['submit']) && !$errors)
{
header ("http://www.xxxxxxxxxxx.com/previewsave.php");
echo "<h1>Image Uploaded Successfully!</h1>";
echo '<img src="' . $newname . '" width="150" border="0"><br />'; 
} 
?>

Link to comment
Share on other sites

I tried doing something like this.....

 

if(!empty(image2))

{

    run the upload script again just changing the variables to work with the filefield name "image2"

}

if(!empty(image3))

{

    run the upload script etc........

}

 

But i think I ran into the problem where everything gets inserted into MySQL

 

Here is the line originally with just having 1 upload box..

<?php
$mysqlcategory = $category;
$imgpath = $newname;
$findit = $_SESSION['id'];
$result=mysql_query("SELECT id FROM members WHERE username = '$findit'");
$row=mysql_fetch_assoc($result);
$user_id = $row['id'];
$sql = "INSERT INTO xxxxxxxxx(item_name, description, in_return, imgpath, category, user_id)VALUES('$item_name','$description','$in_return', '$imgpath', '$mysqlcategory', '$user_id')";
mysql_query($sql) or die(mysql_error());?>

 

Then when I add the other image2, image3, image4 and image5 filefield boxes for optional upload...that would just create a ton variables and a big mess.?..... Could I do a loop of some sort??? I'm not that good with loops.

Link to comment
Share on other sites

Here is a correction of the original script that I'm working with...

<?php
session_start();
include "connection.php";
mysql_connect("$host", "$username", "$password") or die("Could not connect.");
mysql_select_db("$db_name") or die("Could not find database");
$item_name = mysql_real_escape_string($_POST['item_name']);
$description = mysql_real_escape_string($_POST['description']);
$in_return = mysql_real_escape_string($_POST['in_return']);

define ("MAX_SIZE","1000");
function getExtension($str)
{
$i = strrpos($str,".");
if (!$i)
{
	return "";
}
$l = strlen($str) - $i;
$ext = substr($str,$i+1,$l);
return $ext;
}
$errors=0;
if(isset($_POST['submit']))
{
$image=$_FILES['image']['name'];
if ($image)
{
	$filename = stripslashes($_FILES['image']['name']);
	$extension = getExtension($filename);
	$extension = strtolower($extension);
	if (($extension != "jpg") && ($extension != "jpeg") && ($extension != "gif") && ($extension != "png"))
	{
		echo '<h1>Picture is not in correct format.</h1>';
		$errors=1;
	}
	else
	{
		$size=filesize($_FILES['image']['tmp_name']);
		if ($size > MAX_SIZE*1024)
		{
			echo '<h1>You have exceeded the size limit on one of your pictures.</h1>';	$errors=1;
		}

		$category = $_POST['listmenu'];
		$image_name=time().'.'.$extension;
		$newname="userimages/$category/".$image_name;
		$copied = copy($_FILES['image']['tmp_name'], $newname);
		if (!$copied)
		{
			echo '<h1>File #1 unsuccessfull!</h1>';
			$errors=1;
		}
	}
}
}

// if everything is good, post new item for the user
$mysqlcategory = $category;
$imgpath = $newname;
$findit = $_SESSION['id'];
$result=mysql_query("SELECT id FROM members WHERE username = '$findit'");
$row=mysql_fetch_assoc($result);
$user_id = $row['id'];
$sql = "INSERT INTO xxxxxxxxx(item_name, description, in_return, imgpath, category, user_id)VALUES('$item_name','$description','$in_return', '$imgpath', '$mysqlcategory', '$user_id')";
mysql_query($sql) or die(mysql_error());

// go to confirmation page if upload is completed.
if(isset($_POST['submit']) && !$errors)
{
header ("http://www.xxxxxxxxxx.com/previewsave.php");
echo "<h1>Image Uploaded Successfully!</h1>";
echo '<img src="' . $newname . '" width="150" border="0"><br />'; 
} 
?>

Link to comment
Share on other sites

  • 2 years later...
This thread is more than a year old. Please don't revive it unless you have something important to add.

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Restore formatting

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...

Important Information

We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.