should this be an else?