[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 --- Comment #36 from Hans Pålsson --- It would be of great value to many Swedish libraries if this could be backported to 22.05.x (if possible also 21.11.x). Thanks everyone for the great work! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Hans Pålsson changed: What|Removed |Added CC||hans.pals...@hkr.se -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Andrew Isherwood changed: What|Removed |Added Blocks||31592 Referenced Bugs: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=31592 [Bug 31592] Fix ILL API following new date_due property -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Lucas Gass changed: What|Removed |Added Status|Pushed to master|Needs documenting -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Lucas Gass changed: What|Removed |Added CC||lu...@bywatersolutions.com --- Comment #35 from Lucas Gass --- Enhancement will not be backported to 22.05.x series unless requested -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Katrin Fischer changed: What|Removed |Added See Also||https://bugs.koha-community ||.org/bugzilla3/show_bug.cgi ||?id=9799 -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 --- Comment #34 from Tomás Cohen Arazi --- Pushed to master for 22.11. Nice work everyone, thanks! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Tomás Cohen Arazi changed: What|Removed |Added Status|Passed QA |Pushed to master Version(s)||22.11.00 released in|| -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 --- Comment #33 from Tomás Cohen Arazi --- Created attachment 136463 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=136463=edit Bug 24239: (QA follow-up) Make dbrev idempotent Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 --- Comment #32 from Tomás Cohen Arazi --- Created attachment 136462 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=136462=edit Bug 24239: DBIC update Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 --- Comment #31 from Tomás Cohen Arazi --- Created attachment 136461 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=136461=edit Bug 24239: (QA follow-up) Rename date_due => due_date Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Tomás Cohen Arazi changed: What|Removed |Added Attachment #135945|0 |1 is obsolete|| --- Comment #30 from Tomás Cohen Arazi --- Created attachment 136460 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=136460=edit Bug 24239: Let the ILL module set ad hoc hard due dates The Swedish Libris ILL backend lets librarians store a specific due date when an ILL loan is received. This patch set adds a new date_due column to the illrequets table that can be used by the different backends to store a due date. If an illrequest has the date due set, it will be used when the item is checked out instead of the calculation using the circulation conditions. To test: - Apply the patch and make sure the atomic database update is run - Use the FreeForm backend to add one ILL request. Take note of the illrequest_id of the request you created. We refer to this as "x" below. - Connect a biblio (with biblionumber y), that has an item with a barcode, to the ILL request directly in the database: UPDATE illrequests SET biblio_id = y WHERE illrequest_id = x; - Next we set the due date, this would normally be done by or from the backend. UPDATE illrequests SET date_due = "2023-01-01" WHERE illrequest_id = x; - Go to circulation and issue the barcode of the item to the patron associated with the FreeForm ILL request. Verify that the loan gets a due date of 2023-01-01. - Ideally: return the item and issue it again through SIP2 and SCO, and verify that the due date is still 2023-01-01. - Verify that there are no regressions, so that regular calculation of due dates still work. - prove t/db_dependent/Circulation.t (Patch description, test plan and partial code credits to Magnus Enger) Signed-off-by: Katrin Fischer (Patch description and test plan rewritten to reflect changes in development) Signed-off-by: Kyle M Hall Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Tomás Cohen Arazi changed: What|Removed |Added Attachment #135944|0 |1 is obsolete|| --- Comment #29 from Tomás Cohen Arazi --- Created attachment 136459 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=136459=edit Bug 24239: Unit tests Signed-off-by: Katrin Fischer Signed-off-by: Kyle M Hall Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Tomás Cohen Arazi changed: What|Removed |Added Attachment #135943|0 |1 is obsolete|| --- Comment #28 from Tomás Cohen Arazi --- Created attachment 136458 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=136458=edit Bug 24239: Add column illrequests.date_due Signed-off-by: Katrin Fischer Signed-off-by: Kyle M Hall Signed-off-by: Tomas Cohen Arazi -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Andreas Jonsson changed: What|Removed |Added CC||andreas.jons...@kreablo.se -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 --- Comment #27 from Kyle M Hall --- Created attachment 135945 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135945=edit Bug 24239: Let the ILL module set ad hoc hard due dates The Swedish Libris ILL backend lets librarians store a specific due date when an ILL loan is received. This patch set adds a new date_due column to the illrequets table that can be used by the different backends to store a due date. If an illrequest has the date due set, it will be used when the item is checked out instead of the calculation using the circulation conditions. To test: - Apply the patch and make sure the atomic database update is run - Use the FreeForm backend to add one ILL request. Take note of the illrequest_id of the request you created. We refer to this as "x" below. - Connect a biblio (with biblionumber y), that has an item with a barcode, to the ILL request directly in the database: UPDATE illrequests SET biblio_id = y WHERE illrequest_id = x; - Next we set the due date, this would normally be done by or from the backend. UPDATE illrequests SET date_due = "2023-01-01" WHERE illrequest_id = x; - Go to circulation and issue the barcode of the item to the patron associated with the FreeForm ILL request. Verify that the loan gets a due date of 2023-01-01. - Ideally: return the item and issue it again through SIP2 and SCO, and verify that the due date is still 2023-01-01. - Verify that there are no regressions, so that regular calculation of due dates still work. - prove t/db_dependent/Circulation.t (Patch description, test plan and partial code credits to Magnus Enger) Signed-off-by: Katrin Fischer (Patch description and test plan rewritten to reflect changes in development) Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 --- Comment #26 from Kyle M Hall --- Created attachment 135944 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135944=edit Bug 24239: Unit tests Signed-off-by: Katrin Fischer Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Kyle M Hall changed: What|Removed |Added Attachment #135244|0 |1 is obsolete|| Attachment #135245|0 |1 is obsolete|| Attachment #135246|0 |1 is obsolete|| --- Comment #25 from Kyle M Hall --- Created attachment 135943 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135943=edit Bug 24239: Add column illrequests.date_due Signed-off-by: Katrin Fischer Signed-off-by: Kyle M Hall -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Kyle M Hall changed: What|Removed |Added Status|Signed Off |Passed QA -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Kyle M Hall changed: What|Removed |Added QA Contact||k...@bywatersolutions.com CC||k...@bywatersolutions.com -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Katrin Fischer changed: What|Removed |Added Attachment #133477|0 |1 is obsolete|| --- Comment #24 from Katrin Fischer --- Created attachment 135246 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135246=edit Bug 24239: Let the ILL module set ad hoc hard due dates The Swedish Libris ILL backend lets librarians store a specific due date when an ILL loan is received. This patch set adds a new date_due column to the illrequets table that can be used by the different backends to store a due date. If an illrequest has the date due set, it will be used when the item is checked out instead of the calculation using the circulation conditions. To test: - Apply the patch and make sure the atomic database update is run - Use the FreeForm backend to add one ILL request. Take note of the illrequest_id of the request you created. We refer to this as "x" below. - Connect a biblio (with biblionumber y), that has an item with a barcode, to the ILL request directly in the database: UPDATE illrequests SET biblio_id = y WHERE illrequest_id = x; - Next we set the due date, this would normally be done by or from the backend. UPDATE illrequests SET date_due = "2023-01-01" WHERE illrequest_id = x; - Go to circulation and issue the barcode of the item to the patron associated with the FreeForm ILL request. Verify that the loan gets a due date of 2023-01-01. - Ideally: return the item and issue it again through SIP2 and SCO, and verify that the due date is still 2023-01-01. - Verify that there are no regressions, so that regular calculation of due dates still work. - prove t/db_dependent/Circulation.t (Patch description, test plan and partial code credits to Magnus Enger) Signed-off-by: Katrin Fischer (Patch description and test plan rewritten to reflect changes in development) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Katrin Fischer changed: What|Removed |Added Attachment #133476|0 |1 is obsolete|| --- Comment #23 from Katrin Fischer --- Created attachment 135245 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135245=edit Bug 24239: Unit tests Signed-off-by: Katrin Fischer -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Katrin Fischer changed: What|Removed |Added Attachment #133475|0 |1 is obsolete|| --- Comment #22 from Katrin Fischer --- Created attachment 135244 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=135244=edit Bug 24239: Add column illrequests.date_due Signed-off-by: Katrin Fischer -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Katrin Fischer changed: What|Removed |Added Status|Needs Signoff |Signed Off -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 --- Comment #21 from Katrin Fischer --- I believe the test plan requires updating! * As we now have a illrequests.date_due, we should probably update that in step 4. * There is no ILLUseExactDueDate system preference anymore. I am rewriting it accordingly. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Magnus Enger changed: What|Removed |Added Status|Patch doesn't apply |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 --- Comment #20 from Magnus Enger --- As far as I can see new patches were supplied after the status was set to "Patch doesn't apply", so I am setting this to "Needs signoff". -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Lari Taskula changed: What|Removed |Added Attachment #130416|0 |1 is obsolete|| --- Comment #19 from Lari Taskula --- Created attachment 133477 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=133477=edit Bug 24239: Let the ILL module set ad hoc hard due dates The Swedish Libris ILL backend lets librarians store a specific due date when an ILL loan is received. This is stored as a ILL request attribute. This patch adds a syspref that can take the name of the illrequestattribute used to store the specific date. If the syspref is set, and if an item is connected to an ILL request, the due date will be taken from the illrequestattribute, instead of being calculated in the regular way, based on patroncategory and itemtype. To test: - Apply the patch and make sure the atomic database update is run - Use the FreeForm backend to add one ILL request. Take note of the illrequest_id of the request you created. We refer to this as "x" below. - Connect a biblio (with biblionumber y), that has an item with a barcode, to the ILL request directly in the database: UPADTE illrequests SET biblio_id = y WHERE illrequest_id = x; - Add an attribute to the ILL request directly in the database: INSERT INTO illrequestattributes SET illrequest_id = x, type = 'illduedate', value = '2023-01-01'; - Enter 'illduedate' as a value for the ILLUseExactDueDate syspref. - Go to circulation and issue the barcode of the item to the patron associated with the FreeForm ILL request. Verify that the loan gets a due date of 2023-01-01. - Ideally: return the item and issue it again through SIP2 and SCO, and verify that the due date is still 2023-01-01. (I don't think the REST API supports issuing yet?) - Verify that there are no regressions, so that regular calculation of due dates still work, if ILLUseExactDueDate is empty or not. - prove t/db_dependent/Circulation.t (Patch description, test plan and partial code credits to Magnus Enger) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Lari Taskula changed: What|Removed |Added Attachment #130415|0 |1 is obsolete|| --- Comment #18 from Lari Taskula --- Created attachment 133476 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=133476=edit Bug 24239: Unit tests -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Lari Taskula changed: What|Removed |Added Attachment #130414|0 |1 is obsolete|| --- Comment #17 from Lari Taskula --- Created attachment 133475 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=133475=edit Bug 24239: Add column illrequests.date_due -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Tomás Cohen Arazi changed: What|Removed |Added Status|Needs Signoff |Patch doesn't apply --- Comment #16 from Tomás Cohen Arazi --- Please rebase! Willing to test it! -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 --- Comment #15 from Lari Taskula --- Created attachment 130416 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=130416=edit Bug 24239: Let the ILL module set ad hoc hard due dates The Swedish Libris ILL backend lets librarians store a specific due date when an ILL loan is received. This is stored as a ILL request attribute. This patch adds a syspref that can take the name of the illrequestattribute used to store the specific date. If the syspref is set, and if an item is connected to an ILL request, the due date will be taken from the illrequestattribute, instead of being calculated in the regular way, based on patroncategory and itemtype. To test: - Apply the patch and make sure the atomic database update is run - Use the FreeForm backend to add one ILL request. Take note of the illrequest_id of the request you created. We refer to this as "x" below. - Connect a biblio (with biblionumber y), that has an item with a barcode, to the ILL request directly in the database: UPADTE illrequests SET biblio_id = y WHERE illrequest_id = x; - Add an attribute to the ILL request directly in the database: INSERT INTO illrequestattributes SET illrequest_id = x, type = 'illduedate', value = '2023-01-01'; - Enter 'illduedate' as a value for the ILLUseExactDueDate syspref. - Go to circulation and issue the barcode of the item to the patron associated with the FreeForm ILL request. Verify that the loan gets a due date of 2023-01-01. - Ideally: return the item and issue it again through SIP2 and SCO, and verify that the due date is still 2023-01-01. (I don't think the REST API supports issuing yet?) - Verify that there are no regressions, so that regular calculation of due dates still work, if ILLUseExactDueDate is empty or not. - prove t/db_dependent/Circulation.t (Patch description, test plan and partial code credits to Magnus Enger) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 --- Comment #14 from Lari Taskula --- Created attachment 130415 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=130415=edit Bug 24239: Unit tests -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Lari Taskula changed: What|Removed |Added Attachment #130056|0 |1 is obsolete|| Attachment #130057|0 |1 is obsolete|| --- Comment #13 from Lari Taskula --- Created attachment 130414 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=130414=edit Bug 24239: Add column illrequests.date_due -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Lari Taskula changed: What|Removed |Added CC||lari.task...@hypernova.fi --- Comment #12 from Lari Taskula --- As Tomas proposed, I've added a new column illrequests.date_due and it now replaces the attribute/syspref introduced in Magnus's original patch. Populated by backend, and if defined, AddIssue will use it as the due date, but if left null (default behavior), default due date calculation will be used. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 --- Comment #11 from Lari Taskula --- Created attachment 130057 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=130057=edit Bug 24239: Replace syspref with illrequests column date_due This patch replaces system preference introduced in previous patch with a column "date_due" for table illrequests. To test: 1. prove t/db_dependent/Circulation.t Sponsored-by: imCode Partner AB -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Lari Taskula changed: What|Removed |Added Attachment #106409|0 |1 is obsolete|| --- Comment #10 from Lari Taskula --- Created attachment 130056 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=130056=edit Bug 24239: Let the ILL module set ad hoc hard due dates The Swedish Libris ILL backend lets librarians store a specific due date when an ILL loan is received. This is stored as a ILL request attribute. This patch adds a syspref that can take the name of the illrequestattribute used to store the specific date. If the syspref is set, and if an item is connected to an ILL request, the due date will be taken from the illrequestattribute, instead of being calculated in the regular way, based on patroncategory and itemtype. To test: - Apply the patch and make sure the atomic database update is run - Use the FreeForm backend to add one ILL request. Take note of the illrequest_id of the request you created. We refer to this as "x" below. - Connect a biblio (with biblionumber y), that has an item with a barcode, to the ILL request directly in the database: UPADTE illrequests SET biblio_id = y WHERE illrequest_id = x; - Add an attribute to the ILL request directly in the database: INSERT INTO illrequestattributes SET illrequest_id = x, type = 'illduedate', value = '2023-01-01'; - Enter 'illduedate' as a value for the ILLUseExactDueDate syspref. - Go to circulation and issue the barcode of the item to the patron associated with the FreeForm ILL request. Verify that the loan gets a due date of 2023-01-01. - Ideally: return the item and issue it again through SIP2 and SCO, and verify that the due date is still 2023-01-01. (I don't think the REST API supports issuing yet?) - Verify that there are no regressions, so that regular calculation of due dates still work, if ILLUseExactDueDate is empty or not. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 --- Comment #9 from Katrin Fischer --- (In reply to Tomás Cohen Arazi from comment #8) > What if... > we added a column to illrequests, such as 'ill_defined_due_date' and just > tell backend authors to put the data in there? It would be much simpler. I really like that, but I'd probably just name it due_date or date_due. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Tomás Cohen Arazi changed: What|Removed |Added CC||tomasco...@gmail.com --- Comment #8 from Tomás Cohen Arazi --- What if... we added a column to illrequests, such as 'ill_defined_due_date' and just tell backend authors to put the data in there? It would be much simpler. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 --- Comment #7 from Katrin Fischer --- Didn't get to tinker yet... Magnus, would you agree that having the itemnumber in the illrequests table as an optional column would be helpful for the workflows using items? I think it would make it much easier/less error prone to find the illrequest for features like the one proposed here. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 --- Comment #6 from Magnus Enger --- (In reply to Katrin Fischer from comment #5) > Would you mind me tinkering with that a bit? Not at all, tinker away! :-) -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 --- Comment #5 from Katrin Fischer --- (In reply to Magnus Enger from comment #4) > (In reply to Katrin Fischer from comment #3) > > I wonder, if you are still around for this Magnus? > > I am on holiday now, so I will probably not be able to look at it before > August. That's a little late for us :( Would you mind me tinkering with that a bit? I've been wondering about taking a different approach here and adding the itemnumber to the illrequest table as linking by biblionumber and borrowernumber seems not ideal. Would this work for you? -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 --- Comment #4 from Magnus Enger --- (In reply to Katrin Fischer from comment #3) > I wonder, if you are still around for this Magnus? I am on holiday now, so I will probably not be able to look at it before August. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 --- Comment #3 from Katrin Fischer --- I've rebased the patches and done a first code review. I like the concept of this as it's small and flexible. I wonder, if you are still around for this Magnus? I can spend some time on ILL developments right now as we are looking into a backend for our ILL workflow and this would help a great deal. Looking at the code it seems like providing no due date in the ill request would be problematic. I'd suggest to check if the date is set and then have a fallback to normal due_date calculation if it isn't. There is also a leftover warn. I can also sign-off, but I think for QA we'd need tests... -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Katrin Fischer changed: What|Removed |Added Attachment #96246|0 |1 is obsolete|| --- Comment #2 from Katrin Fischer --- Created attachment 106409 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=106409=edit Bug 24239: Let the ILL module set ad hoc hard due dates The Swedish Libris ILL backend lets librarians store a specific due date when an ILL loan is received. This is stored as a ILL request attribute. This patch adds a syspref that can take the name of the illrequestattribute used to store the specific date. If the syspref is set, and if an item is connected to an ILL request, the due date will be taken from the illrequestattribute, instead of being calculated in the regular way, based on patroncategory and itemtype. To test: - Apply the patch and make sure the atomic database update is run - Use the FreeForm backend to add one ILL request. Take note of the illrequest_id of the request you created. We refer to this as "x" below. - Connect a biblio (with biblionumber y), that has an item with a barcode, to the ILL request directly in the database: UPADTE illrequests SET biblio_id = y WHERE illrequest_id = x; - Add an attribute to the ILL request directly in the database: INSERT INTO illrequestattributes SET illrequest_id = x, type = 'illduedate', value = '2023-01-01'; - Enter 'illduedate' as a value for the ILLUseExactDueDate syspref. - Go to circulation and issue the barcode of the item to the patron associated with the FreeForm ILL request. Verify that the loan gets a due date of 2023-01-01. - Ideally: return the item and issue it again through SIP2 and SCO, and verify that the due date is still 2023-01-01. (I don't think the REST API supports issuing yet?) - Verify that there are no regressions, so that regular calculation of due dates still work, if ILLUseExactDueDate is empty or not. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 Magnus Enger changed: What|Removed |Added Patch complexity|--- |Small patch Status|NEW |Needs Signoff -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/
[Koha-bugs] [Bug 24239] Let the ILL module set ad hoc hard due dates
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=24239 --- Comment #1 from Magnus Enger --- Created attachment 96246 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=96246=edit Bug 24239 - Let the ILL module set ad hoc hard due dates The Swedish Libris ILL backend lets librarians store a specific due date when an ILL loan is received. This is stored as a ILL request attribute. This patch adds a syspref that can take the name of the illrequestattribute used to store the specific date. If the syspref is set, and if an item is connected to an ILL request, the due date will be taken from the illrequestattribute, instead of being calculated in the regular way, based on patroncategory and itemtype. To test: - Apply the patch and make sure the atomic database update is run - Use the FreeForm backend to add one ILL request. Take note of the illrequest_id of the request you created. We refer to this as "x" below. - Connect a biblio (with biblionumber y), that has an item with a barcode, to the ILL request directly in the database: UPADTE illrequests SET biblio_id = y WHERE illrequest_id = x; - Add an attribute to the ILL request directly in the database: INSERT INTO illrequestattributes SET illrequest_id = x, type = 'illduedate', value = '2023-01-01'; - Enter 'illduedate' as a value for the ILLUseExactDueDate syspref. - Go to circulation and issue the barcode of the item to the patron associated with the FreeForm ILL request. Verify that the loan gets a due date of 2023-01-01. - Ideally: return the item and issue it again through SIP2 and SCO, and verify that the due date is still 2023-01-01. (I don't think the REST API supports issuing yet?) - Verify that there are no regressions, so that regular calculation of due dates still work, if ILLUseExactDueDate is empty or not. -- You are receiving this mail because: You are watching all bug changes. ___ Koha-bugs mailing list Koha-bugs@lists.koha-community.org 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/