@@ -1171,10 +1171,13 @@ SC.mixin(Array.prototype, SC.Reducers) ; }; // Apply methods if missing... - // Also override prototype for now. Our methods are functionally identical - // and don't break the browsers. for(var key in mixinIfMissing) { if (!mixinIfMissing.hasOwnProperty(key)) continue ; + + // The mixinIfMissing methods should be applied if they are not defined. + // If Prototype 1.6 is included, some of these methods will be defined + // already, but we want to override them anyway in this special case + // because our version is faster and functionally identitical. if (!Array.prototype[key] || ((typeof Prototype === 'object') && Prototype.Version.match(/^1\.6/))) { Array.prototype[key] = mixinIfMissing[key] ; } mixins/enumerable.js 651067f3df1304d5a649a25b2947c06d964a8616 Charles Jolley charles@Acme-Labs.local http://github.com/sproutit/sproutcore/commit/4d082538d63a3eb02f04bac51e572528185ec642 4d082538d63a3eb02f04bac51e572528185ec642 2008-12-17T16:11:50-08:00 2008-12-17T16:11:50-08:00 Clarified comment so that hopefully in the future this Prototype special case will not confuse anyone. 0112014b2f4cd1ed657b013fe0b45c21ee098a6b Charles Jolley charles@Acme-Labs.local