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.

Comments

dan.munn’s picture

StatusFileSize
new2.32 KB
dan.munn’s picture

Status: Active » Needs review
dan.munn’s picture

Version: 7.x-3.3 » 7.x-3.x-dev

This issue is still present on latest dev, patch still applies.

pauloamgomes’s picture

tested today and issue is still in latest dev

mudjunky’s picture

n/m

jribeiro’s picture

The patch is no longer being applied, changed the patch to work with the latest version 3.7.

jribeiro’s picture

StatusFileSize
new2.26 KB
japerry’s picture

Status: Needs review » Closed (outdated)

Drupal 7 is no longer supported, closing.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.