Changeset 228 for trunk/common

Timestamp:

09/28/07 20:18:33 (2 years ago)

Author:

ydnar

Message:

bugid:58987; added Enumerator class for non-IE browsers

Location:

trunk/common

Files:

• Core.js (modified) (1 diff)
• DOM.js (modified) (5 diffs)

trunk/common/Core.js

@@ -883 +883 @@
 
 
+/* enumerable interface */
+
+if( !defined( window.Enumerator ) ) {
+    window.Enumerator = new Class( Object, {
+        init: function( a ) {
+            this.data = a;
+            this.index = 0;
+        },
+        
+        
+        atEnd: function() {
+            return this.index >= (this.data.length - 1) ? true : false;
+        },
+        
+        
+        item: function() {
+            return this.atEnd() ? undefined : this.data[ this.index ];
+        },
+        
+        
+        moveFirst: function() {
+            this.index = 0;
+            return this.item();
+        },
+        
+        
+        moveNext: function() {
+            this.index++;
+            return this.item();
+        }
+    } );
+}
+
+
 /* ajax */
 

trunk/common/DOM.js

@@ -568 +568 @@
         var parentDocument = pw.document;
         var es = parentDocument.getElementsByTagName( "iframe" );
-        for( var i = 0; i < es.length; i++ ) {
-            var e = es[ i ];
+        //% for( var i = 0; i < es.length; i++ ) {
+        //%     var e = es[ i ];
+        for( var en = new Enumerator( es ); !en.atEnd(); en.moveNext() ) {
+            var e = en.item();
             try {
                 var d = e.contentDocument || e.contentWindow.document;
@@ -582 +584 @@
     filterElementsByClassName: function( es, cn ) {
         var filtered = [];
-        for( var i = 0; i < es.length; i++ ) {
-            var e = es[ i ];
+        //% for( var i = 0; i < es.length; i++ ) {
+        //%     var e = es[ i ];
+        for( var en = new Enumerator( es ); !en.atEnd(); en.moveNext() ) {
+            var e = en.item();
             if( DOM.hasClassName( e, cn ) )
                 filtered[ filtered.length ] = e;
@@ -597 +601 @@
             return es;
         var f = [];
-        for( var i = 0; i < es.length; i++ ) {
-            var e = es[ i ];
+        //% for( var i = 0; i < es.length; i++ ) {
+        //%     var e = es[ i ];
+        for( var en = new Enumerator( es ); !en.atEnd(); en.moveNext() ) {
+            var e = en.item();
             if( !e )
                 continue;
@@ -615 +621 @@
         var f = [];
         tn = tn.toLowerCase();
-        for( var i = 0; i < es.length; i++ ) {
-            var e = es[ i ];
+        //% for( var i = 0; i < es.length; i++ ) {
+        //%    var e = es[ i ];
+        for( var en = new Enumerator( es ); !en.atEnd(); en.moveNext() ) {
+            var e = en.item();
             if( e.tagName && e.tagName.toLowerCase() == tn )
                 f.push( e );
@@ -922 +930 @@
             return;
         var es = DOM.getAncestors( ev.target, true );
-        for( var i = 0; i < es.length; i++ ) {
+        //% for( var i = 0; i < es.length; i++ ) {
+        //%     var e = es[ i ];
+        for( var en = new Enumerator( es ); !en.atEnd(); en.moveNext() ) {
+            var e = en.item();
             try {
-                var e = es[ i ];
                 var v = e.getAttribute ? e.getAttribute( a ) : null;
                 if( v ) {