[gatein-issues] [JBoss JIRA] Commented: (GTNPORTAL-1007) Bug when updating existing pageNode with special name

[Minh Hoang TO (JIRA)]

    [ https://jira.jboss.org/jira/browse/GTNPORTAL-1007?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12525322#action_12525322 ] 

Minh Hoang TO commented on GTNPORTAL-1007:
------------------------------------------

for (NavigationNodeData node : src.getNodes())
      {
         String srcId = node.getStorageId();
         Navigation dstChild;
         if (srcId != null)
         {
            dstChild = session.findObjectById(ObjectType.NAVIGATION, srcId);
         }
         else
         {
            dstChild = dst.getChild(node.getName());
            if (dstChild == null)
            {
               dstChild = dst.addChild(node.getName());
            }
            srcId = dstChild.getObjectId();
         }
         save(node, dstChild);
         savedSet.add(srcId);
         orders.add(dstChild.getObjectId());
      }


 Line 239 in org.exoplatform.portal.pom.data.Mapper.java, shows us that the node name is used as default storageId. Once the node is persisted, the node name is immutable.

> Bug when updating existing pageNode with special name
> -----------------------------------------------------
>
>                 Key: GTNPORTAL-1007
>                 URL: https://jira.jboss.org/jira/browse/GTNPORTAL-1007
>             Project: GateIn Portal
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>    Affects Versions: 3.0.0-GA
>            Reporter: Hoat Le
>            Assignee: Minh Hoang TO
>             Fix For: 3.1.0-GA
>
>         Attachments: testDeepSaveNavigation.patch
>
>   Original Estimate: 1 day
>  Remaining Estimate: 1 day
>
> Bug happened when updating existing pageNode with special name.
> Patch file attached for unit test.

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