why do you return True here?