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.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dan.munn’s picture

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