[jbossseam-issues] [JBoss JIRA] Created: (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
Priority: Blocker
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();
}
}
--
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