Comment 7 for bug 1966995

Revision history for this message
Carol Witt (carolwitt) wrote :

I tested this patch at https://terran-testbox.gapines.org/. I added the following subfields in various combinations to field 856 in TCNs 50, 51, and 52 ($z already existed):

$nTerms governing access here
$3Materials specified here

Patron view shows the following:

856$z shows the subfield's text
856$3 shows the subfield's text
856$n shows the subfield's text
856$z and $n shows both subfields' text
856$z and $3 shows both subfields' text
856$n and $3 shows both subfields' text
856$z, $n, and $3 shows all subfields' text

Like the repeatable 856$z issue per Garry's comment, the same is true of the 856$n: it is a repeatable subfield; however, no 856$n is visible if an 856 has more than one $n. This seems to be a separate issue; the patch fixes the initial bug (and perhaps 1350827?).

I have tested this code and consent to signing off on it with my name, Carol Witt and my email address, <email address hidden>.