I think the _id is unnecessary in 'attr_accessible :type_id, :subject_id'