[ 
https://issues.apache.org/jira/browse/TAP5-2100?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14084535#comment-14084535
 ] 

ASF subversion and git services commented on TAP5-2100:
-------------------------------------------------------

Commit dca3b4875a9fe96fb3e6f3ec6bf90cafb086b93a in tapestry-5's branch 
refs/heads/master from [~jkemnade]
[ https://git-wip-us.apache.org/repos/asf?p=tapestry-5.git;h=dca3b48 ]

TAP5-2100: add an additional sleep to wait for the first refresh after clicking 
the link in the test case


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

Reply via email to