did not do a for in correctly