Mike
Mike

Reputation: 1336

Ruby JSON.parse gives unexpected token error

I'm sending a POST with jQuery like this

   var authorisationType = "payment";

   var paymentData = {
        cardholdername: $("#userCardName").val(),
        cardnumber: $("#userCardNumber").val(),
        expmonth: $("#userExpMonth").val(),
        expyear: $("#userExpYear").val(),
        cvv:$("#userCVV").val(),
        amount: "$15.00"
    }

    var postData = new Object();
    postData.username = username;
    postData.type = authorisationType;
    postData.data = paymentData;

    var authorisationType = "payment";

    console.log("sending username and token to server");
    $.ajax({
       type: 'POST',
       url: url,
       data: JSON.stringify(postData), 
       contentType: "application/json; charset=utf-8",
       dataType: "json",
       success: successHandler,
       error: errorHandler
    });  

to my Rails controller:

def authorisation_request
    puts "preparse" 
    hash = JSON.parse(params[:username])
    puts "hash" 
    puts "username:" + JSON.parse(params[:username]).to_s + " type:" +  JSON.parse(params[:type]).to_s + " data:" + JSON.parse(params[:data]).to_s
end

And I get the following error in the Console:

 Parameters: {"username"=>"[email protected]", "type"=>"payment", "data"=>{"cardholdername"=>"me myself", "cardnumber"=>"xxxxxxxxxxxx3456", "expmonth"=>"01", "expyear"=>"2015", "cvv"=>"999", "
amount"=>"$15.00"}, "session"=>{"username"=>"[email protected]", "type"=>"payment", "data"=>{"car
dholdername"=>"me myself", "cardnumber"=>"xxxxxxxxxxxx3456", "expmonth"=>"01", "expyear"=>"2015"
, "cvv"=>"999", "amount"=>"$15.00"}}}

preparse
Completed 500 Internal Server Error in 4ms

JSON::ParserError (757: unexpected token at '[email protected]'):
  app/controllers/sessions_controller.rb:119:in `authorisation_request'

What am I doing wrong?

Upvotes: 0

Views: 1195

Answers (2)

Surya
Surya

Reputation: 16002

It'd be much cleaner and easier for you if you post data from your jQuery like this:

var postData = new Object();
postData.userdata = { 'username': username, 
                      'type': authorisationType,
                      'data': paymentData };

Now, in controller, you can:

def authorisation_request
  puts params[:userdata]
end

No need of JSON.parse as all values are available in parameter as key-value pair using params.

Upvotes: 3

katafrakt
katafrakt

Reputation: 2488

params[:username] contains string ([email protected]) which is not a valid JSON. You don't need to call JSON.parse on this.

Upvotes: 4

Related Questions