Bug 27413

Summary: Cannot add debarment with batch patron modification tool
Product: Koha Reporter: Joonas Kylmälä <joonas.kylmala>
Component: ToolsAssignee: Jonathan Druart <jonathan.druart>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: andrewfh, caroline.cyr-la-rose, david.roberts, fridolin.somers, jonathan.druart, victor
Version: MainKeywords: regression
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
21.05.00,20.11.02,20.05.08
Bug Depends on: 19793    
Bug Blocks:    
Attachments: Bug 27413: Restore debarred and debarredcomment to batch patron mod tools
Bug 27413: Restore debarred and debarredcomment to batch patron mod tools
Bug 27413: Restore debarred and debarredcomment to batch patron mod tools

Description Joonas Kylmälä 2021-01-12 10:34:28 UTC
The patch "Bug 19793: Add a field for email in batch patron modification" broke the patron batch tool so that it is not possible to add debarred or debarredcomment information with it to the patrons.

David could you please provide a fix for this? I'm happy to review it.
Comment 1 Jonathan Druart 2021-01-15 10:29:14 UTC
Added by 13552, removed by bug 19793...
Comment 2 Jonathan Druart 2021-01-15 10:29:36 UTC
Created attachment 115190 [details] [review]
Bug 27413: Restore debarred and debarredcomment to batch patron mod tools

It was added by bug 13552 but removed by 19793.

Test plan:
Update patron in a batch and confirm that you can modify the restriction
expiration and comment.
Comment 3 Joonas Kylmälä 2021-01-15 10:39:45 UTC
Created attachment 115191 [details] [review]
Bug 27413: Restore debarred and debarredcomment to batch patron mod tools

It was added by bug 13552 but removed by 19793.

Test plan:
Update patron in a batch and confirm that you can modify the restriction
expiration and comment.

Signed-off-by: Joonas Kylmälä <joonas.kylmala@helsinki.fi>
Comment 4 Katrin Fischer 2021-01-17 17:11:36 UTC
Created attachment 115227 [details] [review]
Bug 27413: Restore debarred and debarredcomment to batch patron mod tools

It was added by bug 13552 but removed by 19793.

Test plan:
Update patron in a batch and confirm that you can modify the restriction
expiration and comment.

Signed-off-by: Joonas Kylmälä <joonas.kylmala@helsinki.fi>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 5 Jonathan Druart 2021-01-19 15:24:23 UTC
Pushed to master for 21.05, thanks to everybody involved!
Comment 6 Fridolin Somers 2021-01-20 08:27:10 UTC
Pushed to 20.11.x for 20.11.02
Comment 7 Andrew Fuerste-Henry 2021-01-22 17:42:04 UTC
Pushed to 20.05.x for 20.05.08
Comment 8 Victor Grousset/tuxayo 2021-01-22 21:13:41 UTC
Missing dependencies for 19.11.x, it shouldn't be affected, no backport.