Bug 19532: (QA follow-up) Fix fine calculation by inserting biblionumber
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Wed, 2 Mar 2022 14:33:06 +0000 (14:33 +0000)
committerFridolin Somers <fridolin.somers@biblibre.com>
Tue, 15 Mar 2022 08:45:51 +0000 (22:45 -1000)
commita3437fd9303dd57b1d5e0b2cad947b3523ec6b3f
treed9ef982f649d2c767953569c6ac0eaecc47ffd2e
parentb9ba0e131aa974fba10af3b296de29d0d2e73f17
Bug 19532: (QA follow-up) Fix fine calculation by inserting biblionumber

The fines cron job uses Getoverdues to pass issue info to CalcFine.
It took me a while to realize that the overdue hash does not contain
a biblionumber. When testing CalcFine, we pass an item hash that
does include one.

So what happened? $item->{biblionumber} is undefined when it comes from
Getoverdues and no recall overdue fine is calculated, only a regular one.

Simple fix (without any impact): Add a biblionumber to Getoverdues.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Tested with fines.pl: recall fine applied now.
Ran some Circulation and Overdues unit tests.
Signed-off-by: Fridolin Somers <fridolin.somers@biblibre.com>
C4/Overdues.pm