ramesth / truetrade

Automatically exported from code.google.com/p/truetrade
0 stars 0 forks source link

Support deleting instruments #35

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
In the instrument view, would like to support deleting an instrument along
with all of its data.  Should prompt for confirmation before deletion.

Original issue reported on code.google.com by tyrotra...@gmail.com on 23 May 2007 at 11:03

GoogleCodeExporter commented 8 years ago
This will also affect any strategies which may reference this instrument.

May be worth looking into adding referential integrity back into MySQL.

Original comment by tyrotra...@gmail.com on 23 May 2007 at 11:05

GoogleCodeExporter commented 8 years ago
I am looking at this, will post a patch in a day or two.

Original comment by damiaan....@econ.kuleuven.be on 29 May 2007 at 10:08

GoogleCodeExporter commented 8 years ago
I checked in some changes to Instrument.xml, PlatformDAO.java and 
InstrumentView.java
which support deletion.  However, the views don't fully synchronize, especially 
into
the Strategy view.  Not sure what the best way to handle this would be.

Damiaan, your idea of directly invoking refresh() on the other views does 
technically
work, but it violates the notion of separation of responsibilities.  I wonder 
if we
can use an event-based notification which would work better.  That way, each 
view can
listen to events and update itself as necessary.

Original comment by tyrotra...@gmail.com on 30 May 2007 at 11:18