Skip to content

Conversation

@briantopping
Copy link
Member

Per conversation on wicket-dev [1]. These changes to POMs to merge wicket-request, wicket-util and wicket-core into an uber wicket-core that contains the contents of three modules. I've tested that the jars have exactly the same contents and the build executes perfectly.

[1] http://apache-wicket.1842946.n4.nabble.com/Restructure-Packaging-for-OSGi-tt3746217.html#a3748262

@martin-g
Copy link
Member

@Topping: can you please close this pull request because we don't have permissions.

@briantopping
Copy link
Member Author

Haha, Martijn was right about "make work"...

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 Apr 30, 2015
asfgit pushed a commit that referenced this pull request Apr 30, 2015
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.

2 participants