Creo que sería mejor usar unless, en vez de un if que no hace nada, ¿que te parece?.