https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=15565
Victor Grousset/tuxayo <[email protected]> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |[email protected] Status|Needs Signoff |Signed Off --- Comment #259 from Victor Grousset/tuxayo <[email protected]> --- It works! :D === Is that expected that Holds allowed (total) isn't taken into account? my $lowestvalue = min( $maxreserves, $remaining_holds_for_record, $remaining_holds_allowed_today ); === I my 1st test, I had a situation where maxreserves 3 holds allowed (total) = 2 holds allowed (daily) = 2 holds per record (count) = 2 And when I ticked one tick, the alert came. The patron had 0 holds and so the record. I did a reset_all and couldn't reproduce. So there might be something wrong, in case this symptom gives you an hint. If not, let's move on I would say. Can't be blocked by a ghost bug that might not actually exist. === Took David's test plan and put it in the commit. And did the following changes: 7) Set the following circulation rules: ↓↓↓ 7) Set the following circulation rules one by one and test that each one alone can trigger the alert: 8) Perform the same above actions on the OPAC ↓↓↓ 8) Set the circulation rules and syspref all to 4+. Check some items and confirm the holds. 9) Perform the same above actions on the OPAC === oops, inverted two patches during a rebase. Anyway, still applies on top of master. -- You are receiving this mail because: You are watching all bug changes. _______________________________________________ Koha-bugs mailing list [email protected] https://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs website : http://www.koha-community.org/ git : http://git.koha-community.org/ bugs : http://bugs.koha-community.org/
