What is wrong with this jQuery syntax?

Question

This code keeps throwing an "unexpected token' error, but I can't figure out what is wrong with it. Any clues would be great.

function addComment() {
    $.ajax({
        url:'/add/comment/id',
        type:'POST',
        data.JSON.stringify({'Text':$('#comment_text').val()}),
        contentType:'application/json; charset=utf8',
        processData:false,
        success:function(data){
            $('#comments').prepend(data.comment.Text);
        }
    });
}

data : JSON.stringify({'Text':$('#comment_text').val()}),

Clinton Ward
– Clinton Ward

2012-08-06 01:50:05 +00:00
Commented Aug 6, 2012 at 1:50 — Clinton Ward
– Clinton Ward, Commented Aug 6, 2012 at 1:50
oh man, that's embarrassing. Thanks.

MFB
– MFB

2012-08-06 01:51:12 +00:00
Commented Aug 6, 2012 at 1:51 — MFB
– MFB, Commented Aug 6, 2012 at 1:51

Peter Olson · Accepted Answer · 2012-08-06 01:56:02Z

5

The problem is on this line:

data.JSON.stringify({'Text':$('#comment_text').val()}),

The unexpected token is the period (.) after data.

Instead of a period (.), you need a colon (:) after data, like so:

data: JSON.stringify({'Text':$('#comment_text').val()}),

Note that you don't need JSON.stringify here. As @pst pointed out, jQuery does that for you anyways.

data: {
  'Text': $('#comment_text').val()
},

edited Aug 6, 2012 at 1:56

answered Aug 6, 2012 at 1:50

Peter Olson

144k49 gold badges209 silver badges250 bronze badges

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

Comments

Clinton Ward · Accepted Answer · 2012-08-06 01:56:05Z

3

data: JSON.stringify({'Text':$('#comment_text').val()}),

edited Aug 6, 2012 at 1:56

answered Aug 6, 2012 at 1:52

Clinton Ward

2,5211 gold badge23 silver badges26 bronze badges

Comments

Ram · Accepted Answer · 2012-08-06 01:51:17Z

2

data.JSON.stringify({'Text':$('#comment_text').val()})

should be:

data: JSON.stringify({'Text':$('#comment_text').val()})

edited Aug 6, 2012 at 1:51

Ram

145k16 gold badges174 silver badges201 bronze badges

answered Aug 6, 2012 at 1:50

Musa

97.9k17 gold badges123 silver badges144 bronze badges

6 Comments

Cameron Martin Over a year ago

@MFB data can be an object or string. api.jquery.com/jQuery.ajax Best to let jQuery do the JSONing, since not all browsers have native JSON.stringify support

Musa Over a year ago

@CameronMartin the object will be transformed into key=value pairs not JSON

Cameron Martin Over a year ago

@Musa oh yes of course, my bad. A url-encoded query string makes much more sense in this situation than JSON.

MFB Over a year ago

@CameronMartin JSON makes it easy for me because I can throw it straight at my database, but I'm looking for an easy way to convert the string instead.

Cameron Martin Over a year ago

What database are you using? CouchDb?

|

Collectives™ on Stack Overflow

What is wrong with this jQuery syntax?

3 Answers 3

Comments

Comments

6 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

Comments

Comments

6 Comments

Your Answer

Sign up or log in

Post as a guest

Related