<div dir="ltr"><div><div><div><div><div><div><div><div><div><div><div>Hi devs,<br><br></div>I found an obvious bug in serials/<a href="http://routing-preview.pl">routing-preview.pl</a>.<br></div>If the script is called with ok=1 (hum... yes...), the script will add or modify a hold placed on the bibliographic records existing in the routing list.<br></div>So from /serials/<a href="http://serials-collection.pl?subscriptionid=1">serials-collection.pl?subscriptionid=1</a>, click on "Print list" you get a pop-up with a print and close button.<br></div>When the pop-up is generated, we already have the ok=1 and the go through:<br>  66: my ($count2,@bibitems) = GetBiblioItemByBiblioNumber($biblio);<br></div>Then later:<br> 96 AddReserve($branch,$routing->{borrowernumber},$biblio,\@bibitems,$routing->{ranking}, undef, undef, $notes,$title);<br><br></div>The problem is that GetBiblioItemByBiblioNumber will always return only 1 value (for obvious reasons) and @bibitems will always be empty => reserve is placed at bib level<br><br></div>The thing is that this bug exists for ages (at least 2006, maybe always).<br></div>What is the expected behaviour?<br><br></div>No need to tell you that I was going to kick GetBiblioItemByBiblioNumber when I found that bug.<br><br></div>Cheers,<br></div>Jonathan<br></div>