-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PageSession stored directly to view scope #90
PageSession stored directly to view scope #90
Conversation
@avpinchuk , @arjantijms , please review. |
@OndroMih, Looks good. I build GF and try to deploy and undeploy a simple |
But why not use simple raw |
You're right, why not. I've changed it :) |
Storing into a map in view scope preferred page session over view session. But in JSF there's no page session, view scope is equivalent, the old page scope should be the same as JSF view scope. Signed-off-by:Ondro Mihalyi <mihalyi@omnifish.ee>
More efficient if the target class is known. Signed-off-by:Ondro Mihalyi <mihalyi@omnifish.ee>
dada8ef
to
7904fd0
Compare
@avpinchuk , please approve again. I had to fix the author information in my commits and force push to fix the ECA issue. |
Not working deploy war file in glassfish 7.0.3 ->GUI internal error: Archive Path is NULL |
This is fixed in GlassFish 7.0.4. |
Storing into a map in view scope preferred page session over view session. But in JSF there's no page session, view scope is equivalent, the old page scope should be the same as JSF view scope.
This should fix eclipse-ee4j/glassfish#24383 and potentially some other issues in GlassFish, which were introduced by #82.