<div dir="ltr">Hi Dave and Bob,<div><br></div><div>I noticed some inconsistency between Attribute and EntityRef, in terms of how to manage the associations between entity and attribute.</div><div><br></div><div>In Attribute class, there is a ModelEntityItem (m_associations) keeping track of the associated entities, and the Attribute::associateEntity() only updates m_associations without updating the Manager (so the manager's association record does not reflect the system), however Attribute::disassociateEntity() will update both m_associations and the Manager.</div><div><br></div><div>In EntityRef class, the EntityRef::associateAttribute() and EntityRef::disassociateAttribute() depend fully on the Manager to update the associations.</div><div><br></div><div>I suggest to remove the m_associations from Attribute, and use the Manager instead (like EntityRef) to manage associations. This will avoid keeping the same record in two places and potential mismatch between them.</div><div><br></div><div>Comments?</div><div><br></div><div>Yumin</div></div>