Remove Advanced Holds Options when part holds are expected

[Kathy Lussier]

Related to bug 1602419.

Metarecord and part holds don't mix. The above-referenced bug describes a series of steps where a user may inadvertently try to place a hold on a metarecord group when only part holds are appropriate.

Another scenario that will lead to a failure in holds placement is when:

1. The user clicks the place hold link on a record that only contains copies with parts.
2. The user clicks the "Advanced Hold Options" link, changing the hold to a metarecord hold.
3. If they're lucky and other records in the metarecord group contain unparted copies, the hold will succeed. But, if there are only part copies for the other records or this record is alone in the metarecord group, the hold will fail with a message saying "The system could not find any items to match this hold request"

To help prevent this dead end, I recommend removing the "Advanced Hold Options" link in cases where all copies on a record contain parts.

A working branch is forthcoming.

[Kathy Lussier (klussier)]

Working branch is available at http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/kmlussier/lp1604515-remove_advanced-hold-options-for-parts

[Mike Rylander (mrylander)]

Pushed to master for 2.11. Thanks, Kathy!