Probably obvious problem in JavaScript

Question

I think this is just an obvious problem that I can't spot because I've been staring at code too long today. I have this callback as part of a JQuery POST:

                        function(data){
                                if(data == 'yes') {
                                        $('div.done').show();
                                } else if(data != 'yes') {
                                        $('div.error').show();
                                        alert(data);
                                }
                        });

I get the error div shown, yet in the alert all I get is yes. Anyone got any pointers (as I said, I imagine it's some really obvious mistake that I've missed).

Thanks

are you sure that data has no whitespace at it's begin/end? try adding data = data.replace(/^\s+|\s+$/, ''); before if statement. — krcko
– krcko, Commented Dec 24, 2009 at 12:47
Good suggestions, just thought I'd pipe in with if he's using jQuery he might as well use $.trim(data) ;). — Jake
– Jake, Commented Dec 24, 2009 at 13:04
Thanks for the comments. The data.replace one didn't work, neither did $.trim(data). Just somebody, I can't seem to get that to work. What would I do, just put javascript:alert(data.toSource()) in the URL bar? — hrickards
– hrickards, Commented Dec 24, 2009 at 13:12
@hrickards: no, I meant you should augment the alert call in the code you posted. Object.toString is non-standard and I told you to run it in firefox which does have it. what exactly does "can't seem to get that to work" mean? — just somebody
– just somebody, Commented Dec 24, 2009 at 13:23

epascarello · Accepted Answer · 2009-12-24 12:55:11Z

4

Debuggng 101:

function (data) {
    alert(escape(data));
    if (data == 'yes') {
        $('div.done').show();
    } else if (data != 'yes') {
        $('div.error').show();
        alert(data);
    }
});

answered Dec 24, 2009 at 12:55

epascarello

208k20 gold badges206 silver badges246 bronze badges

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

7 Comments

hrickards Over a year ago

Thanks. That turned up yes%0a. Is that a line feed? Do you know how I could remove that, or even better stop PHP sending it? The only thing in my PHP file that sends anything is echo 'yes';.

epascarello Over a year ago

%0A is a line feed you can do a regular expression: data.replace(/\n+$/);

hrickards Over a year ago

Thanks. Any idea why that might be being sent?

hrickards Over a year ago

Hmm, for some reason I still have the same issue. alert(escape(data)); still shows yes%0a.

Casey Chu Over a year ago

The reason PHP is sending an extra line feed is probably because you have whitespace after your closing ?>. Delete the whitespace, or better yet, delete the entire closing tag (it's optional).

|

Collectives™ on Stack Overflow

Probably obvious problem in JavaScript

1 Answer 1

7 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

7 Comments

Your Answer

Sign up or log in

Post as a guest

Related