From e7337b1a56a3a285469a802a76e62a7461d5a6d2 Mon Sep 17 00:00:00 2001 From: Emily Lamancusa Date: Wed, 25 Jun 2025 10:14:14 -0400 Subject: [PATCH] Bug 39572: Improve EDIFACT order confirmation message Add EAN account information to EDIFACT order confirmation message To test: 1. Set EDIFACT system preference to Enable 2. Create at least 2 EANs, one with a description and one without a description i. Go to Acquisitions > Library EANs ii. Click "New EAN" and fill in the fields 3. Create an EDI account i. Go to Acquisitions > EDI accounts ii. Click "New account" iii. Click Submit (for testing purposes you don't need to fill in the fields) 4. Find or create a basket for the same vendor that's on the EDI account 5. Add an order to the basket from any source 6. In the menu bar at the top of the basket screen, click "Create EDIFACT order" and select an EAN --> Note that the confirmation page doesn't tell you which EAN was selected 7. Click "No, don't close" 8. Apply patch and restart_all 9. Repeat step 6 --> The confirmation page should now display both the basket and the selected EAN 10. Click "Cancel" --> Confirm that the basket is still open 11. Repeat step 6 12. Click "Yes, generate order and close basket" --> Confirm the basket is closed 13. Click "EDIFACT messages" in the sidebar menu --> Confirm that you see a message with type ORDERS for that basket Signed-off-by: David Nind --- acqui/basket.pl | 10 +++++++--- .../intranet-tmpl/prog/en/modules/acqui/basket.tt | 12 +++++++++--- 2 files changed, 16 insertions(+), 6 deletions(-) diff --git a/acqui/basket.pl b/acqui/basket.pl index 791acd3250..57621b4e7a 100755 --- a/acqui/basket.pl +++ b/acqui/basket.pl @@ -613,16 +613,20 @@ sub edi_close_and_order { } exit; } else { + + my $ean_description = $query->param('ean_description'); + my $ean_branch = $query->param('ean_branch'); + $template->param( edi_confirm => 1, booksellerid => $booksellerid, basketno => $basket->{basketno}, basketname => $basket->{basketname}, basketgroupname => $basket->{basketname}, + ean => $ean ? $ean : '', + ean_description => $ean_description ? $ean_description : '', + ean_branch => $ean_branch ? $ean_branch : '', ); - if ($ean) { - $template->param( ean => $ean ); - } } return; diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt index 49c9d98fd3..7bd2e38902 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/basket.tt @@ -150,6 +150,8 @@ class="btn btn-default btn-xs submit-form-link" href="#" data-ean="[% eanacct.ean | html %]" + data-ean_description="[% eanacct.description | html %]" + data-ean_branch="[% eanacct.branch.branchname | html %]" data-basketno="[% basketno | html %]" data-action="basket.pl" data-method="post" @@ -1015,7 +1017,11 @@
[% INCLUDE 'csrf-token.inc' %] -

Are you sure you want to generate an EDIFACT order and close basket [% basketname | html %]?

+

Are you sure you want to generate this EDIFACT order?

+
    +
  • An EDIFACT order will be generated for [% ean_branch | html %] ([% ean | html %]) [% IF ean_description %][% ean_description | html %][% END %]
  • +
  • The basket [% basketname | html %] will be closed
  • +
[% IF CAN_user_acquisition_group_manage %]

@@ -1028,11 +1034,11 @@ - +

- +
-- 2.39.5