[
https://issues.apache.org/jira/browse/WICKET-6507?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Kamil updated WICKET-6507:
--------------------------
Comment: was deleted
(was: Oh, I see where the problem is!
I read Javadoc more carefully and I spotted that despite I'm creating the
FeedbackCollector explicitly using Component "form"
{code}
new FeedbackCollector(form)
{code}
the
{code}
includeSession
{code}
is set to true, which I is not so obvious.
Do you think that maybe more reasonable approach would be:
1) If one is creating FeedbackCollector using no argument constructor it sets
includeSession to true (as it is now)
2) If one is creating FeedbackCollector using the constructor with specific
component, includeSession should be set to false by default
3) You could add additional constructor that takes both: specific Component and
includeSession flag
What do you think about it?
)
> FeedbackCollector clears JS events
> ----------------------------------
>
> 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)
