Bug 20780

Summary: EDI: Add support for 'AcqItemSetSubfieldsWhenReceived'
Product: Koha Reporter: Martin Renvoize <martin.renvoize>
Component: AcquisitionsAssignee: Martin Renvoize <martin.renvoize>
Status: CLOSED FIXED QA Contact: Colin Campbell <colin.campbell>
Severity: normal    
Priority: P4 CC: colin.campbell, fiona.borthwick, janet.mcgowan, jonathan.druart, katrin.fischer
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
EDIFACT receipting of items should now respect the `AcqItemSetSubfieldsWhenReceived` system preference
Version(s) released in:
19.11.00
Bug Depends on: 22348    
Bug Blocks:    
Attachments: Bug 20780 - Add handling for AcqItemSetSubfieldsWhenRecieved in EDI
Bug 20780 - Add handling for AcqItemSetSubfieldsWhenRecieved in EDI
Bug 20780 - Add handling for AcqItemSetSubfieldsWhenRecieved in EDI
Bug 20780: (follow-up) Use Koha::Object
Bug 20780 - Add handling for AcqItemSetSubfieldsWhenRecieved in EDI
Bug 20780: (follow-up) Use Koha::Object
Bug 20780: Add handling for AcqItemSetSubfieldsWhenRecieved in EDI
Bug 20780: (follow-up) Use Koha::Object

Description Martin Renvoize 2018-05-18 13:50:16 UTC
EDI, as it stands, does not respect the AcqItemSetSubfieldsWhenRecieved system preference.. it should ;)
Comment 1 Martin Renvoize 2018-05-18 15:57:17 UTC
Created attachment 75439 [details] [review]
Bug 20780 - Add handling for AcqItemSetSubfieldsWhenRecieved in EDI
Comment 2 Martin Renvoize 2018-05-18 16:34:18 UTC
Created attachment 75440 [details] [review]
Bug 20780 - Add handling for AcqItemSetSubfieldsWhenRecieved in EDI
Comment 3 Jonathan Druart 2018-10-17 18:19:19 UTC
* + $schema->resultset('Aqorder') [...]

=> We have Koha::Acquisition::Order

* Tests?
Comment 4 Martin Renvoize 2018-10-31 14:44:47 UTC
Created attachment 81730 [details] [review]
Bug 20780 - Add handling for AcqItemSetSubfieldsWhenRecieved in EDI
Comment 5 Martin Renvoize 2018-10-31 14:44:51 UTC
Created attachment 81731 [details] [review]
Bug 20780: (follow-up) Use Koha::Object
Comment 6 Martin Renvoize 2018-10-31 14:45:52 UTC
Moved to Koha::Object (reason for not doing so before was consistency.. the rest of Koha::EDI uses dbic directly)

No tests.. yup.. because it would entail writing tests for all of Koha::EDI which currently don't exist.
Comment 7 Brendan Gallagher 2019-10-04 07:54:19 UTC
Created attachment 93672 [details] [review]
Bug 20780 - Add handling for AcqItemSetSubfieldsWhenRecieved in EDI

Signed-off-by: Brendan Gallagher <brendan@bywatersolutions.com>
Comment 8 Brendan Gallagher 2019-10-04 07:54:32 UTC
Created attachment 93673 [details] [review]
Bug 20780: (follow-up) Use Koha::Object

Signed-off-by: Brendan Gallagher <brendan@bywatersolutions.com>
Comment 9 Katrin Fischer 2019-10-10 05:55:44 UTC
Created attachment 93946 [details] [review]
Bug 20780: Add handling for AcqItemSetSubfieldsWhenRecieved in EDI

Signed-off-by: Brendan Gallagher <brendan@bywatersolutions.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 10 Katrin Fischer 2019-10-10 05:55:47 UTC
Created attachment 93947 [details] [review]
Bug 20780: (follow-up) Use Koha::Object

Signed-off-by: Brendan Gallagher <brendan@bywatersolutions.com>

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 11 Katrin Fischer 2019-10-10 05:56:06 UTC
Passing QA on this as agreed with RM - both parties heavily using EDIFACT have
been involved in the QA process. Tests and QA script pass.
Comment 12 Martin Renvoize 2019-10-10 07:25:01 UTC
Nice work!

Pushed to master for 19.11.00