[gatein-issues] [JBoss JIRA] Updated: (GTNPORTAL-1605) Merging PageNavigations should also merge child PageNodes

[Matt Wringe (JIRA)]

     [ https://jira.jboss.org/browse/GTNPORTAL-1605?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Matt Wringe updated GTNPORTAL-1605:
-----------------------------------

    Attachment: merge_pageNodes.patch


This patch will merge the PageNodes and the PageNode children together was well as update how the WSRP admin is added so that it gets put in a more correct location. Please note that the merging of the page nodes in the patch should probably be moved from the PageNavigation class to the PageNode class.

> Merging PageNavigations should also merge child PageNodes
> ---------------------------------------------------------
>
>                 Key: GTNPORTAL-1605
>                 URL: https://jira.jboss.org/browse/GTNPORTAL-1605
>             Project: GateIn Portal
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>            Reporter: Matt Wringe
>         Attachments: merge_pageNodes.patch
>
>
> The PageNavigation.merge command only merges in top level PageNodes, if a PageNode with the same name already exists, it will overwrite the previous node with a new one.
> This prevents being able to add a new page node to a subnode which already exists.
> For example, the administration node by default contains the app-registry and page-management PageNodes. I cannot use the extension mechanism to add an extra PageNode here, since the PageNavigation.merge will delete the current administration node and overwrite it with the one I specify in the extension. In this situation I am stuck adding it to another top level location, which is not ideal.
> What should happen is that the child PageNodes get merged together, so that I can add extra nodes anywhere in the PageNode tree.
> This will however change the behaviour of how extensions work. What this means is that you can no longer overwrite or remove PageNodes using the extension mechanism.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira