Skip to content
Snippets Groups Projects

Apply patch from comment #34

Open Pierre Rudloff requested to merge issue/drupal-3121172:3121172- into 11.x
2 unresolved threads

Closes #3121172

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Stephen Mustgrave
  • Stephen Mustgrave
  • Stephen Mustgrave
  • Stephen Mustgrave resolved all threads

    resolved all threads

  • andrew farquharson added 641 commits

    added 641 commits

    Compare with previous version

  • 100 100 this.settings = settings;
    101 101
    102 102 // Add the ajax to exposed forms.
    103 this.$exposed_form = $(
    103 this.$exposedForm = $(
  • 167 * @param {string} url
    168 * The AJAX URL.
    169 * @param {jQuery} $exposedForm
    170 * The exposed form element.
    171 *
    172 * @return {string}
    173 * The modified AJAX URL without the query parameters existing in the exposed form.
    174 */
    175 Drupal.views.ajaxView.prototype.removeExposedFiltersQueryParameters =
    176 function (url, $exposedForm) {
    177 const urlObj = new URL(url, window.location.origin);
    178 // Use copy to keep the iterator consistent.
    179 const urlObjCopy = new URL(url, window.location.origin);
    180 const cleanKeys = {};
    181 // Operate with element names to handle empty values.
    182 $exposedForm.find('[name]').each((i, el) => {
  • Théodore Biadala left review comments

    left review comments

  • Please register or sign in to reply
    Loading