summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJohan Demaret Rivarola <jde@odoo.com>2019-03-11 14:15:13 +0000
committerJohan Demaret Rivarola <jde@odoo.com>2019-03-12 15:17:29 +0000
commit9d3fed6d3fc20bf7f8ff999790e8386a76bcbeef (patch)
treef9313718e4782a510d1649ec5a4f9df9e8586068
parentb8d4fe6d386768a9b1b638358bc95a4a7575e3f9 (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.js10
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);
}
});