Warning: imagejpeg(): Unable to open -ERROR MESSAGE

whobutsb

Hey All I keep getting this error message, no matter how many times I try reformating it. I know that imagejpeg() needs to be pointing to a file in a directory. But how do I point it to a temp file I just submitted to the database? Do I need to add a copy() command in front of it in order to get it to work? Thanks for all your help!!!

<?php
if (isset($HTTP_POST_VARS["Submit"])) 
		{
$title = strip_tags(trim($_POST['title']));
$teaser = strip_tags(trim($_POST['teaser']));
$content = strip_tags(trim($_POST['content']));
$date = $_POST['date'];
//upload directory.
//change to fit your need eg. files, upload .... etc.
$upload_dir = "_images/right_thing/";
//number of files to upload.
$num_files = 1;
//the file size in bytes.
$size_bytes =151200; //51200 bytes = 50KB.
//Extensions you want files uploaded limited to.
$limitedext = array("jpg","jpeg");

define ("FILE_DIR", "_images/right_thing/");
if (is_uploaded_file($_FILES['file']['tmp_name'])) {
	if ($_FILES['imageupload']['type'] == "image/gif" OR $_FILES['imageupload']['type'] == "image/pjpeg" OR $_FILES['imageupload']['type'] == "image/jpeg") {
		echo "Pictures Must Be in .JPG Format!";
		}
	else {
$name = $_POST['title'];
//Resize the image
$filedir = "/_images/right_thing/";
$uploadedfile = $_FILES['file']['tmp_name'];
$src = imagecreatefromjpeg($uploadedfile);
list($width, $height)= getimagesize($uploadedfile);
$newwidth= 100;
$newheight=($height/$width)*100;
$tmp = imagecreatetruecolor($newwidth, $newheight);
imagecopyresampled($tmp, $src, 0,0,0,0, $newwidth, $newheight, $width, $height);
$filename = $_FILES['file']['name'];
$new_image = imagejpeg($tmp, $name, 100);

imagedestroy($src);
imagedestroy($tmp);

$result = move_uploaded_file ($new_image,FILE_DIR."/$name.jpg");

bradgrafelman

You didn't paste the full error message... is it saying there is no file, it doesn't have permission, etc. ? Paste the error message so we can see exactly what the problem is.

In the mean time, I see that you don't ensure that the file was even uploaded correctly. You might want to do this before you try working with the file; check out [man]features.file-upload.errors[/man] in the manual for more information about the different error codes.

whobutsb

Sorry for not posting the error message, here it is:

Warning: imagejpeg(): Unable to open 'Mac' for writing in /usr/home/newporth/public_html/Beacon/New_Site/admin.php on line 95

I have given full privileges to my folder which I want to save it to, which is /_images/right_thing/.

bradgrafelman

whobutsb wrote:
I have given full privileges to my folder which I want to save it to, which is /images/right_thing/.

Then why don't you save it there? As it is now, you're trying to save the file as "Mac" (without any file extension...) - or whatever $POST['title'] is set to - in the current directory of the script. Try saving it in this folder instead (which you say the script has write access to).

Weedpacket

...whereupon the modified file will be promptly overwritten by the one that was originally uploaded.

Short answer is that you're mistaken about what [man]imagejpeg[/man] returns, and what its parameters are for.

whobutsb

Thanks for the help. I actually do have it set to add the .jpg extension to the filename when the file is moved. So If I move the file to the directory it will change the picture dimensions as well?

whobutsb

Man this is really boggling my mind. 😕 😕
I changed the script a tad and tried some different variables but still nothing. All I'm getting now is a serious of odd characters when I try to upload the picture. Here is a sample of the characters: "ÿØÿà�JFIF������ÿþ�<¶wVqÍqj—Py“Ù*...."

Here is the script I'm trying now. Where am I going wrong?

define ("FILE_DIR", "_images/right_thing/");
if (is_uploaded_file($_FILES['file']['tmp_name'])) {
	if ($_FILES['imageupload']['type'] == "image/gif" OR $_FILES['imageupload']['type'] == "image/pjpeg" OR $_FILES['imageupload']['type'] == "image/jpeg") {
		echo "Pictures Must Be in .JPG Format!";
		}
	else {
$newname = $_POST['title'];

//Resize the image
$filedir = "/_images/right_thing/";
$uploadedfile = $_FILES['file']['tmp_name'];
$src = imagecreatefromjpeg($uploadedfile);
list($width, $height)= getimagesize($uploadedfile);
$newwidth= 100;
$newheight=($height/$width)*100;
$tmp = imagecreatetruecolor($newwidth, $newheight);
imagecopyresampled($tmp, $src, 0,0,0,0, $newwidth, $newheight, $width, $height);
$filename = $_FILES['file']['name'];
imagejpeg($tmp, $newphoto, 100);



$result = move_uploaded_file ($newphoto,FILE_DIR."/$newname.jpg") or die("error uploading photo");

imagedestroy($src);
imagedestroy($tmp);

//Insert Information Into Database
if ($result == 1) {
$query = "INSERT INTO Right_thing (date, title, teaser, content, image, status, submitted_by, email) VALUES ('$date', '$title', '$teaser', '$content', '$title', 'Pending', $submitted_by, $email)";
$result = mysql_query($query) or die ("couldn't add entry");
 echo "File Successfuly Uploaded.";
 }
	else {
echo "There was problem uploading the file";

}

I really appreciate everyones help. I'm very new to PHP and I couldn't get through it with out these forums. I think I would've stopped a long time ago without them.

sid

could you please do us all a favor and

1.) break that character salad into some shorter strings to make that thread readable again on < 30" monitors - or just post the first 50 or so characters, since i dont think that anyone will be able to unconvert this anyways. it sure looks like some binary image code, when the headers are sent incorrectly.
2.) post the complete code (or the full snippet).
there are at least 2 brackets missing, as you have it now

define ("FILE_DIR", "_images/right_thing/");

if (is_uploaded_file($_FILES['file']['tmp_name'])) {
    if ($_FILES['imageupload']['type'] == "image/gif" OR     

            $_FILES['imageupload']['type'] == "image/pjpeg" OR 
            $_FILES['imageupload']['type'] == "image/jpeg") {
        echo "Pictures Must Be in .JPG Format!";
    }
    else {
        $newname = $_POST['title'];

    //Resize the image
    $filedir = "/_images/right_thing/";
    $uploadedfile = $_FILES['file']['tmp_name'];
    $src = imagecreatefromjpeg($uploadedfile);
    list($width, $height)= getimagesize($uploadedfile);
    $newwidth= 100;
    $newheight=($height/$width)*100;
    $tmp = imagecreatetruecolor($newwidth, $newheight);
    imagecopyresampled($tmp, $src, 0,0,0,0, $newwidth, $newheight, $width, $height);
    $filename = $_FILES['file']['name'];
    imagejpeg($tmp, $newphoto, 100);

    $result = move_uploaded_file ($newphoto,FILE_DIR."/$newname.jpg") or 
        die("error uploading photo");

    imagedestroy($src);
    imagedestroy($tmp);

    //Insert Information Into Database
    if ($result == 1) {
        $query = "INSERT INTO Right_thing (
            date, 
            title, 
            teaser, 
            content, 
            image, 
            status, 
            submitted_by, 
            email
        ) 
        VALUES (
            '$date', 
            '$title', 
            '$teaser', 
            '$content', 
            '$title', 
            'Pending', 
            $submitted_by, 
            $email
        );";
        $result = mysql_query($query) or die ("couldn't add entry");
        echo "File Successfuly Uploaded.";
    }
    else {
        echo "There was problem uploading the file";
    }

then, i somewhat fail to see what you are trying to do here:

if ($_FILES['imageupload']['type'] == "image/gif" OR     

            $_FILES['imageupload']['type'] == "image/pjpeg" OR 
            $_FILES['imageupload']['type'] == "image/jpeg") {
        echo "Pictures Must Be in .JPG Format!";
    }

it just seems like there is a quirk in the logic of this statement.

Weedpacket

sid wrote:
1) break that character salad into some shorter strings

I'll do that; it's enough to observe that ÿØÿà�JFIF is enough to identify this as a naked JPEG stream.

it just seems like there is a quirk in the logic of this statement.

I'd use [man]getimagesize[/man] to check this myself, instead of trusting the client to get it right.

As for why the whole thing is going wrong; my previous comment still stands. [man]imagejpeg[/man] does not return a jpeg image or a file name.