Bug 32676

Summary: EDI message status uses varying case, breaking EDI status block
Product: Koha Reporter: Kyle M Hall (khall) <kyle>
Component: AcquisitionsAssignee: Kyle M Hall (khall) <kyle>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: normal    
Priority: P5 - low CC: fridolin.somers, martin.renvoize, tomascohen, victor
Version: Main   
Hardware: All   
OS: All   
URL: https://github.com/bywatersolutions/bywater-koha-devel/tree/bug_32676
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
23.11.00,23.05.05,22.11.11
Circulation function:
Attachments: Bug 32676: EDI message status uses varying case, breaking EDI status block
Bug 32676: EDI message status uses varying case, breaking EDI status block

Description Kyle M Hall (khall) 2023-01-18 19:08:50 UTC
Edifact status appear to vary between all lower case and being capitalized. This breaks the switch in the edi_status block of the basket.tt template. We should make this switch case-insensitive.
Comment 1 Kyle M Hall (khall) 2023-01-18 19:10:53 UTC
Created attachment 145405 [details] [review]
Bug 32676: EDI message status uses varying case, breaking EDI status block

Edifact status appear to vary between all lower case and being capitalized. This breaks the switch in the edi_status block of the basket.tt template. We should make this switch case-insensitive.

Test Plan:
1) View the basket for a sent EDI order, not the Sent status does not
   display.
2) Apply this patch
3) Reload the page, note the status is now visible!
Comment 2 Martin Renvoize (ashimema) 2023-01-27 15:42:34 UTC
Created attachment 145758 [details] [review]
Bug 32676: EDI message status uses varying case, breaking EDI status block

Edifact status appear to vary between all lower case and being capitalized.
This breaks the switch in the edi_status block of the basket.tt template.
 We should make this switch case-insensitive.

Test Plan:
1) View the basket for a sent EDI order, not the Sent status does not
   display.
2) Apply this patch
3) Reload the page, note the status is now visible!

Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>
Comment 3 Martin Renvoize (ashimema) 2023-01-27 15:43:21 UTC
All works as expected, nice clean fix.

Going straight for QA here.

Passing.
Comment 4 Tomás Cohen Arazi (tcohen) 2023-01-27 19:09:19 UTC
Please rebase.
Comment 5 Kyle M Hall (khall) 2023-01-31 17:52:05 UTC
I've been hit with the git-bz unicode error again, so here's the patch: https://github.com/bywatersolutions/bywater-koha-devel/tree/bug_32676
Comment 6 Tomás Cohen Arazi (tcohen) 2023-02-03 13:14:56 UTC
Please rebase.
Comment 7 Kyle M Hall (khall) 2023-09-20 16:07:16 UTC
(In reply to Tomás Cohen Arazi from comment #6)
> Please rebase.

Rebased! Code is available at:
https://github.com/bywatersolutions/bywater-koha-devel/tree/bug_32676
Comment 8 Victor Grousset/tuxayo 2023-09-26 15:04:49 UTC
(should have been switched back to PQA instead of Signed Off)
Comment 9 Tomás Cohen Arazi (tcohen) 2023-10-04 13:44:53 UTC
Pushed to master for 23.11.

Nice work everyone, thanks!
Comment 10 Fridolin Somers 2023-10-06 00:25:51 UTC
Pushed to 23.05.x for 23.05.05
Comment 11 Matt Blenkinsop 2023-10-17 17:24:24 UTC
Nice work everyone!

Pushed to oldstable for 22.11.x