The function is executed in opac-detail.pl from Biblio.pm, but the param never gets passed to opac-detail.tt, which asks for MARCISBNS before normalized_isbn. The lack of this parameter means that only the 1st ISBN is ever shown. The other ISBNs are suppressed. Just need to add "MARCISBNS => $marcisbnsarray," to opac-detail.pl around line 588. I would do this, but I'm embroiled in trying to get our first fully functional local version of 3.8 up and running.
Created attachment 10555 [details] [review] Bug 8314: Add MARCISBNs parameter to opac-detail.pl to show multiple isbns in OPAC Opac-detail.pl already runs GetMarcISBNs and stores them in an array where they sit doing nothing. It looks like the template parameter was just missed. It's a tough one to notice, since normalize_isbn will pick up the first isbn and show it while quietly suppressing the rest.
Created attachment 10556 [details] [review] Bug 8314 Add MARCISBNs parameter to opac-detail.pl to show multiple isbns in OPAC Signed-off-by: Chris Cormack <chris@bigballofwax.co.nz>
Created attachment 10645 [details] [review] Bug 8314 Add MARCISBNs parameter to opac-detail.pl to show multiple isbns in OPAC replace \t with 4 spaces
MARCISBNs was never passed to the template. This patch fixes this issues. Marking as Passed QA
Patch pushed (untested, I don't think I have a biblio with more than 1 isbn in my database)
Pushed to 3.8.x will be in 3.8.3