[Bug 14301] New: “No renewal before” does not accept 0 as parameter.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14301 Bug ID: 14301 Summary: “No renewal before” does not accept 0 as parameter. Change sponsored?: --- Product: Koha Version: 3.18 Hardware: All OS: All Status: NEW Severity: normal Priority: P5 - low Component: Circulation Assignee: koha-bugs@lists.koha-community.org Reporter: costalc@gmail.com QA Contact: testopia@bugs.koha-community.org CC: gmcharlt@gmail.com, kyle.m.hall@gmail.com CIRCULATION AND FINE RULES: “No renewal before” does not accept 0 as parameter. Setting this parameter to 0 could enable issue due date as the only date to renew issues Explanation attached. -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14301 --- Comment #1 from bondiurbano <costalc@gmail.com> --- Created attachment 39705 --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=39705&action=edit “No renewal before” -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14301 --- Comment #2 from bondiurbano <costalc@gmail.com> --- Tested in 3.18.* and 3.20 -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14301 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- See Also| |http://bugs.koha-community. | |org/bugzilla3/show_bug.cgi? | |id=14101 CC| |h.meissner.82@web.de, | |katrin.fischer@bsz-bw.de -- You are receiving this mail because: You are the assignee for the bug. You are watching all bug changes.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14301 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Version|3.18 |master --- Comment #3 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- This sounds similar to what we discussed on bug 14101 - linking both bugs together for now. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14301 --- Comment #4 from bondiurbano <costalc@gmail.com> --- (In reply to Katrin Fischer from comment #3)
This sounds similar to what we discussed on bug 14101 - linking both bugs together for now.
Hi Katrin! I'd like to contribute with a comment: As I understand the problem "no renewal before" is independent from "auto_renew". 1) First of all Koha should look for "auto_renew" conditions and apply them if they are true 2) THEN Koha should look for "no renewal before" conditions. "no renewal before" only forces patrons and staff to renew issues in a range of days previous of the issue due-date. Imagine an issue that dues today may 31. If "no renewal before" is set to 1 it means that renewals are allowed on days 31 and 30; if it is set to 2 renewals are allowed on days 31, 30 and 29. But in this schema, which is the current schema, it is not possible to set "no renewal before" to 0. Setting "no renewal before" to 0 would enable renewals to be update only for due-date. So, the problem resides in how Koha calculates the enabled range of dates to renew the issues. Therefore it is independent from "auto_renew". ( Overriding this function in staff intranet works fine now. ) -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14301 Holger Meißner <h.meissner.82@web.de> changed: What |Removed |Added ---------------------------------------------------------------------------- Resolution|--- |DUPLICATE Status|NEW |RESOLVED --- Comment #5 from Holger Meißner <h.meissner.82@web.de> --- *** This bug has been marked as a duplicate of bug 14101 *** -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=14301 --- Comment #6 from Holger Meißner <h.meissner.82@web.de> --- Hi bondiurbano, I have uploaded a patch to fix the problem with bug 14101. Please test and let me know if it works for you. I also uploaded a table to illustrate how automatic renewals and 'No renewal before' work at this moment. If you have any suggestions for further changes please comment. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org