[jbossseam-issues] [JBoss JIRA] Commented: (JBSEAM-524) Lifecycle.isAttributeDirty(Object attribute) doesn't like null attributes

Lifecycle.isAttributeDirty(Object attribute) doesn't like null attributes ------------------------------------------------------------------------- Key: JBSEAM-524 URL: http://jira.jboss.com/jira/browse/JBSEAM-524 Project: JBoss Seam Issue Type: Bug Components: Core Affects Versions: 1.1.0.CR1 Reporter: jarkko Lietolahti Assigned To: Gavin King Priority: Blocker Fix For: 1.1.0.CR2 Sometimes (because of living inside portal environment), WebSessionContext.flush fails when attribute obtained from Session (experimental session handling required for the portal env) is null, which is something that Lifecycle.isAttributeDirty(attribute) doesn't like and throws NPE. A simple fix is to check for null attribute values (others might require changing getNames() or WebSessionContext /PortletSessionImpl.getAttributeNames() ): public void flush() { for ( String name: getNames() ) { Object attribute = session.getAttribute(name); if( attribute == null) { continue; } if ( Lifecycle.isAttributeDirty(attribute) ) { session.setAttribute(name, attribute); } } } And the custom PortletSessionImpl which stores everything in APPLICATION_SCOPE instead of PORTLET_SCOPE /* * JBoss, Home of Professional Open Source * * Distributable under LGPL license. * See terms of license at gnu.org. */ package org.jboss.seam.portlet; import java.util.Collections; import java.util.Enumeration; import java.util.List; import javax.portlet.PortletSession; import org.apache.commons.collections.ListUtils; import org.jboss.seam.ScopeType; import org.jboss.seam.contexts.ContextAdaptor; /** * @author <a href="mailto:theute@jboss.org">Thomas Heute </a> * @version $Revision: 1.4 $ */ public class PortletSessionImpl extends ContextAdaptor { private PortletSession session; public PortletSessionImpl(PortletSession session) { this.session = session; } protected String getPrefix() { return ScopeType.CONVERSATION.getPrefix() + '#'; } protected int getScope(String key) { // if (key.startsWith(getPrefix())) { // return PortletSession.PORTLET_SCOPE; // } // return PortletSession.APPLICATION_SCOPE; } public Object getAttribute(String key) { int scope = getScope(key); Object o = session.getAttribute(key, scope); return o; } public void removeAttribute(String key) { int scope = getScope(key); session.removeAttribute(key, scope); } public void setAttribute(String key, Object value) { int scope = getScope(key); session.setAttribute(key, value, scope); } public Enumeration getAttributeNames() { // if (1 == 1) { // Enumeration portletAttributeNamesEnumOnly = session // .getAttributeNames(); // Enumeration koje = session // .getAttributeNames(PortletSession.APPLICATION_SCOPE); // return koje; // } Enumeration portletAttributeNamesEnum = session.getAttributeNames(); Enumeration portalAttributeNamesEnum = session .getAttributeNames(PortletSession.APPLICATION_SCOPE); List portletAttributeNames = Collections .list(portletAttributeNamesEnum); List portalAttributeNames = Collections.list(portalAttributeNamesEnum); // TODO: Is this correct? List allNames = ListUtils.sum(portletAttributeNames, portalAttributeNames); return Collections.enumeration(allNames); } public void invalidate() { session.invalidate(); } }