[ https://issues.apache.org/jira/browse/ADFFACES-466?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
K SRINATH REDDY updated ADFFACES-466: ------------------------------------- Attachment: trunk_latest.patch Please use this patch istead of the old one. Instead of logging the warning when a child with the already existing ID is added, Now throwing a IllegalStateException as two components cannot have the same ID in a naming container. > Issues in change persistence framework code > ------------------------------------------- > > Key: ADFFACES-466 > URL: https://issues.apache.org/jira/browse/ADFFACES-466 > Project: MyFaces ADF-Faces > Issue Type: Bug > Affects Versions: 2.0.0-incubating-plugins-SNAPSHOT, > 2.0.0-incubating-core-SNAPSHOT, 1.0.1-incubating-plugins-SNAPSHOT, > 1.0.1-incubating-core-SNAPSHOT > Reporter: K SRINATH REDDY > Priority: Critical > Fix For: 2.0.0-incubating-plugins-SNAPSHOT, > 2.0.0-incubating-core-SNAPSHOT, 1.0.1-incubating-plugins-SNAPSHOT, > 1.0.1-incubating-core-SNAPSHOT > > Attachments: trunk.patch, trunk_latest.patch > > > 1. The datastructures used in the SessionChangeManager are not synchronized > 2. The _getUniqueIdForComponent() method in BaseChangeManager doesn't return > the unique ID in the component tree instead it returns the unique id in the > containing NamingContainer > 3. In the AddChildComponentChange, if the child with the already existing ID > is added, the child is being removed and the new child is added which is > faulty. > 4. The changes are applied in each of the restoreView phase of the lifecycle > which is not needed and also impacts the performance. the changes must be > applied only on the creation -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.