SQL error for some cases with custom best-hold feature

[Lebbeous Fogle-Weekley]

For Evergreen 2.4+

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/senator/best-hold-sql-error

When you're using a best-hold order that includes htime or shtime, (such
as those with Holds-go-home) and if there are mutiple circs or transits
related to the copy in hand with certain characteristics, you can
trigger this bug. The end user at the staff client gets one of those
nasty "Network error" dialogs, and the error in the SQL logs is:

> ERROR: more than one row returned by a subquery used as an expression

Several subselects within SQL auxiliary queries can possibly return
multiple rows in situations where outer code requires that they return
only one or zero rows.

[Mike Rylander (mrylander)]

Picked to master for great justice. Thank, Lebbeous! (And thanks, Kathy, for testing!)