On Monday 19 August 2013 17:32:45 Martin Grigorov wrote: > Hi Emond, > > I think this change is OK. > Maybe we can improve it a bit by using Application.get().getPageSettings(). > getRecreateMountedPagesAfterExpiry() in the checks above ? > > With the new check as you can see the produced urls contain the class name. > Some users don't like this. > Application.get().getPageSettings().getRecreateMountedPagesAfterExpiry() > return true by default. If someone doesn't like the extra info in the > produced urls then she can disable it this way.
I've thought some more about this, and I think the current behavior is ok. If you don't want bookmarkable urls, but your page has a bookmarkable constructor, you should override isBookmarkable. The setting is more about request handling than it is about rendering urls. The reason the pageparameters are not rendered in the url is, that they would otherwise be rendered as query parameters, overriding other (such as form) parameters. I'll merge the branch somewhere tomorrow and forward port it to 7 if nobody objects. Best regards, Emond
