[ http://issues.apache.org/jira/browse/TAPESTRY-720?page=all ]
     
Howard M. Lewis Ship closed TAPESTRY-720:
-----------------------------------------

    Fix Version: 4.0
     Resolution: Fixed

Changed the signature of constructLink() to take the IEngineService as the 
first parameter, and added a check for the service name being null.

> Tapestry should validate that all engine services provide a non-null service 
> parameter
> --------------------------------------------------------------------------------------
>
>          Key: TAPESTRY-720
>          URL: http://issues.apache.org/jira/browse/TAPESTRY-720
>      Project: Tapestry
>         Type: Bug
>   Components: Framework
>     Versions: 4.0
>     Reporter: Howard M. Lewis Ship
>     Assignee: Howard M. Lewis Ship
>      Fix For: 4.0

>
> 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)

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
   http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see:
   http://www.atlassian.com/software/jira


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to