Skip to content
Snippets Groups Projects
Verified Commit cd700017 authored by Théodore Biadala's avatar Théodore Biadala
Browse files

Issue #3238870 by shubh_, mstrelan, nod_, hooroomoo, kostyashupenko,...

Issue #3238870 by shubh_, mstrelan, nod_, hooroomoo, kostyashupenko, smustgrave, larowlan, thebumik, lauriii: Refactor (if feasible) use of jquery parseHTML function to use vanillaJS

(cherry picked from commit b4a02610)
parent 8fedc4e2
No related branches found
No related tags found
31 merge requests!10663Issue #3495778: Update phpdoc in FileSaveHtaccessLoggingTest,!10451Issue #3472458 by watergate, smustgrave: CKEditor 5 show blocks label is not translated,!103032838547 Fix punctuation rules for inline label suffix colon with CSS only,!10150Issue #3467294 by quietone, nod_, smustgrave, catch, longwave: Change string...,!10130Resolve #3480321 "Second level menu",!9936Issue #3483087: Check the module:// prefix in the translation server path and replace it with the actual module path,!9933Issue #3394728 by ankondrat4: Undefined array key "#prefix" and deprecated function: explode() in Drupal\file\Element\ManagedFile::uploadAjaxCallback(),!9914Issue #3451136 by quietone, gapple, ghost of drupal past: Improve...,!9882Draft: Issue #3481777 In bulk_form ensure the triggering element is the bulk_form button,!9839Issue #3445469 by pooja_sharma, smustgrave: Add additional test coverage for...,!9815Issue #3480025: There is no way to remove entity cache items,!9757Issue #3478869 Add "All" or overview links to parent links,!9752Issue #3439910 by pooja_sharma, vensires: Fix Toolbar tests that rely on UID1's super user behavior,!9749Issue #3439910 by pooja_sharma, vensires: Fix Toolbar tests that rely on UID1's super user behavior,!9678Issue #3465132 by catch, Spokje, nod_: Show test run time by class in run-tests.sh output,!9578Issue #3304746 by scott_euser, casey, smustgrave: BigPipe cannot handle (GET)...,!9449Issue #3344041: Allow textarea widgets to be used for text (formatted) fields,!8945🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥🔥...,!8893Resolve #3444391 "Navigation center sm logo",!8772Issue #3445909 by seanB, smustgrave, alexpott, catch: Add static caching to...,!8723Make dblog entities,!8708Filter out disabled display extenders on save,!8691#3459116 - Update Manager Settings: Validate same email address added multiple times,!8665Issue #3449743 by catch: Try to optimize test ordering when run-tests.sh is...,!8598Draft: Issue #3458215: Migrate Toolbar button to SDC,!8572Reorder checkboxes on "Development settings" page,!8538Issue #3457009: Fixing xxception message thrown.,!8505Issue #3456528: _node_mass_update_batch_process fails during user cancel when revision is deleted,!8494Issue #3452511: Convert ProviderRepositoryTest to a kernel test,!8482Resolve #3456202 "Phpunit tests support",!8463Prevent re-install if site already exists
Pipeline #201171 passed with warnings
Pipeline: drupal

#201176

    ...@@ -35,7 +35,7 @@ ...@@ -35,7 +35,7 @@
    "jquery/no-param": 2, "jquery/no-param": 2,
    "jquery/no-parent": 0, "jquery/no-parent": 0,
    "jquery/no-parents": 0, "jquery/no-parents": 0,
    "jquery/no-parse-html": 0, "jquery/no-parse-html": 2,
    "jquery/no-prop": 0, "jquery/no-prop": 0,
    "jquery/no-proxy": 2, "jquery/no-proxy": 2,
    "jquery/no-ready": 2, "jquery/no-ready": 2,
    ......
    ...@@ -1333,7 +1333,20 @@ ...@@ -1333,7 +1333,20 @@
    const settings = response.settings || ajax.settings || drupalSettings; const settings = response.settings || ajax.settings || drupalSettings;
    // Parse response.data into an element collection. // Parse response.data into an element collection.
    let $newContent = $($.parseHTML(response.data, document, true)); const parseHTML = (htmlString) => {
    const fragment = document.createDocumentFragment();
    // Create a temporary div element
    const tempDiv = fragment.appendChild(document.createElement('div'));
    // Set the innerHTML of the div to the provided HTML string
    tempDiv.innerHTML = htmlString;
    // Return the contents of the temporary div
    return tempDiv.childNodes;
    };
    let $newContent = $(parseHTML(response.data));
    // For backward compatibility, in some cases a wrapper will be added. This // For backward compatibility, in some cases a wrapper will be added. This
    // behavior will be removed before Drupal 9.0.0. If different behavior is // behavior will be removed before Drupal 9.0.0. If different behavior is
    // needed, the theme functions can be overridden. // needed, the theme functions can be overridden.
    ......
    0% Loading or .
    You are about to add 0 people to the discussion. Proceed with caution.
    Finish editing this message first!
    Please register or to comment