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

Dan Allen (JIRA) jira-events at lists.jboss.org
Wed Oct 3 09:06:03 EDT 2007


    [ http://jira.jboss.com/jira/browse/JBSEAM-1969?page=comments#action_12380140 ] 
            
Dan Allen commented on JBSEAM-1969:
-----------------------------------

If you override getEntityClass() to provide your own return value, it will lead to a NullPointerException when you call one of the message convenience methods because they reference the private field entityClass from the parent class Home. Since the subclass does not have access to this field (other than calling setEntityClass()), it remains null. This fix has those methods use getEntityClass() rather than the field directly, allowing the convenience methods to utilize the overridden method.

I have a test case as well.  Attaching.

> updatedMessage, deletedMessage and createdMessage in framework.Home cause NPE if getEntityClass is overridden
> -------------------------------------------------------------------------------------------------------------
>
>                 Key: JBSEAM-1969
>                 URL: http://jira.jboss.com/jira/browse/JBSEAM-1969
>             Project: JBoss Seam
>          Issue Type: Bug
>          Components: Framework
>         Environment: Mac OSX 10.4.8, Jboss 4.2.1, Seam 1.3.0 
>            Reporter: Damian Harvey
>         Assigned To: Dan Allen
>            Priority: Trivial
>             Fix For: 2.0.1.GA
>
>         Attachments: JBSEAM-1969-v1.txt
>
>
> I am extending an existing class (BookingHome) that extends EntityHome. My BookingHome class needs to override getEntityClass. Doing this then results in a NPE as updatedMessage, deletedMessage and createdMessage refer to entityClass rather than getEntityClass.

-- 
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

        



More information about the seam-issues mailing list