diff --git a/addons/point_of_sale/models/pos_order.py b/addons/point_of_sale/models/pos_order.py index ec9f38a9890..35cc1340c8b 100644 --- a/addons/point_of_sale/models/pos_order.py +++ b/addons/point_of_sale/models/pos_order.py @@ -332,6 +332,20 @@ class PosOrder(models.Model): move.sudo().post() return True + def _reconcile_payments(self): + for order in self: + aml = order.statement_ids.mapped('journal_entry_ids').mapped('line_ids') | order.account_move.line_ids | order.invoice_id.move_id.line_ids + aml = aml.filtered(lambda r: not r.reconciled and r.account_id.internal_type == 'receivable' and r.partner_id == order.partner_id.commercial_partner_id) + try: + aml.reconcile() + except: + # There might be unexpected situations where the automatic reconciliation won't + # work. We don't want the user to be blocked because of this, since the automatic + # reconciliation is introduced for convenience, not for mandatory accounting + # reasons. + _logger.error('Reconciliation did not work for order %s', order.name) + continue + def _default_session(self): return self.env['pos.session'].search([('state', '=', 'opened'), ('user_id', '=', self.env.uid)], limit=1) diff --git a/addons/point_of_sale/models/pos_session.py b/addons/point_of_sale/models/pos_session.py index 9f2e9ddddd6..3c51f279bb2 100644 --- a/addons/point_of_sale/models/pos_session.py +++ b/addons/point_of_sale/models/pos_session.py @@ -27,6 +27,8 @@ class PosSession(models.Model): if order.state not in ('paid'): raise UserError(_("You cannot confirm all orders of this session, because they have not the 'paid' status")) order.action_pos_order_done() + orders = session.order_ids.filtered(lambda order: order.state in ['invoiced', 'done']) + orders.sudo()._reconcile_payments() config_id = fields.Many2one( 'pos.config', string='Point of Sale',