Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Having just had an issue where additional javascript extended the Array prototype, it was noticable that IPE stopped functioning accross pages.
At the point of attaching behaviour, there is a for loop done in Javascript, however prototype additions are included in this loop leaving a Drupal thrown JS execution and an incomplete attachment of the IPE.
As such, changing the for from a for-in iterator to a tradition for across the javascript array makes sense and prevents the inclusion of any prototype additions and interface javascript errors.
Comment | File | Size | Author |
---|---|---|---|
#7 | panels.for-in-loop-replace-2161223-6.patch | 2.26 KB | jribeiro |
#1 | panels.for-in-loop-replace-2161223-1.patch | 2.32 KB | dan.munn |
Comments
Comment #1
dan.munn CreditAttribution: dan.munn commentedComment #2
dan.munn CreditAttribution: dan.munn commentedComment #3
dan.munn CreditAttribution: dan.munn commentedThis issue is still present on latest dev, patch still applies.
Comment #4
pauloamgomes CreditAttribution: pauloamgomes commentedtested today and issue is still in latest dev
Comment #5
mudjunky CreditAttribution: mudjunky commentedn/m
Comment #6
jribeiro CreditAttribution: jribeiro at CI&T for Pfizer, Inc. commentedThe patch is no longer being applied, changed the patch to work with the latest version 3.7.
Comment #7
jribeiro CreditAttribution: jribeiro at CI&T for Pfizer, Inc. commented