thanks! fixed.

Jan Mikkelsen wrote:

Hi

I think I have spotted a small bug in PageMap:

        public final Page get(final String id)
{
        final Page page = (Page)getPages().get(id);
        page.setDirty(true);
        return page;
}

page is null in case the requested page is not in MostRecentlyUsedMap,
causing a NPE . I tried this instead:
public final Page get(final String id)
{
        final Page page = (Page)getPages().get(id);
        if (page != null) {
                page.setDirty(true);
        }
        return page;
}

This is better, I now get "Page Expired" instead of an exception.

kind regards, Jan



-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
_______________________________________________
Wicket-user mailing list
Wicket-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wicket-user





-------------------------------------------------------
SF email is sponsored by - The IT Product Guide
Read honest & candid reviews on hundreds of IT Products from real users.
Discover which products truly live up to the hype. Start reading now.
http://ads.osdn.com/?ad_id=6595&alloc_id=14396&op=click
_______________________________________________
Wicket-user mailing list
Wicket-user@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wicket-user

Reply via email to