Comment 19 for bug 1464709

IMO, "Can be checked out" should not be the same thing as "Can show up on the pull list". "On Holds Shelf" is a perfect example of that, in fact. You can check the copy out, but it shouldn't be on the pull list.

I see the holdable flag as covering "Can be on the *potentials* list". As this covers whether or not there are possible copies for a hold, already placed or otherwise, this is important to allow even for things you don't want on the pull list right now, but will be available to fill the hold later. This covers things like "On Order" and "In Process" that shouldn't be checked out or on the pull list, but can be on the potentials list.

Based on that, I personally think that we should either keep the hold targeter hardcoded to 0 and 7 or we should have a flag for "this status makes a copy eligible for the pull list if it is also a potential copy".