Comment 2 for bug 1061622

Nice catch, Steve. Cherry picked to my 2.2.2 server and tested. I don't want to blame the patch, but I started getting errors querying the database in the Browse Hold Shelf after applying it. Most of the shelf loads, but about 5 rows stay in "Retrieving..." status after the network errors, but, for what it's worth the deleted copy on the shelf didn't display.

I rolled back to a virgin Holds.pm and didn't have any issues loading the full hold shelf, but, of course my deleted copy was there.

I think Thomas brings up a good point, though. If an item is on the hold shelf it shouldn't be deleted without notifying the patron.