fixed conditional which was always true