Summary: | 'Reserve' should be passed through as transfer reason appropriately in branchtransfers | ||
---|---|---|---|
Product: | Koha | Reporter: | Martin Renvoize (ashimema) <martin.renvoize> |
Component: | Circulation | Assignee: | Martin Renvoize (ashimema) <martin.renvoize> |
Status: | CLOSED FIXED | QA Contact: | Joonas Kylmälä <joonas.kylmala> |
Severity: | normal | ||
Priority: | P5 - low | CC: | fridolin.somers, gmcharlt, joonas.kylmala, kyle.m.hall, kyle, nick, 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.01
|
|
Circulation function: | |||
Bug Depends on: | 27064 | ||
Bug Blocks: | 28520 | ||
Attachments: |
Bug 28382: Set trigger to 'Reserve' when actioning a resver transfer
Bug 28382: Set trigger to 'Reserve' when actioning a resver transfer Bug 28382: Set trigger to 'Reserve' when actioning a reserve transfer |
Description
Martin Renvoize (ashimema)
2021-05-19 08:30:09 UTC
Created attachment 121145 [details] [review] Bug 28382: Set trigger to 'Reserve' when actioning a resver transfer This patch sets the transfer trigger to 'Reserve' when the staff user confirms they are transfering to fulfil a reserve. Test plan 1 - Place a hold for delivery at Library B 2 - Sign in at Library A 3 - Browse to Circulation->Transfers 4 - Attempt to transfer an item on the title with the hold to Library B 5 - You get a notice that hold was found 6 - You have the option transfer the for the hold - click it 7 - Confirm hold is set to transit and transfer generated 8 - Check the branchtransfers table and confirm the 'reason' is 'Reserve' not 'Manual'. Created attachment 121152 [details] [review] Bug 28382: Set trigger to 'Reserve' when actioning a resver transfer This patch sets the transfer trigger to 'Reserve' when the staff user confirms they are transfering to fulfil a reserve. Test plan 1 - Place a hold for delivery at Library B 2 - Sign in at Library A 3 - Browse to Circulation->Transfers 4 - Attempt to transfer an item on the title with the hold to Library B 5 - You get a notice that hold was found 6 - You have the option transfer the for the hold - click it 7 - Confirm hold is set to transit and transfer generated 8 - Check the branchtransfers table and confirm the 'reason' is 'Reserve' not 'Manual'. Signed-off-by: David Nind <david@davidnind.com> Can/Should we turn "Reserve" into "hold" to follow terminology guidelines? It's pre-existing, and it's a code that translates in templates to a suitable string.. I'm not too worried either way really.. it might be nice, but at the same time I think it should be done on it's own bug..? (In reply to Martin Renvoize from comment #4) > It's pre-existing, and it's a code that translates in templates to a > suitable string.. I'm not too worried either way really.. it might be nice, > but at the same time I think it should be done on it's own bug..? I was nore sure if it was introduced here or reused - if it's preexisting I completely agree that it's out of scope here. Created attachment 121724 [details] [review] Bug 28382: Set trigger to 'Reserve' when actioning a reserve transfer This patch sets the transfer trigger to 'Reserve' when the staff user confirms they are transfering to fulfil a reserve. Test plan 1 - Place a hold for delivery at Library B 2 - Sign in at Library A 3 - Browse to Circulation->Transfers 4 - Attempt to transfer an item on the title with the hold to Library B 5 - You get a notice that hold was found 6 - You have the option transfer the for the hold - click it 7 - Confirm hold is set to transit and transfer generated 8 - Check the branchtransfers table and confirm the 'reason' is 'Reserve' not 'Manual'. Signed-off-by: David Nind <david@davidnind.com> JK: fix typo resver -> reserve Signed-off-by: Joonas Kylmälä <joonas.kylmala@helsinki.fi> PQA and switching status to bug because the reason was not right previously. Pushed to master for 21.11, thanks to everybody involved! Pushed to 21.05.x for 21.05.01 Depends on Bug 27064 not in 20.11.x |