Changeset 12798

Timestamp:

02/29/08 21:16:48 (9 months ago)

Author:

alex

Message:

lastIndexOf() was totally borked. This fixes, rolling back changes from r10203. Refs #3961. !strict

Files:

• dojo/trunk/_base/array.js (modified) (1 diff)

dojo/trunk/_base/array.js

@@ -24 +24 @@
                         //                      http://developer.mozilla.org/en/docs/Core_JavaScript_1.5_Reference:Global_Objects:Array:indexOf
 
-                        var step = 1, end = array.length, i = 0;
+                        var step = 1, end = array.length, i = (fromIndex||0);
                         if(findLast){
-                                i = end - 1;
-                                step = end = -1;
+                                step = -1, i = (fromIndex||array.length - 1), end = -1;
                         }
-                        for(i = fromIndex || i; i < end; i += step){
+                        for(; i!=end; i+=step){
                                 if(array[i] == value){ return i; }
                         }