Summary: | Add support for circulation status 1 ( other ) for damaged items | ||
---|---|---|---|
Product: | Koha | Reporter: | Kyle M Hall (khall) <kyle> |
Component: | SIP2 | Assignee: | Kyle M Hall (khall) <kyle> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | minor | ||
Priority: | P5 - low | CC: | fridolin.somers, martin.renvoize, victor |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | --- |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
21.11.00,21.05.02,20.11.09
|
|
Circulation function: | |||
Bug Depends on: | 27907 | ||
Bug Blocks: | |||
Attachments: |
Bug 27908: Add unit test
Bug 27908: Add unit test Bug 27908: Add support for circulation status 1 ( other ) for damaged items Bug 27908: Add unit test Bug 27908: Add support for circulation status 1 ( other ) for damaged items Bug 27908: Add unit test Bug 27908: Add support for circulation status 1 ( other ) for damaged items |
Description
Kyle M Hall (khall)
2021-03-09 18:29:11 UTC
Created attachment 118031 [details] [review] Bug 27908: Add unit test Created attachment 118074 [details] [review] Bug 27908: Add unit test Created attachment 118075 [details] [review] Bug 27908: Add support for circulation status 1 ( other ) for damaged items Some libraries would like to transmit if an item is damaged via the circulation status field. There is no specific code for a damaged item, so we should use value 1 (other) along with an AF screen message to describe the reason for the the circulation status of 1. Test Plan: 1) Apply these patches 2) prove t/db_dependent/SIP/Transaction.t Created attachment 119991 [details] [review] Bug 27908: Add unit test Signed-off-by: Christopher Kellermeyer - Altadena Library District <ckellermeyer@altadenalibrary.org> Created attachment 119992 [details] [review] Bug 27908: Add support for circulation status 1 ( other ) for damaged items Some libraries would like to transmit if an item is damaged via the circulation status field. There is no specific code for a damaged item, so we should use value 1 (other) along with an AF screen message to describe the reason for the the circulation status of 1. Test Plan: 1) Apply these patches 2) prove t/db_dependent/SIP/Transaction.t Signed-off-by: Christopher Kellermeyer - Altadena Library District <ckellermeyer@altadenalibrary.org> Created attachment 122733 [details] [review] Bug 27908: Add unit test Signed-off-by: Christopher Kellermeyer - Altadena Library District <ckellermeyer@altadenalibrary.org> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Created attachment 122734 [details] [review] Bug 27908: Add support for circulation status 1 ( other ) for damaged items Some libraries would like to transmit if an item is damaged via the circulation status field. There is no specific code for a damaged item, so we should use value 1 (other) along with an AF screen message to describe the reason for the the circulation status of 1. Test Plan: 1) Apply these patches 2) prove t/db_dependent/SIP/Transaction.t Signed-off-by: Christopher Kellermeyer - Altadena Library District <ckellermeyer@altadenalibrary.org> Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> All good again here.. Passing QA Pushed to master for 21.11, thanks to everybody involved! Pushed to 21.05.x for 21.05.02 Pushed to 20.11.x for 20.11.09 Not backported to oldoldstable (20.05.x). Feel free to ask if it's needed. |