Claus Niesen added a comment - 08/Apr/14 8:46 AM Tested on KRAD Demo's Travel Account maintenance document and KRMS' Context maintenance document.

Sona Sona (Inactive) added a comment - 08/Apr/14 2:49 PM Context,Term and Termspecification copy are broken. Travel Account works fine. // Since this will be a new object, we also need to blank out the object ID and version number fields // ( if they exist). If the object uses a different locking key or unique ID field, the blanking of // these will need to be done in the Maintainable.processAfterCopy() method. if ( maintainable.getDataObject() instanceof DataObjectBase ) { ((DataObjectBase) maintainable.getDataObject()).setObjectId( null ); ((DataObjectBase) maintainable.getDataObject()).setVersionNumber( null ); } else if ( maintainable.getDataObject() instanceof PersistableBusinessObject ) { // Legacy KNS Support - since they don't use DataObjectBase ((PersistableBusinessObject) maintainable.getDataObject()).setObjectId( null ); ((PersistableBusinessObject) maintainable.getDataObject()).setVersionNumber( null ); } On copy the version number and objectId are set to null since its a new object, but since none of the above objects extend DataObjectBase or PersistableBusinessObject this was not happening for them and so there was an OptimisticLockException being thrown in the background causing them to not go Final. Added a fix for that else { // If neither then use reflection to see if the object has setVersionNumber and setObjectId methods if (ObjectPropertyUtils.getWriteMethod(maintainable.getDataObject().getClass(), "versionNumber" ) != null ) { ObjectPropertyUtils.setPropertyValue(maintainable.getDataObject(), "versionNumber" , null ); } if (ObjectPropertyUtils.getWriteMethod(maintainable.getDataObject().getClass(), "objectId" ) != null ) { ObjectPropertyUtils.setPropertyValue(maintainable.getDataObject(), "objectId" , null ); }

Sona Sona (Inactive) added a comment - 08/Apr/14 2:54 PM Will reopen if there are changes from the review