Comment 5 for bug 1779467

Responding to Mike's observations in comment #3, except points 3 & 4 that Elaine responded to:

1. This development does not affect marking an item lost nor marking an item missing pieces. Specifically it will affect in some way the following statuses:

* Damaged
* Missing
* Discard/Weed
* Bindery
* On Order
* ILL
* Cataloging
* Reserves

The client currently only exposes Damaged and Missing to staff in any meaningful way. This development will add Discard/Weed to the client. The other status have their own back end calls that could be used in the future.

2. The implementation that I am working on is that the changing of the status will fail unless the circulation is checked in. If that is a problem, I can limit that only to the Discard/Weed and Damaged statuses.

Currently, Evergreen only checks in an open circulation in the event of marking an item Damaged. I think that it should check in any outstanding circulations before changing the copy status to any of the above, but I'm willing to be shown that I am wrong.