https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=23415
--- Comment #97 from Emmi Takkinen <[email protected]> --- (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. _______________________________________________ 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/
