shouldn't this be a foreign key?