[
https://issues.apache.org/jira/browse/TAP5-2100?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14084499#comment-14084499
]
Hudson commented on TAP5-2100:
------------------------------
FAILURE: Integrated in tapestry-trunk-freestyle #1295 (See
[https://builds.apache.org/job/tapestry-trunk-freestyle/1295/])
TAP5-2100: ZoneRefresh mixin triggers refresh event with wrong context
(jochen.kemnade: rev e087cad11d609bc251727be91bf3d523811d541d)
*
tapestry-core/src/main/java/org/apache/tapestry5/corelib/mixins/ZoneRefresh.java
* tapestry-core/src/test/app1/ZoneRefreshWithContext.tml
* tapestry-core/src/test/app1/Index.tml
*
tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/pages/ZoneRefreshWithContext.java
*
tapestry-core/src/test/java/org/apache/tapestry5/integration/app1/ZoneRefreshTest.java
> ZoneRefresh mixin triggers refresh event with wrong context
> -----------------------------------------------------------
>
> Key: TAP5-2100
> URL: https://issues.apache.org/jira/browse/TAP5-2100
> Project: Tapestry 5
> Issue Type: Bug
> Components: tapestry-core
> Affects Versions: 5.3.6, 5.4
> Reporter: Ivan Khalopik
> Assignee: Jochen Kemnade
> Labels: patch
> Fix For: 5.4
>
> Attachments:
> 0001-TAP5-2100-ZoneRefresh-mixin-triggers-refresh-event-w.patch
>
>
> ZoneRefresh mixin triggers refresh event with context populated from
> parameter instead of context from event link.
> Before render script it creates event link with context:
> private Object createEventLink()
> {
> Link link = resources.createEventLink("zoneRefresh", context);
> return link.toAbsoluteURI();
> }
>
> But then it uses context parameter instead of context from event link:
> Object onZoneRefresh()
> {
> CaptureResultCallback<Object> callback = new
> CaptureResultCallback<Object>();
> resources.triggerEvent(EventConstants.REFRESH, context, callback);
>
> if(callback.getResult() != null){
> return callback.getResult();
> }
>
> return zone.getBody();
> }
--
This message was sent by Atlassian JIRA
(v6.2#6252)
