KeplerGraphFrame has much duplicated code
I looked in to why KeplerGraphFrame has so much duplicated code
and it looks like Comad uses KeplerGraphFrameExtension
From the 2010-11-04 comment that added this class:
// "Added an extension for a different Drag and Drop behavior.
// Currently this is only activated in Comad. Once an Actor is
// dragged from the Library over another actor on the canvas it
// will replace this actor and Ports are reconeected. Also
// Parameter values are used for parameters of the new actor if
// they have the same name.
// If an actor is dragged over a link this actor will be inserted
// there. Currently this only works in Comad, where all actors
// have an "input" and one "output" port."
The right thing is to add these extension points to the ptII repository
and remove the duplicated code.
#1 Updated by Christopher Brooks over 7 years ago
Moved org.kepler.gui.KeplerGraphFrameExtension to ptolemy.vergil.basic.BasicGraphFrameExtension.
Removed duplicate KeplerGraphFrame.delete() and paste() methods.
Reformatted KeplerGraphFrame so that is human readable.
This change will break Comad, the fix is to extend BasicGraphFrameExtension
instead of KeplerGraphFrameExtension.
ant change-to -Dsuite=comad
and there were no errors.