[Rt-commit] rt branch, 4.4/fix-ajax-edit-format, created. rt-4.4.3-200-g23a848df5

? sunnavy sunnavy at bestpractical.com
Sun Feb 24 16:33:26 EST 2019


The branch, 4.4/fix-ajax-edit-format has been created
        at  23a848df5ffcb7637e77dec6e52c55b98852b675 (commit)

- Log -----------------------------------------------------------------
commit 23a848df5ffcb7637e77dec6e52c55b98852b675
Author: sunnavy <sunnavy at bestpractical.com>
Date:   Mon Feb 25 04:59:50 2019 +0800

    Switch to "POST" when updating ticket search result format via AJAX
    
    Adding all the form fields to URL could exceed web server/browser URL
    length limit, so move it to request body instead.
    
    This commit is to fix the issue of "Failed to update format. Reason:
    error" because of the over-sized URL when updating section "Display
    Columns" in ticket search builder.

diff --git a/share/html/Search/Elements/EditFormat b/share/html/Search/Elements/EditFormat
index 49ebde13c..fbeccbdcd 100644
--- a/share/html/Search/Elements/EditFormat
+++ b/share/html/Search/Elements/EditFormat
@@ -61,7 +61,9 @@ jQuery( function() {
         var name = jQuery(this).attr('name');
         var form = jQuery(this).closest('form');
         jQuery.ajax({
-            url: '<% RT->Config->Get('WebPath') %>/Helpers/BuildFormatString?' + name + '=1&' + form.serialize(),
+            url: '<% RT->Config->Get('WebPath') %>/Helpers/BuildFormatString?' + name + '=1',
+            method: 'POST',
+            data: form.serialize(),
             success: function (data) {
                 if ( data.status == 'success' ) {
                     form.find('input[name=Format]').val(data.Format);

-----------------------------------------------------------------------


More information about the rt-commit mailing list