Javascript code is running in a limited loop need to refactor