Re: [Improvement] getPageParameters()

2009-05-03 Thread Alex Objelean
gt; - > To unsubscribe, e-mail: users-unsubscr...@wicket.apache.org > For additional commands, e-mail: users-h...@wicket.apache.org > > > -- View this message in context: http://www.nabble.com/-Improvement--getPagePa

Re: [Improvement] getPageParameters()

2009-05-03 Thread Matej Knopp
PagePameters is mutable object. I don't think one shared empty instance is a good idea. -Matej On Sun, May 3, 2009 at 11:44 AM, Johan Compagner wrote: > make a jira issue for this. > > On Sun, May 3, 2009 at 11:38, Objelean Alex wrote: > >> When using getPageParameters(), I used to have some tr

Re: [Improvement] getPageParameters()

2009-05-03 Thread Johan Compagner
make a jira issue for this. On Sun, May 3, 2009 at 11:38, Objelean Alex wrote: > When using getPageParameters(), I used to have some troubles because > forgetting to call page super(params). As a result, getPageParameters() > returns null. This problem also occur when working with non bookmarkab

[Improvement] getPageParameters()

2009-05-03 Thread Objelean Alex
When using getPageParameters(), I used to have some troubles because forgetting to call page super(params). As a result, getPageParameters() returns null. This problem also occur when working with non bookmarkable page. My suggestion is to take advantage of Null Object Pattern