[open-ils-commits] r12631 - trunk/Open-ILS/web/js/dojo/openils (erickson)

svn at svn.open-ils.org svn at svn.open-ils.org
Fri Mar 20 14:53:45 EDT 2009


Author: erickson
Date: 2009-03-20 14:53:41 -0400 (Fri, 20 Mar 2009)
New Revision: 12631

Modified:
   trunk/Open-ILS/web/js/dojo/openils/User.js
Log:
let's not clobber the loop variable

Modified: trunk/Open-ILS/web/js/dojo/openils/User.js
===================================================================
--- trunk/Open-ILS/web/js/dojo/openils/User.js	2009-03-20 18:18:17 UTC (rev 12630)
+++ trunk/Open-ILS/web/js/dojo/openils/User.js	2009-03-20 18:53:41 UTC (rev 12631)
@@ -205,16 +205,18 @@
                 var permMap = {};
                 if(r) permMap = openils.Util.readResponse(r);
                 var orgList = [];
+
                 for(var i = 0; i < permList.length; i++) {
                     var perm = permList[i];
                     var permOrgList = permMap[perm] || self.permOrgCache[self.user.id()][perm];
                     self.permOrgCache[self.user.id()][perm] = permOrgList;
-                    for(var i in permOrgList) {
+
+                    for(var j in permOrgList) {
                         if(includeDescendents) {
                             orgList = orgList.concat(
-                                fieldmapper.aou.descendantNodeList(permOrgList[i]));
+                                fieldmapper.aou.descendantNodeList(permOrgList[j]));
                         } else {
-                            orgList = orgList.concat(fieldmapper.aou.findOrgUnit(permOrgList[i]));
+                            orgList = orgList.concat(fieldmapper.aou.findOrgUnit(permOrgList[j]));
                         }
                     }
                 }



More information about the open-ils-commits mailing list