Help me with jpg upload through HTTP Post with PHP!

[cyberock]

I'm trying to upload a jpg image through a form. I am very new to the linux anvironment and php. I was a recent coldfusion developer and always worked on windows so I have a lot to learn. I ran and test my code locally on my windows IIS. I moved it up to a linux server and now it doesn't work. I don't even get an error. It just doesn't upload anything. Here's the important parts of my code...

form page:
<form method="post" action="add_image.php" enctype="multipart/form-data">
Thumb Nail Image:
<input type="file" name="ThumbImg" size="30">
<input type="submit" name="Submit" value="Submit">
</form>

Upload Page: ( I don't know the direct path on the server so that's the reasoning for the "../.../")

$images_query = "SELECT Max(models_id) FROM models_main";
$images_result = mysql_query($images_query, $db_connection)or die(mysql_error());
$image_id = mysql_result($images_result, 0, 0);


$_FILES['ThumbImg']['name'];
if(file_exists($_FILES['ThumbImg']['name'])) {
rename('' . $_FILES['ThumbImg']['tmp_name'],'../../images/thumb/' . $image_id .'.jpg');


Thanks in adavance for the help!
}

 

[Entity]

Do you have write access in all the proper directories?

Also, your syntax for rename seems incorrect, and I'm not sure why you are renaming the image rather than copying it. Generally tmp_name is something like /tmp/randomimagename, and you'd want to do something like this:

$newfile = $dirname . "/" . $_FILES['photo']['name'];

if(!copy($_FILES['photo']['tmp_name'], $newfile))
{
echo "Could not copy files.";
exit;
}

If you continue having problems, check the _FILES array to make sure that it is properly being uploaded, etc.

Rob

 

[cyberock]

Thanks for the help! I finally got it to work by doing this but I feel I added in an extra step...

$newfile = $_FILES['ThumbImg']['name'];
copy($_FILES['ThumbImg']['tmp_name'], $newfile);
rename( $_FILES['ThumbImg']['name'],'../../images/thumb/' . $image_id .'.jpg');

 

[Entity]

Originally posted by: cyberock
Thanks for the help! I finally got it to work by doing this but I feel I added in an extra step...

$newfile = $_FILES['ThumbImg']['name'];
copy($_FILES['ThumbImg']['tmp_name'], $newfile);
rename( $_FILES['ThumbImg']['name'],'../../images/thumb/' . $image_id .'.jpg');

First of all, $_FILES['ThumbImg']['name'] should already have a jpg extension, if that is the file someone uploaded. It's just the name of the file that was uploaded. Either way, though, it doesn't look like you want the final jpg file to have that name, but instead a pre-assigned name (based on $image_id). So why not do this? Since I can't see all your code, I don't know where you get the $image_id variable from, but this should still work.

$newfile = "../../images/thumb/" . $image_id . ".jpg";
copy($_FILES['ThumbImg']['tmp_name'], $newfile);

Rob