diff options
| author | Johan Demaret Rivarola <jde@odoo.com> | 2019-03-11 14:15:13 +0000 |
|---|---|---|
| committer | Johan Demaret Rivarola <jde@odoo.com> | 2019-03-12 15:17:29 +0000 |
| commit | 9d3fed6d3fc20bf7f8ff999790e8386a76bcbeef (patch) | |
| tree | f9313718e4782a510d1649ec5a4f9df9e8586068 | |
| parent | b8d4fe6d386768a9b1b638358bc95a4a7575e3f9 (diff) | |
[FIX] payment: wire transfer redirection
When sequentially paying multiple SO with wire transfer, the user was
always redirected to the first paid SO.
This commit fixes this issue by redirecting to the most recent one.
Technical choice:
payment_transaction rows are ordered by id desc so we can trust the
first one will be the most recent on the client side (in the scope of
this diff).
opw-1949629
Closes #31754
Signed-off-by: Toufik Benjaa (tbe) <tbe@odoo.com>
| -rw-r--r-- | addons/payment/static/src/js/payment_processing.js | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/addons/payment/static/src/js/payment_processing.js b/addons/payment/static/src/js/payment_processing.js index 3497297d665..4397bf134f1 100644 --- a/addons/payment/static/src/js/payment_processing.js +++ b/addons/payment/static/src/js/payment_processing.js @@ -81,14 +81,16 @@ odoo.define('payment.processing', function (require) { 'tx_cancel': [], 'tx_error': [], }; + + if (transactions.length > 0 && transactions[0].acquirer_provider == 'transfer') { + window.location = transactions[0].return_url; + return; + } + // group the transaction according to their state transactions.forEach(function (tx) { var key = 'tx_' + tx.state; if(key in render_values) { - if (tx.acquirer_provider === 'transfer') { - window.location = tx.return_url; - return; - } render_values[key].push(tx); } }); |
