why have this logic in two places?