Excluding wysigyg selectors from ajax_html_ids list adds all other ids

This closed wysiwyg issue has introduced a pre-serialize callback/hook which tries to filter the ajax_html_ids list from wysiwyg selectors in order to avoid submitting unnecessary post parameters and avoid hitting server limit.

However, the code implementing the filtering assumes that all html element ids are included in the ajax_html_ids list and re-generates it with all html element ids, excluding the wysiwyg ones.

However, there are cases where some or all html element ids have already been filtered by the ajax_html_ids list, eg. by views exposed filters ajax. And this code re-adds them which may result in a HTTP 414 “Request URI too long” error.

My proposed solution would be sth like the following:

wysiwyg.js:
From:

    if (excludeSelectors.length > 0) {
      options.data['ajax_html_ids[]'] = [];
      $('[id]:not(' + excludeSelectors.join(',') + ')').each(function () {
      options.data['ajax_html_ids[]'].push(this.id);
      });
    }

To:

    if (excludeSelectors.length > 0) {
      var ajaxHtmlIdsArray = options.data['ajax_html_ids[]'];
      if (ajaxHtmlIdsArray === undefined || ajaxHtmlIdsArray.length == 0) {
		  return;
      }
      options.data['ajax_html_ids[]'] = [];
      $('[id]:not(' + excludeSelectors.join(',') + ')').each(function () {
		  if ($.inArray(this.id, ajaxHtmlIdsArray) != -1) {
			options.data['ajax_html_ids[]'].push(this.id);
		  }
      });

I do now know how to write module patches, please feel free to write one if my suggestion is right.