http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7408 --- Comment #10 from Kyle M Hall <kyle.m.hall@gmail.com> 2012-02-23 18:37:00 UTC --- (In reply to comment #7) I have updated my patch to address that issue. The benefit of this version is there are no worries about cronjob conflicts. larryb also tested the cronjob version and felt that is was not working quite right. I don't know the specifics of what that meant though.
Kyle,
Your patch does not include a provision for charging the patron for not picking up the hold, like the cronjob-based patch did, so it doesn't seem like a sufficient replacement (but, of course, I'm biased here). Reimplementing this to include not only a change to a C4 subroutine, but also a new syspref and consequential DB rev, seems to me to be a less worthwhile tradeoff.
I open this for discussion, in the hopes others will be able to provide a more balanced opinion than my own.
-- Configure bugmail: http://bugs.koha-community.org/bugzilla3/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are watching all bug changes.