https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23415 --- Comment #97 from Emmi Takkinen <emmi.takkinen@koha-suomi.fi> --- (In reply to Martin Renvoize from comment #96)
This is really close.. but.. May I ask why you chose to handle this at the controller level rather than adding/moving the check into CanBookBeRenewed?
I see you do handle it in the _CanBookBeAutoRenewed version, but I think I am confused that it's not there in parent function.
Hmm, do you mean why there is a line my
$fine_no_renewals = C4::Context->preference("FineNoRenewals"); (and everything after that line) in _CanBookBeAutoRenewed? Those have been added in different bug, I just renamed syspref from "OPACFineNoRenewals" -> "FineNoRenewals".
-- You are receiving this mail because: You are watching all bug changes.