Gnats 4662: Crash Deleting redundantly represented Supplies in the
  Supplies Manager.

http://cadswes2.colorado.edu/internal/cgi-bin/gnats-query-single.pl?Number=4662

-----------------------------
QtAccounting/QtSplyMgrDlg.cpp
-----------------------------

It is a bug for the Supplies Manager to have more than one listview item
(row) for any single Supply.  But that can be seen in the model file for
Gnats 4630.  One particular supply (of about 52 Supplies in that model)
between two Passthrough Accounts (one on a Reach and another on a Diversion)
is shown TWICE.

When both of those Supply Items (for the same Supply) are deleted in a
single operation from the Supplies Manager, a CRASH results (on Windows
-- I haven't produced this crash on Solaris).

See the Supply Manager Snapshot bug4662.gif. The Supply name is
  "TRIB BLW CONF LCRA to DIVOBJ LCRA Offstream PTA Div".

This supply is not part of any Exchange configuration.  It is a "standalone"
(role) Supply.

At least, for now, that CRASH needs to be addressed.  The redundacy, while
incorrect, behaves reasonably, e.g. both items are updated when a change to
any of the the Supply's properties is effected (from either within or
outside of the Supplies Mananager dialog).

Crash protection was added in Prerel (5.0.5 future) and Builds (5.1).

Change in method:
  void QtSplyMgrDlg::deleteSupplies (cwSlist<Supply*> supplyList)

A list of Supplies (Supply Pointers) that are Deleted within the current
invocation of this method is maintained.  Any Supply which has already been
deleted is not deleted again.

---