Comment 31 for bug 907056

Another day, another rebase and commit.

I've squashed most of my commits together, as the meandering of the code won't help someone who's trying to follow the evolution of this later on.

Tonight's commits add the following exciting properties:

1) (64d55f94a) A preferred library, if set and not identical to the search library, gets its copies (if any) pushed to the top of the list of copies in search results and record details. Only that library's copies get moved up; children of the preferred library are not affected, so as to prevent preferred library copies from preventing any of the chosen search library copies from being displayed in search results.

2) (0ff2680d08) A fix for the incorrect syntax for filtering non-opac-visible org units that ended up displaying all org units in copy results. I've kept this as a separate commit in case it gets applied independently in the hopes that it makes merging easier...

Note that sorting of call numbers by most available copy status is restricted to search results, as the number of copies to display is limited. Search result copies are sorted by: preferred library, search library depth, library name, call number copy availability, call number label, copy availability.

Record detail copies are sorted by: preferred library, search library depth, library name, call number label, copy availability.

user/dbs/unapi_improve_limits_and_tpac_display in working