[ 
https://issues.apache.org/jira/browse/WICKET-3377?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Igor Vaynberg resolved WICKET-3377.
-----------------------------------

    Resolution: Duplicate
      Assignee: Pedro Santos

> 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
>            Assignee: Pedro Santos
>            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.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to