newPageMap at Session delete the default pageMap when excess Pagemap, and 
unneded pageMap are created for popupSettings with window name.
-----------------------------------------------------------------------------------------------------------------------------------------

                 Key: WICKET-3377
                 URL: https://issues.apache.org/jira/browse/WICKET-3377
             Project: Wicket
          Issue Type: Bug
          Components: wicket
    Affects Versions: 1.4.15
            Reporter: Yaron
            Priority: Critical


Related to: Session causing memory leak after WICKET-3108

When excess Pagemaps are reached then it might delete the default page map.
It seems unwise to delete the default page map as it usualy will live longer 
then page maps that will be created after it.

Seting a window name for PopupSettings will set it a page map name, this will 
create a pageMap at link creation.
It is not clear why the link target get the name using the pageMap (This create 
the page map) and not use the window name from the popup setting directly:
Code from Link class:
                        if (popupSettings != null)
                        {
                                IPageMap popupPageMap = 
popupSettings.getPageMap(this);
                                if (popupPageMap != null && 
popupPageMap.getName() != null)
                                {
                                        tag.put("target", 
popupPageMap.getName());
                                }
                        }



We have a table with links at the columns that each link needs a diffrent 
window name, the number of rows in the table changes and could reach 500.
This now mean the page than containes the table with the link will lose its 
page map.

It is needed to keep the default page map, and to create the page map for a 
popup link at link activation and not link creation.


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to