From aed01394e296ccc087b74de2bbedf9e0dac2b3f6 Mon Sep 17 00:00:00 2001 From: Benjamin Dauvergne Date: Sat, 10 Oct 2020 09:01:42 +0200 Subject: [PATCH 6/7] lingo: pass reference_id to eopayment as orderid for unique item payment (#47506) --- combo/apps/lingo/views.py | 3 +++ tests/test_lingo_payment.py | 4 +++- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/combo/apps/lingo/views.py b/combo/apps/lingo/views.py index 5191da3f..0c681486 100644 --- a/combo/apps/lingo/views.py +++ b/combo/apps/lingo/views.py @@ -416,6 +416,9 @@ class PayMixin(object): if regie.can_pay_only_one_basket_item: item = (items or remote_items)[0] kwargs['subject'] = item.subject + # copy command reference / invoice number + if item.reference_id: + kwargs['orderid'] = item.reference_id # allow easy testing/use of backend specific keyword arguments EOPAYMENT_REQUEST_KWARGS_PREFIX = 'eopayment_request_kwargs_' if hasattr(item, 'request_data'): diff --git a/tests/test_lingo_payment.py b/tests/test_lingo_payment.py index 6e03b875..efc4a4ce 100644 --- a/tests/test_lingo_payment.py +++ b/tests/test_lingo_payment.py @@ -1569,7 +1569,8 @@ def mono_regie(regie): def test_successfull_items_can_pay_only_one_basket_item(app, basket_page, mono_regie, user): - item = BasketItem.objects.create(user=user, regie=mono_regie, amount=42, subject='foo item') + item = BasketItem.objects.create(user=user, regie=mono_regie, amount=42, subject='foo item', + reference_id='form-3-23') BasketItem.objects.create(user=user, regie=mono_regie, amount=84, subject='bar item') resp = login(app).get('/test_basket_cell/') @@ -1581,6 +1582,7 @@ def test_successfull_items_can_pay_only_one_basket_item(app, basket_page, mono_r # successful payment qs = urlparse.parse_qs(urlparse.urlparse(resp.location).query) assert qs['subject'][0] == 'foo item', 'item.subject was not provided to eopayment' + assert qs['orderid'][0] == 'form-3-23' args = {'transaction_id': qs['transaction_id'][0], 'signed': True, 'ok': True, 'reason': 'Paid'} with mock.patch('combo.utils.requests_wrapper.RequestsSession.request'): resp = app.get(get_url(True, 'lingo-callback', mono_regie), params=args) -- 2.28.0