[JS2-511] org.apache.jetspeed.om.portlet.impl.PortletDefinitionImpl line 558 throws NPE

• Type: Bug
• Status: Closed
• Priority: Blocker
• Resolution: Fixed
• Affects Version/s: 2.1-dev
• Fix Version/s: 2.1-dev, 2.1
• Environment: WinXP SP2, JDK 1.4.2_11, Geronimo SVN head, Tomcat 5.5.15
• Assignee: Ate Douma
• Reporter: Jian Liao
• Created: 14/Mar/06 10:20
• Updated: 15/Apr/06 02:08
• Resolved: 15/Apr/06 02:08

Description

Geronimo default log level is debug. When an porltet app try to register itself to j2, commons-beanutils will call PortletDefinitionImpl.hashCode() to write a log. The issue is caused by PortletDefinitionImpl.getId() will return a null in PortletDefinitionImpl.hashCode(), line 558.

Reproduce:
1. Change your log level to debug(jetspeed\WEB-INF\conf\Log4j.properties).
2. Have a full build and call j2:quickStart
3. Start your Tomcat Server if you do not has a Geronimo.

I've tested it with Tomcat 5.5.15 and Geronimo SVN head.