would it be better to prevent null values from even being an id or catch it here?