What's wrong with this simple jquery get?

Question

I have code that looks something like this:

<html>
<head>
    <script type="text/javascript" src="http://code.jquery.com/jquery-latest.min.js"></script>
    <script type="text/javascript">
        function myButton_click(){
            alert('got here');
            $.get("myPage.cfm", function(data){alert('load was performed')});
        }
    </script>
</head>
<body>
    <input type="button" id="myButton" value="My Button" onClick="myButton_click()" />
</body>
</head>

The first alert ("got here") shows up, but the "load was performed" doesn't. Is the get being executed? What am I doing wrong?

Edit:

I've tried it with "http://www.google.com" instead of "myPage.cfm" to make sure it's not a problem with my page and it's still not alerting...

Is "myPage.cfm" in the same DIR as this page? You aren't specifying a location, so I am assuming so. — MattW, Nov 14 '11 at 16:41
@Boldewyn - yup, I use Firebug, but never used the network panel... hold on while I activate it :) — froadie, Nov 14 '11 at 16:45
@MattW - yup, I meant to call it "myRelativePath" or something so that would be clear — froadie, Nov 14 '11 at 16:46
Ok, the network panel seems to show that it's being loaded - displaying a "302 Found" (but I've never used it before so may not know what I'm doing) — froadie, Nov 14 '11 at 16:47
Means it's getting redirected in which case the success callback is never called. — aziz punjani, Nov 14 '11 at 16:48
@Boldewyn - thanks a LOT! This network tab is amazing! I think I should take a firebug tutorial :) I discover new things about it every day — froadie, Nov 14 '11 at 17:12
@froadie: You're welcome! There are also great extensions, if you miss a functionality: http://getfirebug.com/wiki/index.php/Firebug_Extensions — Boldewyn, Nov 15 '11 at 07:31
@Interstellar_Coder: 302's are followed, and `success` will be fired eventually. Only 4** and 5** HTTP states won't fire it. — Boldewyn, Nov 15 '11 at 07:34

score 1 · Accepted Answer · answered Nov 14 '11 at 16:52

1

According to the docs, the function you pass as the second parameter to the $.get call is only called on success. So if your server is returning any kind of error (400, 403, 404, 500 etc.), that function will not be called. If you want a function to be called no matter what, use complete:

$.get("myPage.cfm").complete(function(jqXHR, textStatus) {
  alert("Load was performed");
});

answered Nov 14 '11 at 16:52

Felix

88,392
43
149
167

ok, got it. using the network tab (thanks boldewyn :) ), I can see that the page is returning an error. – froadie Nov 14 '11 at 16:58

score 0 · Answer 2 · edited May 23 '17 at 12:03

0

The callback only executes on success (Http status code in the 200s or 304), try using the longhand jQuery.ajax method and following the advice in this SO question for how to handle this response in a reasonable manner.

edited May 23 '17 at 12:03

Community

1
1

answered Nov 14 '11 at 16:50

Rich O'Kelly

41,274
9
83
114

2

Actually that's slightly incomplete, jQuery considers these status codes "successful" `status >= 200 && status < 300 || status === 304` – aziz punjani Nov 14 '11 at 16:54

What's wrong with this simple jquery get?

2 Answers2