asset.copy_vis_attr_cache upgrade script includes deleted items

[Jason Boyer]

Eg 3.0

With bug 1698206 we can drop staged search thanks due in part to the asset.copy_vis_attr_cache table. Unfortunately, the upgrade script forgot to exclude deleted copies when generating the table. The triggers that maintain the table are fine but the deleted copies need to be removed to prevent "phantom" search results.

The presence of an entry in this table can cause inaccurate "X of Y copies available at $location" in search results when copies were deleted before the upgrade script was run. This is especially annoying when *all* copies were deleted and users are looking for titles that don't exist.

Quick branch incoming.

[Jason Boyer (jboyer)]

Branching out:
http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/jboyer/lp1731281-acvac-cleanup

Testing:
make sure there's at least one entry in the asset.copy_vis_attr_cache for an item that has been deleted attached to a record that the owning library no longer holds any copies of. A search for that title at that library will return the record with no holdings and the message "1 of 1 available" (numbers depending on how many deleted copies from that ou were attached.)

After running script, the record above will no longer appear in search results scoped to locations where all remaining copies were deleted.

[Galen Charlton (gmc)]

Tested and pushed to master and rel_3_0. Thanks, Jason!