[jbossseam-issues] [JBoss JIRA] Commented: (JBSEAM-1969) updatedMessage, deletedMessage and createdMessage in framework.Home cause NPE if getEntityClass is overridden

[ http://jira.jboss.com/jira/browse/JBSEAM-1969?page=comments#action_12380205 ] Dan Allen commented on JBSEAM-1969: ----------------------------------- Okay, I thought about this one a little bit more. Here is what I am saying: 1. Should we stop using the entityClass private field directly, so as to allow the user to override getEntityClass() without exception OR should we say, "too bad, you have to call setEntityClass() if you want to avoid the NullPointerException because we are going to use the private field" 2. We should make getId() final. The Home class uses this.id all over the place and if the developer were to override getId() it would lead to lots of NullPointerExceptions. Besides, I don't see a reason for overriding getId(). If we don't make it final, then we need to stop using this.id all over Home. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://jira.jboss.com/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira