(we have to set it to true for that pagemap because that really needs it by default)
Will see if i can do the window.name change. But pretty busy the last days (there are some people on vaction...)
johan
On 7/5/06, Eelco Hillenius <
[EMAIL PROTECTED]> wrote:
Well, a couple of remarks to this:
- We didn't even officially agree yet that
SecondLevelCacheSessionStore is the best default store. It is the
default in 2.0, and it's fine as that is still an alpha branch, but
imo not the official default yet.
- *If* it will be the official default, multi window support should be
false by default. If not, it depends on the store impl which is then
the default, but it should probably default to true.
- So, say it's false by default, and thus a user explicitly sets multi
window support to true, while SecondLevelCacheSessionStore doesn't
want to support it, it should throw an exception, clearly explaining
that it can't be used with multiwindow support.
But now here's the deeper issue: I don't think
SecondLevelCacheSessionStore should not support it. I think the only
setting that should be of consequence is the user setting it. That's
related to the functionality I've been talking about. We don't have
that now, and though I'd like to have it, there is no guarantee we'll
have it soon, but it this is a case where a bit of ahead planning
actually does makes sense.
So for now, I think the best way is to make multiwindow support false,
at least as long as we're using SecondLevelCacheSessionStore.
Btw, who is going to implement that window.name change when? It would
be veeeery nice to have that in the next 1.2 release too, because it
solves some issues people are having with our current cookie based
one.
Eelco
On 7/5/06, Johan Compagner < [EMAIL PROTECTED]> wrote:
> Eelco did that:
>
> "do not set application settings automatically. 1) I don't agree with it,
> because this multi window support is about more than just supporting the
> back button 2) Even if we wanted to prevent that setting, it should check
> and throw an exception if set, rather than quietly setting it and leaving
> clients possibly with a big wtf if they find out this setting suddenly
> change"
>
> and i don't agree again with that.
> Because what the multi window does now is not what eelco wants.
> The multi window check is now pure for back button support and not that
> window session support what eelco also wants to have.
> I also don't get where an exception should be thrown..
>
> Again the multi window check is currently only there because whe had the
> AccessStackPageMap that one needs it.
> But the SecondLevelPageMap doesn't need it because pagemaps are not needed
> anymore.
>
> When i am in the SecondLevel Mode i don't want to create new pagemaps if a
> new browser window is detected... Never because new pagemaps (the current
> concept)
> is just a bad thing for the second level store.. I don't want new pagemaps
> because that only means more use of memory and a potential mem leak.
>
> The window support eelco wants is a bit different in my eyes. That is forces
> by the developer not by the user when opening a new browser tab.
>
> johan
>
>
>
>
> Using Tomcat but need to do more? Need to support web services, security?
> Get stuff done quickly with pre-integrated technology to make your job
> easier
> Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
> http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
>
>
> _______________________________________________
> Wicket-develop mailing list
> Wicket-develop@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/wicket-develop
>
>
>
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________
Wicket-develop mailing list
Wicket-develop@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wicket-develop
Using Tomcat but need to do more? Need to support web services, security? Get stuff done quickly with pre-integrated technology to make your job easier Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
_______________________________________________ Wicket-develop mailing list Wicket-develop@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wicket-develop
