this would be fine for a refactor, but it would be nice to centralize SQL strings and given them meaningful names