jQuery AJAX POST executes twice

Question

My AJAX call always executes twice. I'm using normal Javascript Event Handlers and the <input type="button">. I'm not quite sure where I'm going wrong, the server returns a JSON array which AJAX can receive successfully and parse. I'm quite sure it is my AJAX because when I ran the code the alert popped up twice. alert("ran"). The server does not return any redirect headers.

HTML

<form action="/" onsubmit="return false">
    <b>Enter your announcement here</b>
    <span class="dropdown"><img src="/index/img/p/help" style="width: 20px; height: 20px;"/>
        <span class='dropdown-content'>
            <b>tip:</b> try to make your announcement less than 100 words. Choose what you say wisely.
        </span>
    </span>
    <textarea class='form-control' id='announcetxtbox' style='resize: vertical'></textarea>
    <select id="announce-type">
        <option value="general">General</option>
        <option value="social">Social</option>
        <option value="world">World</option>
        <option value="maint">Site Maintenence</option>

    </select>
    <input type="button" id="announce-submit" class="btn btn-primary" value="Submit">

    <script>
        CKEDITOR.replace("announcetxtbox");
        CKEDITOR.config.toolbar = [
            {name: 'basicstyles', items: ['Bold', 'Italic', 'Underline', 'Strike', 'Subscript', 'Superscript', '-', 'RemoveFormat']},
        ];
    </script>

</form>

JAVASCRIPT

function _(str) {
    return d.getElementById(str);
}
function get_announcement_data(ele) {

    var type, answer = _("announce-msg");
    switch (ele) {
        case "general":
            type = "G";
            break;
        case "social":
            type = "S";
            break;
        case "world":
            type = "W";
            break;
        case "maint":
            type = "M";
            break;
        default:
            type = "G";
            break;
    }

    $.ajax({
        url: "/profile_scripts/announcements.php?u=get",
        type: "POST",
        data: {type: type, CSRF_TOKEN: _("CSRF_TOKEN_VALUE").value},
        success: function (data) {
            data = JSON.parse(data);
            answer.innerHTML = "";
            for (var i = 0; i < data.length; i++) {
                if (data[i].response == 1) {
                    answer.innerHTML += data[i].msg + "<small> Written By: " + data[i].author + " on " + data[i].date + "</small>"
                } else {
                    answer.innerHTML = data[i].msg;
                }
            }

        }
    })

}
function add_announcement() { //THIS FUNCTION RUNS TWICE!

    var announcement = CKEDITOR.instances.announcetxtbox.getData();
    var type = _("announce-type").value;
    var code;
    switch (type) {
        case "general":
            code = "G";
            break;
        case "social":
            code = "S";
            break;
        case "world":
            code = "W";
            break;
        case "maint":
            code = "M";
            break;
        default:
            code = "G";
            break;
    }

    $.ajax({
        url: "/profile_scripts/announcements.php?u=post",
        type: "POST",
        data: {type: code, CSRF_TOKEN: _("CSRF_TOKEN_VALUE").value, msg: announcement},
        success: function (data) {
            data = JSON.parse(data);
            if (data.response == 1) {
                alert("ran");
                get_announcement_data(type);

            } else {

                alert("Something went wrong!");

            }

        }
    });

}
d.addEventListener("DOMContentLoaded", function () {

    _("quick-actions-go").addEventListener("click", quick_action_search)

    _("manual-data-pull").addEventListener("click", pull_data);


    _("announce-submit").addEventListener("click", function (e) {

        add_announcement(); <--- Event Listener
        e.preventDefault();
        e.stopPropagation();
    });

    var tab_elements = d.querySelectorAll("[id=ann-tab]");

    for (var i = 0; i < tab_elements.length; i++) {
        tab_elements[i].addEventListener("click", function (e) {
            e.preventDefault();
            get_announcement_data(this.dataset.modeval)
        });
    }


});

looks like `announce-submit` click handler is triggered twice from your program.you can check the stack trace on your dev tools to see the source for it . — dreamweiver, Feb 22 '16 at 13:02

score 0 · Answer 1 · answered Feb 22 '16 at 06:11

0

When submitting form. The both ajax will call. You first get method and post method that makes the problem

answered Feb 22 '16 at 06:11

Pradeep Andrewson

77
1
10

$( "#announce-submit" ).click(function(e) { add_announcement(); <--- Event Listener e.preventDefault(); e.stopPropagation(); }); – Pradeep Andrewson Feb 22 '16 at 06:23

score 0 · Accepted Answer · answered Feb 22 '16 at 06:13

0

Here your form is submitting twice. You ca 1. Pass 'e' when you are calling fun add_announcement. Aad add line

e.preventDefault();

as first line in fun add_announcement(e).

Also, unbind submit actiom from form
```
$('form').unbind('submit').submit();
```

Hope it helps.

answered Feb 22 '16 at 06:13

khushboo29

906
6
16

but the `submit()` function refreshes the page. any way to work around that? – David Yue Feb 22 '16 at 06:23
Remove action attr from form tag. Try it – khushboo29 Feb 22 '16 at 06:26
Refre this - http://stackoverflow.com/questions/23507608/form-submission-without-page-refresh – khushboo29 Feb 22 '16 at 06:30

jQuery AJAX POST executes twice

2 Answers2