I have been looking at this, and I see that Josh's changes work when the --items option is used.
I've experimented a bit to exclude MARC records without opac visible items even when the --items option isn't used. It seems to work, but I think it needs to take into account records with located (or opac visible) URIs.
If I'm off base, let me know before I spend too much time on it.
Seeing this code again, makes me want to refactor the bulk of it into a reusable Perl module and then just have marc_export be a command line wrapper around that library. Oh well, that's a different wishlist bug.
I have been looking at this, and I see that Josh's changes work when the --items option is used.
I've experimented a bit to exclude MARC records without opac visible items even when the --items option isn't used. It seems to work, but I think it needs to take into account records with located (or opac visible) URIs.
If I'm off base, let me know before I spend too much time on it.
Seeing this code again, makes me want to refactor the bulk of it into a reusable Perl module and then just have marc_export be a command line wrapper around that library. Oh well, that's a different wishlist bug.