[CAY-2171] Undo db Entity Sync throws error

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 4.0.M4
• Fix Version/s: 4.0.B1
• Component/s: Modeler
• Assignee: Nikita Timofeev
• Reporter: ana benko
• Created: 13/Dec/16 08:22
• Updated: 10/Apr/17 14:14
• Resolved: 10/Apr/17 14:14

Description

1. Reverse Engineer db
2. Remove one DbEntity attribute
3. Run Migrate db Schema (to Model) and restore attribute
4. Run Undo
-> CayenneModeler Info
Version: 4.0.M4-SNAPSHOT
Build Date: Dec 06 2016 07:36:54
Exception:
=================================
java.lang.IllegalArgumentException: An attempt to override attribute 'salary_amount'
at org.apache.cayenne.map.Entity.addAttribute(Entity.java:145)
at org.apache.cayenne.map.DbEntity.addAttribute(DbEntity.java:233)
at org.apache.cayenne.modeler.action.CreateAttributeAction.createDbAttribute(CreateAttributeAction.java:170)
at org.apache.cayenne.modeler.undo.RemoveAttributeUndoableEdit.undo(RemoveAttributeUndoableEdit.java:121)
at javax.swing.undo.UndoManager.undoTo(Unknown Source)
at javax.swing.undo.UndoManager.undo(Unknown Source)
at org.apache.cayenne.modeler.undo.CayenneUndoManager.undo(CayenneUndoManager.java:84)
at org.apache.cayenne.modeler.action.UndoAction.performAction(UndoAction.java:57)
at org.apache.cayenne.modeler.util.CayenneAction.actionPerformed(CayenneAction.java:162)
at javax.swing.AbstractButton.fireActionPerformed(Unknown Source)
at javax.swing.AbstractButton$Handler.actionPerformed(Unknown Source)
at javax.swing.DefaultButtonModel.fireActionPerformed(Unknown Source)
at javax.swing.DefaultButtonModel.setPressed(Unknown Source)
at javax.swing.AbstractButton.doClick(Unknown Source)
at javax.swing.plaf.basic.BasicMenuItemUI.doClick(Unknown Source)
at javax.swing.plaf.basic.BasicMenuItemUI$Handler.mouseReleased(Unknown Source)
at java.awt.AWTEventMulticaster.mouseReleased(Unknown Source)
at java.awt.Component.processMouseEvent(Unknown Source)
at javax.swing.JComponent.processMouseEvent(Unknown Source)
at java.awt.Component.processEvent(Unknown Source)
at java.awt.Container.processEvent(Unknown Source)
at java.awt.Component.dispatchEventImpl(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.LightweightDispatcher.retargetMouseEvent(Unknown Source)
at java.awt.LightweightDispatcher.processMouseEvent(Unknown Source)
at java.awt.LightweightDispatcher.dispatchEvent(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Window.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
at java.awt.EventQueue.access$500(Unknown Source)
at java.awt.EventQueue$3.run(Unknown Source)
at java.awt.EventQueue$3.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue$4.run(Unknown Source)
at java.awt.EventQueue$4.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.ProtectionDomain$JavaSecurityAccessImpl.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue.dispatchEvent(Unknown Source)
at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.run(Unknown Source)

Issue Links

• links to: GitHub Pull Request #191