app/assets/javascripts/jqr-helpers.js in jqr-helpers-1.0.57 vs app/assets/javascripts/jqr-helpers.js in jqr-helpers-1.0.58
- old
+ new
@@ -180,10 +180,20 @@
// if the button is inside a form, allowAction is already called.
if ($(this).closest('form').length || $.rails.allowAction(element)) {
element.data('confirm', null); // we've already fired it
// largely copied from rails_jquery.js
var href = element.data('url');
+ if (element.data('params')) {
+ $.each(element.data('params'), function(name, value) {
+ if (href.indexOf('?') > 0) {
+ href += '&' + name + '=' + encodeURIComponent(value);
+ }
+ else {
+ href += '?' + name + '=' + encodeURIComponent(value);
+ }
+ });
+ }
var method = element.data('method');
var csrf_token = $('meta[name=csrf-token]').attr('content');
var csrf_param = $('meta[name=csrf-param]').attr('content');
var form = $('<form method="post" action="' + href + '"></form>');
var metadata_input =
@@ -193,15 +203,9 @@
metadata_input += '<input name="' + csrf_param + '" value="' +
csrf_token + '" type="hidden" />';
}
form.hide().append(metadata_input).appendTo('body');
- if ($(element).data('params')) {
- $.each($(element).data('params'), function(name, value) {
- var input = $j('<input>', { 'name': name, 'value': value});
- form.append(input);
- });
- }
$(form).data(element.data()); // copy to form
$(form).data('remote', true);
$(form).addClass('ujs-ajax');
$(form).data('real-element', element.attr('id'));
form.submit();