For loop runs once