PHP Delete file

Question

I am trying to develop a user page for a forum and I'm kinda struggling with the image upload. The problem is that I would like to limit the user to only be able to upload one single image, but be able to change it anytime. so basically, I would like to either overwrite the existing file either delete the old picture and add a new one instead. At this point I have a piece of code that adds a timestamp at the end of the file (which I don't really need actually).

CODE:

if(isset($_POST['upload']))
{

$extension=strstr($_FILES['uploadedfile']['name'], ".");
    $filename = "_/userfiles/userpics/".basename($_FILES['uploadedfile']['name'], 
$extension);
    $target = "_/userfiles/userpics/".basename($_FILES['uploadedfile']['name']);

$valid = true;  
if(file_exists($target))
{

    $filename = $filename . time();
    $target = $filename . $extension;
}
if($valid)
{
    // move the file into the folder of our choise
    move_uploaded_file($_FILES['uploadedfile']['tmp_name'], $target);       
    $img_sql = "INSERT INTO sp_userimage (imageid, path, id) value ('', '".$target."', '".$_SESSION['userid']."')";
    $img_result = mysql_query($img_sql);
    echo "upload sucessfull";
}

So you want to delete a file? Search for unlink.

putvande
– putvande

2013-09-12 10:18:50 +00:00
Commented Sep 12, 2013 at 10:18 — putvande
– putvande, Commented Sep 12, 2013 at 10:18

Shankar Narayana Damodaran · Accepted Answer · 2013-09-12 10:18:41Z

2

Make use of unlink() in PHP Manual.

if(file_exists($target))
{
    unlink($target); // deletes file
    //$filename = $filename . time();
    //$target = $filename . $extension;
}

answered Sep 12, 2013 at 10:18

Shankar Narayana Damodaran

68.6k43 gold badges102 silver badges129 bronze badges

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

3 Comments

Paraschiv Ionut Over a year ago

doesnt that only search for the filename? the new file could have a different name tho..

Paraschiv Ionut Over a year ago

for example, if the first picture is "cat.jpg" and the second is "cat02.jpg" it wont delete anything, because that filename does not exist already.

Shankar Narayana Damodaran Over a year ago

Whatever filename you pass inside the unlink gets deleted. Idea will be unlink your old file and upload the one the user has currently uploaded.

Eoin Murphy · Accepted Answer · 2013-09-12 10:27:39Z

0

I think this might be a bit better suited for you. You might have to edit it a tad.

if($valid)
{
    // Check if user has a file.

    $img_check = mysql_query("SELECT * FROM sp_userimage WHERE id = " . (int) $_SESION['user_id']);
    if( mysql_num_rows($img_check) > 0 ){
       $row = mysql_fetch_object($img_check);

       // Delete the file.
       unlink($row->path);
    }

// move the file into the folder of our choise
move_uploaded_file($_FILES['uploadedfile']['tmp_name'], $target);       
$img_sql = "INSERT INTO sp_userimage (imageid, path, id) value ('', '".$target."', '".$_SESSION['userid']."')";
$img_result = mysql_query($img_sql);
echo "upload sucessfull";

}

answered Sep 12, 2013 at 10:27

Eoin Murphy

8136 silver badges9 bronze badges

Comments

cbednarski · Accepted Answer · 2013-09-12 10:25:21Z

0

It might be easier to normalize the image type (e.g. only jpegs) and then name the file as the userid. For example:

$target = 'userpics' . DIRECTORY_SEPARATOR . $_SESSION['userid'];
move_uploaded_file($_FILES['uploadedfile']['tmp_name'], $target);

This will simply overwrite the old picture with the new one. Given that this type of filename is deterministic, you also don't need to store the filename in the database.

answered Sep 12, 2013 at 10:25

cbednarski

11.8k4 gold badges29 silver badges34 bronze badges

Comments

Aditya · Accepted Answer · 2013-09-12 10:28:31Z

0

Use unlink() function

Comments

Dinesh · Accepted Answer · 2013-09-12 10:34:36Z

0

Its very simple by unlink()

as:

unlink(dirname(__FILE__) . "/../../public_files/" . $filename);

answered Sep 12, 2013 at 10:34

Dinesh

4471 gold badge6 silver badges22 bronze badges

Comments

DS9 · Accepted Answer · 2013-09-12 10:38:05Z

0

if (file_exists($path))
{
  $filename= rand(1,99).$filename; 
  unlink($oldfile);
}
move_uploaded_file($_FILES['file']['tmp_name'],$filename);

answered Sep 12, 2013 at 10:38

DS9

3,0436 gold badges61 silver badges106 bronze badges

Collectives™ on Stack Overflow

PHP Delete file

6 Answers 6

3 Comments

Comments

Comments

Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

6 Answers 6

3 Comments

Comments

Comments

Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related