From becb4dc6367330c1dd79afb0c94af508bbbf9750 Mon Sep 17 00:00:00 2001
From: wan <wan@odoo.com>
Date: Wed, 6 Mar 2019 15:10:04 +0000
Subject: [PATCH] [IMP] account: improve journal entries recording Task 1943019
 onchange should ensure that you don't have both a credit and a debit <>0
 prefill account/partner if the user is recording several lines a row with the
 same account/partner the journal should be set by default

closes odoo/odoo#31593

Signed-off-by: Quentin De Paoli (qdp) <qdp@openerp.com>
---
 addons/account/models/account_move.py | 25 +++++++++++++++++++++++++
 1 file changed, 25 insertions(+)

diff --git a/addons/account/models/account_move.py b/addons/account/models/account_move.py
index fde3db36f04..c1eac5dd307 100644
--- a/addons/account/models/account_move.py
+++ b/addons/account/models/account_move.py
@@ -22,6 +22,13 @@ class AccountMove(models.Model):
     _order = 'date desc, id desc'
     _inherit = ['mail.thread', 'mail.activity.mixin']
 
+    @api.model
+    def default_get(self, fields):
+        rec = super(AccountMove, self).default_get(fields)
+        if not rec.get('journal_id'):
+            rec.update({'journal_id': self.env['account.journal'].search([('type', '=', 'general'), ('company_id', '=', self.env.user.company_id.id)], limit=1).id})
+        return rec
+
     @api.multi
     @api.depends('name', 'state')
     def name_get(self):
@@ -497,6 +504,18 @@ class AccountMoveLine(models.Model):
         for line in self:
             line.recompute_tax_line = True
 
+    @api.onchange('debit')
+    def _onchange_debit(self):
+        self.ensure_one()
+        if self.debit != 0:
+            self.credit = 0
+
+    @api.onchange('credit')
+    def _onchange_credit(self):
+        self.ensure_one()
+        if self.credit != 0:
+            self.debit = 0
+
     @api.model_cr
     def init(self):
         """ change index on partner_id to a multi-column index on (partner_id, ref), the new index will behave in the
@@ -682,6 +701,12 @@ class AccountMoveLine(models.Model):
         for line in self.move_id.resolve_2many_commands(
                 'line_ids', self._context['line_ids'], fields=['credit', 'debit']):
             balance += line.get('debit', 0) - line.get('credit', 0)
+        if len(self._context['line_ids']) > 1:
+            lines = self.move_id.resolve_2many_commands('line_ids', self._context['line_ids'][-2:], fields=['partner_id', 'account_id'])
+            if lines[0].get('partner_id', False) == lines[1].get('partner_id', False):
+                rec.update({'partner_id': lines[0].get('partner_id', False)})
+            if lines[0].get('account_id', False) == lines[1].get('account_id', False):
+                rec.update({'account_id': lines[0].get('account_id', False)})
         if balance < 0:
             rec.update({'debit': -balance})
         if balance > 0: