akamai/boomerang

Global collision issue

RyanGao-EM opened this issue · 2 comments

Here “i" doesn’t have var(var i = 0;) and it becomes global variable, so it’s causing issues with our script.

 isSinglePageApp: function(e) {
                var t = !1
                  , n = this.supported_frameworks();
                for (i = 0; i < n.length; i++) { // <========== var i
                    var r = n[i];
                    if (e[r] && e[r].enabled) {
                        t = !0;
                        break
                    }
                }
                return t
            } 

Good catch.

We'll backport this fix to our mPulse-specific boomerang.js as well