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();