Allow body for remote link with method POST

[emerleite]

Allow link_to with remote => true for method POST with request body. I needed this in my project for a link inside another form, wich I could not insert a form inside another form. Should be usefull for other people.

Example: <%= link_to "something", path, :remote => true, :method => 'post', "data-body" => "key1=val1&key2=val2" %>

[emerleite]

Forgot a comment in src/rails.js. Please remove when integrate.

Cheers

[neerajsingh0101]

Nice work. And thanks for the tests.

I have two comments.

1. Delete the commented line.

2. Change element.attr('data-body') to element.attr('data). I find the name 'data-body' highly confusing.

[JangoSteve]

The data-body attribute name isn't terribly descriptive. What do you guys think of either data-post or data-params?

[emerleite]

I've used data-body because it is actualy de body for a POST or PUT request. data-params can be ok too, but I still prefer data-body, because it's more glued to the real purpose, that is send a request body. What do you think? Any other ideas?

[JangoSteve]

I agree with that part. However, "body" only makes sense in the context that it's for a POST request. But that won't always be the case.

In the more generic context that this is an attribute for a data-remote link, I was thinking we should use a more generic, descriptive attribute. For example, notice that jQuery calls the attribute data, not body. By their definition:

data: Data to be sent to the server. It is converted to a query string, if not already a string. It's appended to the url for GET-requests. See processData option to prevent this automatic processing. Object must be Key/Value pairs. If value is an Array, jQuery serializes multiple values with same key based on the value of the traditional setting (described below).

To me, hands down, it should be called data, to align with jQuery. However, telling people to set the data-data attribute on their remote link would be weird and confusing. Which is why I'm leaning toward data-params; params in the Rails world is more or less synonymous with data in the jQuery world.

I think data-params is more glued to the actual functionality, since the purpose changes depending on the context. But I'm open to other ideas.

[emerleite]

Changed to data-params. Really makes more sense.

[JangoSteve]

Awesome, Emerson, thanks for your patience and work on this new feature :-) Especially with the test coverage, much appreciated. I squashed your commits into 1 and pulled them in. dad6982

[emerleite]

;)