jQuery $.ajax and php to delete a file

Question

I'm trying to delete a file with jQuery's $.ajax and php. I have the following.

/js/whatever.js

$('.deleteimage').live('click', function() {
    $imagefile = 'http://domain.com/images/1/whatever.jpg';
    $imagethumb = 'http://domain.com/images/1/thumbnail/whatever.jpg';
    $.ajax({
        type: 'POST',
        data: {
            action: 'deleteimage',
            imagefile: $imagefile,
            imagethumb: $imagethumb,
        },
        url: 'script.php',
        success: function(msg) {
            alert(msg);
        }
    })
})

/php/script.php

<?php
    if($_GET["action"]=="deleteimage")
    {
        $imagefile = $_REQUEST['imagefile'];
        $imagethumb = $_REQUEST['imagethumb'];
        $imagefileend = '../images'.end(explode('images',$imagefile)); //This will get me the path to the image ../images/1/whatever.jpg without the domain which is the correct path to the file. I tried that path directly and it deleted the file. 
        $imagethumbend = '../images'.end(explode('images',$imagethumb));

        unlink($imagefileend);
        unlink($imagethumbend);
    }
?>

All path are correct. In firebug i see the post variables are being sent correctly to script.php, however files are not being deleted. What am i doing wrong.

check with if (file_exists($imagefileend)) { unlink($imagefileend); } else{echo "error";} — Wasim Karani
– Wasim Karani, Commented Oct 5, 2011 at 7:25
also, you're not echoing anything in php script, so you won't get the msg response in Ajax — Damien Pirsy
– Damien Pirsy, Commented Oct 5, 2011 at 7:28
I personally would post them all and use $_POST instead of $_GET and $_REQUEST in this condition. — Arda
– Arda, Commented Oct 5, 2011 at 7:29

matino · Accepted Answer · 2011-10-05 07:28:26Z

4

In jQuery you use POST but in PHP you use GET. Change to if($_POST["action"]=="deleteimage") and please don't use $_REQUEST but $_POST

answered Oct 5, 2011 at 7:28

matino

17.8k8 gold badges51 silver badges63 bronze badges

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

4 Comments

Pinkie Over a year ago

GREAT. That was exactly the problem. It's working now using POST. Do you know the reason why it wouldn't work with REQUEST.

Arda Over a year ago

This was what I said at the comment. This works because you are sending as POST. Also I suggest avoiding using $_REQUEST in anywhere in your script.

matino Over a year ago

@Pinkie - stackoverflow.com/questions/2142497/…

Pinkie Over a year ago

Yes i see why request is not a good idea. Thanks for the link.

Collectives™ on Stack Overflow

jQuery $.ajax and php to delete a file

1 Answer 1

4 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

4 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related