Skip to content

Conversation

@briantopping
Copy link
Member

This was talked about on the dev list, but apparently I never generated a pull request? Oh well, here it is.

@briantopping
Copy link
Member Author

Oops, should improve this somewhat

asfgit pushed a commit that referenced this pull request Mar 14, 2014
…duration'

Remove detaching of the pages in RequestAdapter#commitRequest().

The page has been detached few calls ago:
In org.apache.wicket.request.cycle.RequestCycle#onDetach:

try
{
  requestHandlerExecutor.detach();   // detaches pages #1
}
finally
{
  listeners.onDetach(this);          // detaches pages #2 (no more)
}
asfgit pushed a commit that referenced this pull request Jun 24, 2014
…duration'

Remove detaching of the pages in RequestAdapter#commitRequest().

The page has been detached few calls ago:
In org.apache.wicket.request.cycle.RequestCycle#onDetach:

try
{
  requestHandlerExecutor.detach();   // detaches pages #1
}
finally
{
  listeners.onDetach(this);          // detaches pages #2 (no more)
}

(cherry picked from commit 48eaae6)
asfgit pushed a commit that referenced this pull request Mar 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant