[TAPESTRY-720] Tapestry should validate that all engine services provide a non-null service parameter

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 4.0
• Fix Version/s: 4.0
• Component/s: Framework
• Assignee: Howard M. Lewis Ship
• Reporter: Howard M. Lewis Ship
• Created: 23/Oct/05 05:40
• Updated: 13/Nov/05 03:23
• Resolved: 13/Nov/05 03:23

Description

This could be checked by the LinkFactory.

Currently, a service implementation that does not provide a value for the service parameter can cause NPEs inside service encoders:

Stack Trace:

• org.apache.tapestry.engine.encoders.PageServiceEncoder.encode(PageServiceEncoder.java:41)
• org.apache.tapestry.services.impl.LinkFactoryImpl.createServiceEncoding(LinkFactoryImpl.java:138)
• org.apache.tapestry.services.impl.LinkFactoryImpl.constructLink(LinkFactoryImpl.java:106)
• $LinkFactory_1071a0bf7a7.constructLink($LinkFactory_1071a0bf7a7.java)
• lab.services.LocaleService.getLink(LocaleService.java:54)
• $IEngineService_1071a0bf83a.getLink($IEngineService_1071a0bf83a.java)
• org.apache.tapestry.services.impl.EngineServiceInnerProxy.getLink(EngineServiceInnerProxy.java:69)
• org.apache.tapestry.services.impl.EngineServiceOuterProxy.getLink(EngineServiceOuterProxy.java:60)
• lab.components.LocaleLink.renderComponent(LocaleLink.java:80)
• org.apache.tapestry.AbstractComponent.render(AbstractComponent.java:617)
• org.apache.tapestry.AbstractComponent.renderBody(AbstractComponent.java:434)
• org.apache.tapestry.html.Body.renderComponent(Body.java:129)