<div dir="ltr"><div>Hello Jonathan,</div><div><br></div>I would say we should add holds only when the new issue is and syspref RoutingListAddReserves is on. Print list is bad action/time for that. The hold should be item level - as biblio level hold on serial is not really useful...<div><br></div><div>Regards</div><div>Josef<br></div></div><br><div class="gmail_quote"><div dir="ltr">po 13. 3. 2017 v 17:26 odesílatel Jonathan Druart <<a href="mailto:jonathan.druart@bugs.koha-community.org">jonathan.druart@bugs.koha-community.org</a>> napsal:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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" target="_blank">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" target="_blank">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>
_______________________________________________<br>
Koha-devel mailing list<br>
<a href="mailto:Koha-devel@lists.koha-community.org" target="_blank">Koha-devel@lists.koha-community.org</a><br>
<a href="http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel" rel="noreferrer" target="_blank">http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel</a><br>
website : <a href="http://www.koha-community.org/" rel="noreferrer" target="_blank">http://www.koha-community.org/</a><br>
git : <a href="http://git.koha-community.org/" rel="noreferrer" target="_blank">http://git.koha-community.org/</a><br>
bugs : <a href="http://bugs.koha-community.org/" rel="noreferrer" target="_blank">http://bugs.koha-community.org/</a></blockquote></div>