Skip to content

Wicket 7.x please correction doc about CDI#203

Closed
jac-czerwinski wants to merge 444 commits into
masterfrom
wicket-7.x
Closed

Wicket 7.x please correction doc about CDI#203
jac-czerwinski wants to merge 444 commits into
masterfrom
wicket-7.x

Conversation

@jac-czerwinski

@jac-czerwinski jac-czerwinski commented Jan 10, 2017

Copy link
Copy Markdown
Contributor

Please set somewhere visible in the documentation, that wicket-cdi-1.1 is current and compatible dependency. General convention - if someone try to guess - lead to name wicket-cdi , which goes to compilation errors in general examples (CdiConfiguration constructor with/without arguments). Or mark older version to some degree deprecated. Both branches have active commits and hard to investigate by date

I can't find such information and lost hour or two.

klopfdreh and others added 30 commits March 21, 2016 20:20
…ket.metrics.applicationName"

Add more docu that this JVM setting will not help when there are more than one Wicket apps deployed in the web container.
…ady shown

Remove the workaround for the incorrect behavior in old Tomcat versions.
… disabled will now log a warning instead of ignoring the click
 - When using Tomcats o.a.c.h.t.SimpleTCPCluster to replicate Sessions
   in a Tomcat cluster the o.a.c.h.s.DeltaSession is the default
   implementation which handles session replication
 - Unfortunately, to calculate its deltas, it assumes that session
   attributes are only modified by calling accessor methods (i.e.
   DeltaSession.setAttribute(n,v))
 - Modifying an existing Object (e.g. wicket's PageTable) in place of
   the current session, would not trigger a delta replication
 - To make Wicket work with SimpleTCPCluster and DeltaSessions, we will
   always rewrite the PageTable into Session by calling setAttribute
@klopfdreh

Copy link
Copy Markdown
Member

It seems that you have selected the wrong branch for the PR.

@martin-g

Copy link
Copy Markdown
Member

@jac-czerwinski Please close this PR. It cannot be merged.

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.