testing update , removed returns in loops