Bug 31296

Summary: Add ability to disable demagnetizing items via SIP2 based on itemtypes
Product: Koha Reporter: Kyle M Hall <kyle>
Component: SIP2Assignee: Kyle M Hall <kyle>
Status: CLOSED FIXED QA Contact: Martin Renvoize <martin.renvoize>
Severity: enhancement    
Priority: P5 - low CC: lucas, martin.renvoize
Version: MainKeywords: release-notes-needed
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
22.11.00, 22.05.07
Bug Depends on: 26370    
Bug Blocks:    
Attachments: Bug 31296: Add ability to disable demagnetizing items via SIP2 based on itemtypes
Bug 31296: Add ability to disable demagnetizing items via SIP2 based on itemtypes
Bug 31296: Add ability to disable demagnetizing items via SIP2 based on itemtypes
Bug 31296: (QA follow-up) Add to debian templates
Bug 31296: (QA follow-up) Fix trivial spelling errors

Description Kyle M Hall 2022-08-05 11:12:36 UTC
Some libraries have certain item types that can only do in house checkouts via SIP self check machines. In these cases, the items should not be demagnetized since the items cannot leave the library.
Comment 1 Kyle M Hall 2022-08-05 11:15:19 UTC
Created attachment 138681 [details] [review]
Bug 31296: Add ability to disable demagnetizing items via SIP2 based on itemtypes

Some libraries have certain item types that can only do in house checkouts via SIP self check machines. In these cases, the items should not be demagnetized since the items cannot leave the library.

Test Plan:
1) Apply this patch
2) prove t/db_dependent/SIP/Message.t
Comment 2 Martin Renvoize 2022-08-05 12:49:28 UTC
Created attachment 138691 [details] [review]
Bug 31296: Add ability to disable demagnetizing items via SIP2 based on itemtypes

Some libraries have certain item types that can only do in house checkouts via SIP self check machines. In these cases, the items should not be demagnetized since the items cannot leave the library.

Test Plan:
1) Apply this patch
2) prove t/db_dependent/SIP/Message.t

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 3 Martin Renvoize 2022-08-05 12:53:40 UTC
Created attachment 138692 [details] [review]
Bug 31296: Add ability to disable demagnetizing items via SIP2 based on itemtypes

Some libraries have certain item types that can only do in house checkouts via SIP self check machines. In these cases, the items should not be demagnetized since the items cannot leave the library.

Test Plan:
1) Apply this patch
2) prove t/db_dependent/SIP/Message.t

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 4 Martin Renvoize 2022-08-05 12:53:45 UTC
Created attachment 138693 [details] [review]
Bug 31296: (QA follow-up) Add to debian templates

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 5 Martin Renvoize 2022-08-05 12:54:56 UTC
Going straight for QA on this one.

Works as expected and is covered by tests.. example configs also updated.

Passing QA.

Please also add it to the wiki page detailing all the SIP options ;P

I'd love to eventually see some of this stuff move to UI side configurations, but didn't want to hold this one up.
Comment 6 Tomás Cohen Arazi 2022-08-08 16:19:28 UTC
Created attachment 138842 [details] [review]
Bug 31296: (QA follow-up) Fix trivial spelling errors

Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io>
Comment 7 Tomás Cohen Arazi 2022-08-08 16:21:18 UTC
Pushed to master for 22.11.

Nice work everyone, thanks!
Comment 8 Lucas Gass 2022-09-29 17:21:37 UTC
Enhancement will not be backported to 22.05.x series
Comment 9 Lucas Gass 2022-11-22 20:20:16 UTC
Backported upon request to 22.05.x for upcoming 22.05.07 release