Browse Source

FIX : multiple calls to action on 'start transaction' click event

pull/124/head
Aurélien DUMAINE 9 years ago
committed by Alexis de Lattre
parent
commit
587f8f77c4
  1. 2
      pos_payment_terminal/static/src/css/pos_payment_terminal.css
  2. 45
      pos_payment_terminal/static/src/js/pos_payment_terminal.js

2
pos_payment_terminal/static/src/css/pos_payment_terminal.css

@ -1,4 +1,4 @@
.pos .payment-terminal-transaction-start button {
.payment-terminal-transaction-start button {
width: 150px; width: 150px;
height: 60px; height: 60px;
font-size: 18px; font-size: 18px;

45
pos_payment_terminal/static/src/js/pos_payment_terminal.js

@ -33,59 +33,22 @@ odoo.define('pos_payment_terminal.pos_payment_terminal', function (require) {
var data = {'amount' : line.get_amount(), var data = {'amount' : line.get_amount(),
'currency_iso' : currency_iso, 'currency_iso' : currency_iso,
'payment_mode' : line.cashregister.journal.payment_mode}; 'payment_mode' : line.cashregister.journal.payment_mode};
//alert(JSON.stringify(data));
//console.log(JSON.stringify(data));
this.message('payment_terminal_transaction_start', {'payment_info' : JSON.stringify(data)}); this.message('payment_terminal_transaction_start', {'payment_info' : JSON.stringify(data)});
}, },
}); });
// TODO : overload instead of redefine
// BUG : its works fine when redefine but calls 5 times the function is overload
screens.PaymentScreenWidget.include({ screens.PaymentScreenWidget.include({
render_paymentlines : function(){ render_paymentlines : function(){
this._super.apply(this, arguments); this._super.apply(this, arguments);
var self = this; var self = this;
this.$('.paymentlines-container').on('click','.payment-terminal-transaction-start',function(){
this.$('.paymentlines-container').unbind('click').on('click','.payment-terminal-transaction-start',function(event){
// Why this "on" thing links severaltime the button to the action if I don't use "unlink" to reset the button links before ?
//console.log(event.target);
self.pos.proxy.payment_terminal_transaction_start($(this).data('cid'), self.pos.currency.name); self.pos.proxy.payment_terminal_transaction_start($(this).data('cid'), self.pos.currency.name);
}); });
/*
var self = this;
var order = this.pos.get_order();
if (!order) {
return;
}
var lines = order.get_paymentlines();
var due = order.get_due();
var extradue = 0;
if (due && lines.length && due !== order.get_due(lines[lines.length-1])) {
extradue = due;
}
this.$('.paymentlines-container').empty();
var lines = $(QWeb.render('PaymentScreen-Paymentlines', {
widget: this,
order: order,
paymentlines: lines,
extradue: extradue,
}));
lines.on('click','.delete-button',function(){
self.click_delete_paymentline($(this).data('cid'));
});
lines.on('click','.paymentline',function(){
self.click_paymentline($(this).data('cid'));
});
lines.on('click','.payment-terminal-transaction-start',function(){
self.pos.proxy.payment_terminal_transaction_start($(this).data('cid'), self.pos.currency.name);
});
lines.appendTo(this.$('.paymentlines-container'));
*/
}, },
}); });

Loading…
Cancel
Save