[ https://jira.jboss.org/browse/GTNPORTAL-1288?page=com.atlassian.jira.plug... ] Trong Tran updated GTNPORTAL-1288: ---------------------------------- Attachment: exception.txt -- 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

[ https://jira.jboss.org/browse/GTNPORTAL-1288?page=com.atlassian.jira.plug... ] kien nguyen reassigned GTNPORTAL-1288: -------------------------------------- Assignee: kien nguyen -- 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

[ https://jira.jboss.org/browse/GTNPORTAL-1288?page=com.atlassian.jira.plug... ] Minh Hoang TO commented on GTNPORTAL-1288: ------------------------------------------ if (parentId != null) { // Get the new parent UIContainer parent = session.findObjectById(ObjectType.CONTAINER, parentId); // Perform the move parent.getComponents().add(dstChild); // changes.add(new ModelChange.Move(dst.getObjectId(), parentId, dstChildId)); // julien : we do not need to create an update operation // as later the update operation will be created when the object // will be processed } } In Mapper, line 802, there is a missing case where the new parent 's ObjectType is no longer UIContainer, but Page. That lead to classcast exception while converting JCR Node to MOP object -- 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

[ https://jira.jboss.org/browse/GTNPORTAL-1288?page=com.atlassian.jira.plug... ] Minh Hoang TO reassigned GTNPORTAL-1288: ---------------------------------------- Assignee: Minh Hoang TO (was: tung do thanh) -- 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

[ https://jira.jboss.org/browse/GTNPORTAL-1288?page=com.atlassian.jira.plug... ] Trong Tran updated GTNPORTAL-1288: ---------------------------------- Original Estimate: 4 hours Remaining Estimate: 4 hours Environment: (was: ) Workaround Description: (was: ) Steps to Reproduce: (was: ) -- 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

[ https://jira.jboss.org/browse/GTNPORTAL-1288?page=com.atlassian.jira.plug... ] Work on GTNPORTAL-1288 started by Minh Hoang TO. -- 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

[ https://jira.jboss.org/browse/GTNPORTAL-1288?page=com.atlassian.jira.plug... ] Work on GTNPORTAL-1288 stopped by Minh Hoang TO. -- 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

[ https://jira.jboss.org/browse/GTNPORTAL-1288?page=com.atlassian.jira.plug... ] Minh Hoang TO commented on GTNPORTAL-1288: ------------------------------------------ As i mentioned in earlier comments, there are order-related in recursive method saveChildren from class Mapper. // Delete removed children for (Iterator<UIComponent> i = dst.getComponents().iterator(); i.hasNext();) { UIComponent dstChild = i.next(); String dstChildId = dstChild.getObjectId(); if (!modelObjectMap.containsKey(dstChildId)) { i.remove(); changes.add(new ModelChange.Destroy(dstChildId)); } } The statement i.remove() in above code snippet would remove the Node specified by ID dstChildId from the workspace ( that is what we don't expect in case we permute the UIComponent but do not destroy them). Hence, the consequent recursive calls would fail if the object having storageId dstChildId is required. My idea to fix this issue is to make an overall 'clean' outside the method saveChildren. For each recursive call in saveChildren, a ModelChange.Destroy is registered like below. // Delete removed children for (Iterator<UIComponent> i = dst.getComponents().iterator(); i.hasNext();) { UIComponent dstChild = i.next(); String dstChildId = dstChild.getObjectId(); if (!modelObjectMap.containsKey(dstChildId)) { //i.remove(); //changes.add(new ModelChange.Destroy(dstChildId)); changes.add(new ModelChange.Destroy(dst.getObjectId(), dstChildId)); } } -- 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

[ https://jira.jboss.org/browse/GTNPORTAL-1288?page=com.atlassian.jira.plug... ] Julien Viet resolved GTNPORTAL-1288. ------------------------------------ Resolution: Done -- 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