Author: bdaw Date: 2008-07-25 04:03:26 -0400 (Fri, 25 Jul 2008) New Revision: 11587 Modified: modules/identity/branches/JBP_IDENTITY_BRANCH_1_0/identity/src/main/org/jboss/portal/identity/db/HibernateRoleImpl.java Log: - stupid bug... Modified: modules/identity/branches/JBP_IDENTITY_BRANCH_1_0/identity/src/main/org/jboss/portal/identity/db/HibernateRoleImpl.java =================================================================== --- modules/identity/branches/JBP_IDENTITY_BRANCH_1_0/identity/src/main/org/jboss/portal/identity/db/HibernateRoleImpl.java 2008-07-24 20:44:01 UTC (rev 11586) +++ modules/identity/branches/JBP_IDENTITY_BRANCH_1_0/identity/src/main/org/jboss/portal/identity/db/HibernateRoleImpl.java 2008-07-25 08:03:26 UTC (rev 11587) @@ -151,8 +151,13 @@ if (broadcaster != null) { - RoleUpdatedEvent event = new RoleUpdatedEvent(getId(), getName(), displayName); - broadcaster.fireEvent(event); + // This can be called on object creation by hibernate so make sure that all fields are populated first + if (getId() != null && getName() != null && getDisplayName() != null) + { + RoleUpdatedEvent event = new RoleUpdatedEvent(getId(), getName(), displayName); + broadcaster.fireEvent(event); + } + } }