You didn't declare i in DrawRow() with the var keyword -- that's your problem, since it's overwriting the variable i you're using to loop over the query...
I also I don't see aRow declared anywhere... i.e. aRow = arrayNew(1)... don't know if that's a problem... And your setValue function could be shorter: function SetValue(someNo, someAmt) { switch (someNo) { case "082": { ; } case "126": { ; } case "153": { ; } case "176": { ; } case "301": { ; } aRow[5] = someAmt; break; } } return aRow; // return an array } s. isaac dealey 954.522.6080 new epoch : isn't it time for a change? add features without fixtures with the onTap open source framework http://www.fusiontap.com http://coldfusion.sys-con.com/author/4806Dealey.htm ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~| Find out how CFTicket can increase your company's customer support efficiency by 100% http://www.houseoffusion.com/banners/view.cfm?bannerid=49 Message: http://www.houseoffusion.com/lists.cfm/link=i:4:210386 Archives: http://www.houseoffusion.com/cf_lists/threads.cfm/4 Subscription: http://www.houseoffusion.com/lists.cfm/link=s:4 Unsubscribe: http://www.houseoffusion.com/cf_lists/unsubscribe.cfm?user=11502.10531.4 Donations & Support: http://www.houseoffusion.com/tiny.cfm/54