PHP file upload/move doesn't work

Question

I'm trying to move/copy/upload an image from tmp directory to another directory using following code, but I don't see my image being moved/copied to the destination. Is there any alternative?

    public function uploadToS3($file, $tmp_url) {
    if (strrpos($file, '.tmp') == strlen($file)-4) {
                $file = substr($file, 0, strlen($file)-4);
            }
        $destination = "var/www/html/image" . $file;

    if (file_exists($destination)) {
        echo 'File '. $destination. ' already exists!';
    } else {
    move_uploaded_file($tmp_url, $destination);
    }
}
    //Ex: $tmp_url = http://localhost/mage/media/tmp/catalog/product/s/c/abc.png
    //Ex: $destination = /var/www/html/image/s/c/abc.png

Following comments below I've tried following php code which fails too

 <?php
$tmp_url = "var/www/html/abc/abc.png";
$destination = "var/www/html/des/abc.png";
 if(move_uploaded_file($tmp_url, $destination)){echo "success";}else{echo "fail";}

Perhaps you're missing the starting / in $destination = "var/www/html/image" . $file; — pvg
– pvg, Commented Dec 1, 2015 at 9:06
do if(move_uploaded_file($tmp_url, $destination)){echo "success";}else{echo "fail";} and post the output...done — William Madede
– William Madede, Commented Dec 1, 2015 at 9:06
Update $destination = "var/www/html/image" . $file; to $destination = "var/www/html/image/" . $file; — William Madede
– William Madede, Commented Dec 1, 2015 at 9:11

Nana Partykar · Accepted Answer · 2015-12-01 09:37:23Z

1

Also please check the destination image folder has 777 / 775 permission

$ImageName = $_FILES['file']['name'];
$fileElementName = 'file';
$path = 'var/www/html/image/'; 
$location = $path . $_FILES['file']['name']; 
move_uploaded_file($_FILES['file']['tmp_name'], $location);

edited Dec 1, 2015 at 9:37

Nana Partykar

10.5k10 gold badges50 silver badges79 bronze badges

answered Dec 1, 2015 at 9:08

H.Dave

211 bronze badge

Sign up to request clarification or add additional context in comments.

Comments

score 0 · Accepted Answer · 2016-07-29 13:26:27Z

When it comes to uploading files it's always a good idea to construct the destination in a full path manner - why not create a global helper that gives you access to your base path:

/**
 * Return the base path.
 *
 * @param void
 * @return string
 */
function base_path()
{
    // I go back 2 directories here - it all depends where
    // you decide to place this global helper function.
    return realpath(__DIR__ . '/../..');
}

Great, now this is available globally, you may decide to use it like so:

$destination = base_path() . '/var/www/html/image';

This way, you don't have to worry about your current location - it's just too easy.

Collectives™ on Stack Overflow

PHP file upload/move doesn't work

2 Answers 2

Comments

Comments

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Related