Shouldn't this be an `if...then...else`?