jQuery .post params not sending

Question

I'm having an issue that I have been trying to research and fix for hours now and I am at my wits-end in not understanding why it's not working correctly.

I have a jQuery dialog (via jQuery UI) pop-up when someone wants to delete an item (by clicking on a delete icon image and calling a JS function with a supplied "id" integer). The pop-up dialog is working properly and comes up with all the available information and button options. Here's the code for that:

function removeItem(id) {
    var rid = id;
    $("#dialog").dialog({
        title: "Removal Confirmation",
        resizable: false,
        height: 200,
        modal: true,
        show: {
            effect: "blind",
            duration: 800
        },
        buttons: {
            "Confirm": function() {
                $.post("includes/inc_details.php", {
                    "type":"remove",
                    "id":rid
                });
                window.location.reload();
                $(this).dialog("close");
            },
            Cancel: function() {
                $(this).dialog("close");
            }
        }
    });
}

When the "Confirm" button within the dialog is clicked, it's suppose to .post to a php file for further processing (code for that portion is below), and when complete will reload the browser window and then close the dialog.

However, what is routinely happening is the page just looks like it's reloading and then the dialog closes but the item doesn't get deleted. I've watch the processing via chromeDev tools and it appears that when the .post php file is called the two parameters are not sent with it. (I've used similar .posts other places - albeit without a jQuery UI dialog box - with no issues).

HELP!

PHP code being called via the .post method:

 $sql = "select * from shopping_cart WHERE id=$_POST[id] LIMIT 1";
            $result = mysql_query($sql);
            $row = mysql_fetch_assoc($result);

            if ($row["Type"] == "CouponUsed") {
                $sql_delete_free_item = "delete from shopping_cart where ProductID=".$_SESSION["freeProductId"]." and SessionID='".session_id()."'";
                $query = mysql_query($sql_delete_free_item);
                unset($_SESSION["freeProductId"]);
            }
$sql_id = "SELECT ProductID FROM shopping_cart WHERE id=$_POST[id] LIMIT 1";
    $result_id = mysql_query($sql_id);
    $row_id = mysql_fetch_assoc($result_id);
    $sql_remove = "DELETE FROM shopping_cart WHERE id=$_POST[id] LIMIT 1";
if (mysql_query($sql_remove)) {
        $sql_remove2 = "DELETE FROM shopping_cart_single WHERE singleid=$_POST[id]";
        mysql_query($sql_remove2);
    }
mysql_close($conn);
    exit();

Thank you, in advance!

**Danger**: You are using [an **obsolete** database API](http://stackoverflow.com/q/12859942/19068) and should use a [modern replacement](http://php.net/manual/en/mysqlinfo.api.choosing.php). You are **vulnerable to [SQL injection attacks](http://bobby-tables.com/)** that a modern API would make it easier to [defend](http://stackoverflow.com/questions/60174/best-way-to-prevent-sql-injection-in-php) yourself from. — Quentin, Sep 24 '15 at 20:23

Katana314 · Accepted Answer · 2015-09-24T20:25:56.100

Just including the relevant part.

"Confirm": function() {
            $.post("includes/inc_details.php", {
                "type":"remove",
                "id":rid
            }).then(function() {
                window.location.reload();
                $(this).dialog("close");
            });
        },

Remember what the A in AJAX stands for - asynchronous!

EDIT: To explain what was going on, here's how the browser reacts.

POST request queued up for submission. Asynchronous, so not immediate. Requests take much longer than other user actions, so it won't run it and block the UI thread.
Window told to reload. The browser cancels all pending requests - iframes, images, ajax requests, and begins loading a new page.
(If it hadn't reloaded, then once the javascript stack ended, it would actually send the request)

Nevertheless the request is sent to php file, so item should be deleted. — u_mulder, Sep 24 '15 at 20:22

score 1 · Answer 2 · answered Sep 24 '15 at 20:37

1

select * from shopping_cart WHERE id='".$_POST[id]."' LIMIT 1

Your problem is that you didn't add double quote and dot around the $_POST[id],

and also your $sql_remove, please double-check all your $_POST[id] is '".$_POST[id]."'

answered Sep 24 '15 at 20:37

zif

51
6

score 0 · Answer 3 · answered Sep 24 '15 at 20:22

When the "Confirm" button within the dialog is clicked, it's suppose to .post to a php file for further processing (code for that portion is below), and when complete will reload the browser window and then close the dialog.

Your call to reload() is after the $.post() call. So you tell the browser to make the POST request, then immediately reload the page. Since you are reloading the page before the POST request is sent, you cancel the POST request.

You need to move the call to reload() into a callback function that you pass to $.post().

Better still: Don't use Ajax. Just submit a regular form and have the server side handler redirect to the URL you want to reload.

jQuery .post params not sending

3 Answers3