[
https://issues.apache.org/jira/browse/WICKET-6507?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16294310#comment-16294310
]
Kamil commented on WICKET-6507:
-------------------------------
[~bitstorm], of course in our real app we have correct version of
"modelChanged" called (the one from AbstractWindow). This "bug" was introduced
during preparation of this "quickstart" and final effect was identical to the
real problem, so I thought it was the cause.
But the real problem lies in line:
{code}
new
FeedbackCollector(form).collect().stream().forEach(FeedbackMessage::markRendered);
{code}
Would you be so kind, and look at it again using this version of "onSubmit":
{code}
@Override
protected void onSubmit(final AjaxRequestTarget target) {
AbstractWindow.this.onSubmit(target);
AbstractWindow.this.modelChanged();
close(target);
}
{code}
and see what happens when you comment/uncomment FeedbackCollector line?
Thank you in advance.
> Race condition
> --------------
>
> Key: WICKET-6507
> URL: https://issues.apache.org/jira/browse/WICKET-6507
> Project: Wicket
> Issue Type: Bug
> Components: wicket
> Affects Versions: 8.0.0-M8
> Reporter: Kamil
> Assignee: Andrea Del Bene
> Attachments: wicket race condition.gif, wicket race condition.mp4,
> wicket-race-condition-quickstart.zip
>
>
> This bug is weird, but it has probably something to do with race condition.
> I recorded a screencast, so you can clearly see what is wrong.
> When I have some computation directly in "onSubmit" method, then JS alert is
> invoked properly. If I try to move computation somewhere else (here I just do
> "extract method" refactoring, but in real life this operation would be
> executed in some top-level panel or service) then JS is not being invoked
> (good thing: this behavior is consistent across all of my tries)!
> I attached quickstart so you can try it by yourself. The lines I'm extracting
> are: AbstractWindow#55
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
