diff options
Diffstat (limited to 'addons/account/models/account_move.py')
| -rw-r--r-- | addons/account/models/account_move.py | 3605 |
1 files changed, 2805 insertions, 800 deletions
diff --git a/addons/account/models/account_move.py b/addons/account/models/account_move.py index b77793a6b3e..040f2bbeceb 100644 --- a/addons/account/models/account_move.py +++ b/addons/account/models/account_move.py @@ -1,354 +1,1903 @@ # -*- coding: utf-8 -*- -import time -from datetime import date -from collections import OrderedDict from odoo import api, fields, models, _ -from odoo.osv import expression from odoo.exceptions import RedirectWarning, UserError, ValidationError +from odoo.tools import float_is_zero, float_compare, safe_eval, date_utils from odoo.tools.misc import formatLang, format_date -from odoo.tools import float_is_zero, float_compare -from odoo.tools.safe_eval import safe_eval from odoo.addons import decimal_precision as dp -from lxml import etree -#---------------------------------------------------------- -# Entries -#---------------------------------------------------------- +from collections import OrderedDict +from datetime import date +from itertools import groupby +from stdnum.iso7064 import mod_97_10 +from itertools import zip_longest + +import json +import re +import logging + +_logger = logging.getLogger(__name__) + class AccountMove(models.Model): _name = "account.move" + _inherit = ['portal.mixin', 'mail.thread', 'mail.activity.mixin'] _description = "Journal Entries" _order = 'date desc, name 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.company.id)], limit=1).id}) - return rec + def _get_default_journal(self): + ''' Get the default journal. + It could either be passed through the context using the 'default_journal_id' key containing its id, + either be determined by the default type. + ''' + move_type = self._context.get('default_type', 'entry') + journal_type = 'general' + if move_type in self.get_sale_types(include_receipts=True): + journal_type = 'sale' + elif move_type in self.get_purchase_types(include_receipts=True): + journal_type = 'purchase' + + if self._context.get('default_journal_id'): + journal = self.env['account.journal'].browse(self._context['default_journal_id']) + + if move_type != 'entry' and journal.type != journal_type: + raise UserError(_("Cannot create an invoice of type %s with a journal having %s as type.") % (move_type, journal_type)) + else: + company_id = self._context.get('default_company_id', self.env.company.id) + domain = [('company_id', '=', company_id), ('type', '=', journal_type)] + + journal = None + if self._context.get('default_currency_id'): + currency_domain = domain + [('currency_id', '=', self._context['default_currency_id'])] + journal = self.env['account.journal'].search(currency_domain, limit=1) + + if not journal: + journal = self.env['account.journal'].search(domain, limit=1) + + if not journal: + error_msg = _('Please define an accounting miscellaneous journal in your company') + if journal_type == 'sale': + error_msg = _('Please define an accounting sale journal in your company') + elif journal_type == 'purchase': + error_msg = _('Please define an accounting purchase journal in your company') + raise UserError(error_msg) + return journal + + @api.model + def _get_default_currency(self): + ''' Get the default currency from either the journal, either the default journal's company. ''' + journal = self._get_default_journal() + return journal.currency_id or journal.company_id.currency_id + + @api.model + def _get_default_invoice_incoterm(self): + ''' Get the default incoterm for invoice. ''' + return self.env.company.incoterm_id + + # ==== Business fields ==== + name = fields.Char(string='Number', required=True, readonly=True, copy=False, default='/') + date = fields.Date(string='Date', required=True, index=True, readonly=True, + states={'draft': [('readonly', False)]}, + default=fields.Date.context_today) + ref = fields.Char(string='Reference', copy=False, readonly=True, + states={'draft': [('readonly', False)]}) + narration = fields.Text(string='Internal Note') + state = fields.Selection(selection=[ + ('draft', 'Unposted'), + ('posted', 'Posted'), + ('cancel', 'Cancelled') + ], string='Status', required=True, readonly=True, copy=False, tracking=True, + default='draft') + type = fields.Selection(selection=[ + ('entry', 'Journal Entry'), + ('out_invoice', 'Customer Invoice'), + ('out_refund', 'Customer Credit Note'), + ('in_invoice', 'Vendor Bill'), + ('in_refund', 'Vendor Credit Note'), + ('out_receipt', 'Sales Receipt'), + ('in_receipt', 'Purchase Receipt'), + ], String='Type', required=True, store=True, index=True, readonly=True, tracking=True, + default="entry") + to_check = fields.Boolean(string='To Check', default=False, + help='If this checkbox is ticked, it means that the user was not sure of all the related informations at the time of the creation of the move and that the move needs to be checked again.') + journal_id = fields.Many2one('account.journal', string='Journal', required=True, readonly=True, + states={'draft': [('readonly', False)]}, + domain=lambda self: [('company_id', '=', self.env.company.id)], + default=_get_default_journal) + company_id = fields.Many2one(string='Company', store=True, readonly=True, + related='journal_id.company_id') + company_currency_id = fields.Many2one(string='Company Currency', readonly=True, + related='journal_id.company_id.currency_id') + currency_id = fields.Many2one('res.currency', store=True, readonly=True, tracking=True, required=True, + states={'draft': [('readonly', False)]}, + string='Currency', + default=_get_default_currency) + line_ids = fields.One2many('account.move.line', 'move_id', string='Journal Items', copy=True, readonly=True, + states={'draft': [('readonly', False)]}) + partner_id = fields.Many2one('res.partner', readonly=True, tracking=True, + states={'draft': [('readonly', False)]}, + string='Customer/Vendor') + commercial_partner_id = fields.Many2one('res.partner', string='Commercial Entity', store=True, readonly=True, + compute='_compute_commercial_partner_id') + + # === Amount fields === + amount_untaxed = fields.Monetary(string='Untaxed Amount', store=True, readonly=True, tracking=True, + compute='_compute_amount') + amount_tax = fields.Monetary(string='Tax', store=True, readonly=True, + compute='_compute_amount') + amount_total = fields.Monetary(string='Total', store=True, readonly=True, + compute='_compute_amount', + inverse='_inverse_amount_total') + amount_residual = fields.Monetary(string='Amount Due', store=True, + compute='_compute_amount') + amount_untaxed_signed = fields.Monetary(string='Untaxed Amount Signed', store=True, readonly=True, + compute='_compute_amount') + amount_tax_signed = fields.Monetary(string='Tax Signed', store=True, readonly=True, + compute='_compute_amount') + amount_total_signed = fields.Monetary(string='Total Signed', store=True, readonly=True, + compute='_compute_amount') + amount_residual_signed = fields.Monetary(string='Amount Due Signed', store=True, + compute='_compute_amount') + amount_by_group = fields.Binary(string="Tax amount by group", + compute='_compute_invoice_taxes_by_group', + help="Technical field used by web_studio to allow an easy edition of the invoice report by drag/drop of the field. Return type: [(name, amount, base, formated amount, formated base)]") + + # ==== Cash basis feature fields ==== + tax_cash_basis_rec_id = fields.Many2one( + 'account.partial.reconcile', + string='Tax Cash Basis Entry of', + help="Technical field used to keep track of the tax cash basis reconciliation. " + "This is needed when cancelling the source: it will post the inverse journal entry to cancel that part too.") + + # ==== Auto-post feature fields ==== + auto_post = fields.Boolean(string='Post Automatically', default=False, + help='If this checkbox is ticked, this entry will be automatically posted at its date.') + + # ==== Reverse feature fields ==== + reversed_entry_id = fields.Many2one('account.move', string="Reverse entry", readonly=True, copy=False) + + # ========================================================= + # Invoice related fields + # ========================================================= + + # ==== Business fields ==== + fiscal_position_id = fields.Many2one('account.fiscal.position', string='Fiscal Position', readonly=True, + states={'draft': [('readonly', False)]}, + help="Fiscal positions are used to adapt taxes and accounts for particular customers or sales orders/invoices. " + "The default value comes from the customer.") + invoice_user_id = fields.Many2one('res.users', readonly=True, copy=False, tracking=True, + states={'draft': [('readonly', False)]}, + string='Salesperson', + default=lambda self: self.env.user) + invoice_payment_state = fields.Selection(selection=[ + ('not_paid', 'Not Paid'), + ('in_payment', 'In Payment'), + ('paid', 'paid') + ], string='Payment Status', store=True, readonly=True, copy=False, tracking=True, + compute='_compute_amount') + invoice_date = fields.Date(string='Invoice/Bill Date', readonly=True, index=True, copy=False, + states={'draft': [('readonly', False)]}, + help="Keep empty to use the current date") + invoice_date_due = fields.Date(string='Due Date', readonly=True, index=True, copy=False, + states={'draft': [('readonly', False)]}, + help="If you use payment terms, the due date will be computed automatically at the generation " + "of accounting entries. The Payment terms may compute several due dates, for example 50% " + "now and 50% in one month, but if you want to force a due date, make sure that the payment " + "term is not set on the invoice. If you keep the Payment terms and the due date empty, it " + "means direct payment.") + invoice_payment_ref = fields.Char(string='Payment Reference', index=True, copy=False, readonly=True, + states={'draft': [('readonly', False)]}, + help="The payment reference to set on journal items.") + invoice_sent = fields.Boolean(readonly=True, default=False, copy=False, + help="It indicates that the invoice has been sent.") + invoice_origin = fields.Char(string='Origin', readonly=True, tracking=True, + help="The document(s) that generated the invoice.") + invoice_payment_term_id = fields.Many2one('account.payment.term', string='Payment Terms', + readonly=True, states={'draft': [('readonly', False)]}, + help="If you use payment terms, the due date will be computed automatically at the generation " + "of accounting entries. If you keep the payment terms and the due date empty, it means direct payment. " + "The payment terms may compute several due dates, for example 50% now, 50% in one month.") + # /!\ invoice_line_ids is just a subset of line_ids. + invoice_line_ids = fields.One2many('account.move.line', 'move_id', string='Invoice lines', + copy=False, readonly=True, + domain=[('exclude_from_invoice_tab', '=', False)], + states={'draft': [('readonly', False)]}) + invoice_partner_bank_id = fields.Many2one('res.partner.bank', string='Bank Account', + help='Bank Account Number to which the invoice will be paid. A Company bank account if this is a Customer Invoice or Vendor Credit Note, otherwise a Partner bank account number.', + readonly=True, states={'draft': [('readonly', False)]}) + invoice_incoterm_id = fields.Many2one('account.incoterms', string='Incoterm', + default=_get_default_invoice_incoterm, + help='International Commercial Terms are a series of predefined commercial terms used in international transactions.') + + # ==== Payment widget fields ==== + invoice_outstanding_credits_debits_widget = fields.Text(groups="account.group_account_invoice", + compute='_compute_payments_widget_to_reconcile_info') + invoice_payments_widget = fields.Text(groups="account.group_account_invoice", + compute='_compute_payments_widget_reconciled_info') + invoice_has_outstanding = fields.Boolean(groups="account.group_account_invoice", + compute='_compute_payments_widget_to_reconcile_info') + + # ==== Vendor bill fields ==== + invoice_vendor_bill_id = fields.Many2one('account.move', store=False, + string='Vendor Bill', + help="Auto-complete from a past bill.") + invoice_source_email = fields.Char(string='Source Email', tracking=True) + invoice_vendor_display_name = fields.Char(compute='_compute_invoice_vendor_display_info', store=True) + invoice_vendor_icon = fields.Char(compute='_compute_invoice_vendor_display_info', store=False) + + # ==== Cash rounding fields ==== + invoice_cash_rounding_id = fields.Many2one('account.cash.rounding', string='Cash Rounding Method', + readonly=True, states={'draft': [('readonly', False)]}, + help='Defines the smallest coinage of the currency that can be used to pay by cash.') + + # ==== Fields to set the sequence, on the first invoice of the journal ==== + invoice_sequence_number_next = fields.Char(string='Next Number', + compute='_compute_invoice_sequence_number_next', + inverse='_inverse_invoice_sequence_number_next') + invoice_sequence_number_next_prefix = fields.Char(string='Next Number Prefix', + compute="_compute_invoice_sequence_number_next") + + # ==== Display purpose fields ==== + invoice_filter_type_domain = fields.Char(compute='_compute_invoice_filter_type_domain', + help="Technical field used to have a dynamic domain on journal / taxes in the form view.") + bank_partner_id = fields.Many2one('res.partner', help='Technical field to get the domain on the bank', compute='_compute_bank_partner_id') + invoice_has_matching_supsense_amount = fields.Boolean(compute='_compute_has_matching_suspense_amount', + groups='account.group_account_invoice', + help="Technical field used to display an alert on invoices if there is at least a matching amount in any supsense account.") + + # ------------------------------------------------------------------------- + # ONCHANGE METHODS + # ------------------------------------------------------------------------- + + @api.onchange('invoice_date') + def _onchange_invoice_date(self): + if self.invoice_date: + self.invoice_date_due = self.date = self.invoice_date + self._onchange_currency() + + @api.onchange('invoice_date_due') + def _onchange_invoice_date_due(self): + self._recompute_dynamic_lines() + + @api.onchange('journal_id') + def _onchange_journal(self): + if self.journal_id and self.journal_id.currency_id: + new_currency = self.journal_id.currency_id + if new_currency != self.currency_id: + self.currency_id = new_currency + self._onchange_currency() + + @api.onchange('partner_id') + def _onchange_partner_id(self): + warning = {} + if self.partner_id: + rec_account = self.partner_id.property_account_receivable_id + pay_account = self.partner_id.property_account_payable_id + if not rec_account and not pay_account: + action = self.env.ref('account.action_account_config') + msg = _('Cannot find a chart of accounts for this company, You should configure it. \nPlease go to Account Configuration.') + raise RedirectWarning(msg, action.id, _('Go to the configuration panel')) + p = self.partner_id + if p.invoice_warn == 'no-message' and p.parent_id: + p = p.parent_id + if p.invoice_warn and p.invoice_warn != 'no-message': + # Block if partner only has warning but parent company is blocked + if p.invoice_warn != 'block' and p.parent_id and p.parent_id.invoice_warn == 'block': + p = p.parent_id + warning = { + 'title': _("Warning for %s") % p.name, + 'message': p.invoice_warn_msg + } + if p.invoice_warn == 'block': + self.partner_id = False + return {'warning': warning} + for line in self.line_ids: + line.partner_id = self.partner_id.commercial_partner_id + if self.is_sale_document(include_receipts=True): + self.invoice_payment_term_id = self.partner_id.property_payment_term_id + elif self.is_purchase_document(include_receipts=True): + self.invoice_payment_term_id = self.partner_id.property_supplier_payment_term_id + + self._compute_bank_partner_id() + self.invoice_partner_bank_id = self.bank_partner_id.bank_ids and self.bank_partner_id.bank_ids[0] + + # Find the new fiscal position. + delivery_partner_id = self._get_invoice_delivery_partner_id() + new_fiscal_position_id = self.env['account.fiscal.position'].get_fiscal_position( + self.partner_id.id, delivery_id=delivery_partner_id) + self.fiscal_position_id = self.env['account.fiscal.position'].browse(new_fiscal_position_id) + self._recompute_dynamic_lines() + if warning: + return {'warning': warning} + + @api.onchange('date', 'currency_id') + def _onchange_currency(self): + company_currency = self.company_id.currency_id + has_foreign_currency = self.currency_id and self.currency_id != company_currency + + for line in self.line_ids: + new_currency = has_foreign_currency and self.currency_id + line.currency_id = new_currency + line._onchange_currency() + self._recompute_dynamic_lines() + + @api.onchange('invoice_payment_ref') + def _onchange_invoice_payment_ref(self): + for line in self.line_ids.filtered(lambda line: line.account_id.user_type_id.type in ('receivable', 'payable')): + line.name = self.invoice_payment_ref + + @api.onchange('invoice_vendor_bill_id') + def _onchange_invoice_vendor_bill(self): + if self.invoice_vendor_bill_id: + # Copy invoice lines. + for line in self.invoice_vendor_bill_id.invoice_line_ids: + copied_vals = line.copy_data()[0] + copied_vals['move_id'] = self.id + new_line = self.env['account.move.line'].new(copied_vals) + new_line.recompute_tax_line = True + + # Copy payment terms. + self.invoice_payment_term_id = self.invoice_vendor_bill_id.invoice_payment_term_id + + # Copy currency. + if self.currency_id != self.invoice_vendor_bill_id.currency_id: + self.currency_id = self.invoice_vendor_bill_id.currency_id + + # Reset + self.invoice_vendor_bill_id = False + self._recompute_dynamic_lines() + + @api.onchange('type') + def _onchange_type(self): + ''' Onchange made to filter the partners depending of the type. ''' + if self.is_sale_document(include_receipts=True): + if self.env['ir.config_parameter'].sudo().get_param('account.use_invoice_terms'): + self.narration = self.company_id.invoice_terms or self.env.company.invoice_terms + return {'domain': {'partner_id': [('customer', '=', True)]}} + elif self.is_purchase_document(include_receipts=True): + return {'domain': {'partner_id': [('supplier', '=', True)]}} + + @api.onchange('invoice_line_ids') + def _onchange_invoice_line_ids(self): + current_invoice_lines = self.line_ids.filtered(lambda line: not line.exclude_from_invoice_tab) + others_lines = self.line_ids - current_invoice_lines + if others_lines and current_invoice_lines - self.invoice_line_ids: + others_lines[0].recompute_tax_line = True + self.line_ids = others_lines + self.invoice_line_ids + self._onchange_recompute_dynamic_lines() + + @api.onchange('line_ids', 'invoice_payment_term_id', 'invoice_date_due', 'invoice_cash_rounding_id', 'invoice_vendor_bill_id') + def _onchange_recompute_dynamic_lines(self): + self._recompute_dynamic_lines() + + @api.model + def _get_tax_grouping_key_from_tax_line(self, tax_line): + ''' Create the dictionary based on a tax line that will be used as key to group taxes together. + /!\ Must be consistent with '_get_tax_grouping_key_from_base_line'. + :param tax_line: An account.move.line being a tax line (with 'tax_repartition_line_id' set then). + :return: A dictionary containing all fields on which the tax will be grouped. + ''' + return { + 'tax_repartition_line_id': tax_line.tax_repartition_line_id.id, + 'account_id': tax_line.account_id.id, + 'currency_id': tax_line.currency_id.id, + 'analytic_tag_ids': [(6, 0, tax_line.tax_line_id.analytic and tax_line.analytic_tag_ids.ids or [])], + 'analytic_account_id': tax_line.tax_line_id.analytic and tax_line.analytic_account_id.id, + 'tax_ids': [(6, 0, tax_line.tax_ids.ids)], + 'tag_ids': [(6, 0, tax_line.tag_ids.ids)], + } + + @api.model + def _get_tax_grouping_key_from_base_line(self, base_line, tax_vals): + ''' Create the dictionary based on a base line that will be used as key to group taxes together. + /!\ Must be consistent with '_get_tax_grouping_key_from_tax_line'. + :param base_line: An account.move.line being a base line (that could contains something in 'tax_ids'). + :param tax_vals: An element of compute_all(...)['taxes']. + :return: A dictionary containing all fields on which the tax will be grouped. + ''' + tax_repartition_line = self.env['account.tax.repartition.line'].browse(tax_vals['tax_repartition_line_id']) + account = base_line._get_default_tax_account(tax_repartition_line) or base_line.account_id + return { + 'tax_repartition_line_id': tax_vals['tax_repartition_line_id'], + 'account_id': account.id, + 'currency_id': base_line.currency_id.id, + 'analytic_tag_ids': [(6, 0, base_line.analytic_tag_ids.ids)], + 'analytic_account_id': base_line.analytic_account_id.id, + 'tax_ids': [(6, 0, tax_vals['tax_ids'])], + 'tag_ids': [(6, 0, tax_vals['tag_ids'])], + } @api.multi - @api.depends('name', 'state') - def name_get(self): - result = [] - for move in self: - name = move.name - if self.env.context.get('name_groupby'): - name = "**{date}**, {name} {partner}".format(date=format_date(self.env, move.date), name=move.name, partner=move.partner_id.name or "") - if move.state == 'draft': - name = '* ' + str(move.id) - result.append((move.id, name)) - return result + def _recompute_tax_lines(self): + ''' Compute the dynamic tax lines of the journal entry. + + :param lines_map: The line_ids dispatched by type containing: + * base_lines: The lines having a tax_ids set. + * tax_lines: The lines having a tax_line_id set. + * terms_lines: The lines generated by the payment terms of the invoice. + * rounding_lines: The cash rounding lines of the invoice. + ''' + self.ensure_one() + in_draft_mode = self != self._origin + + def _serialize_tax_grouping_key(grouping_dict): + ''' Serialize the dictionary values to be used in the taxes_map. + :param grouping_dict: The values returned by '_get_tax_grouping_key_from_tax_line' or '_get_tax_grouping_key_from_base_line'. + :return: A string representing the values. + ''' + return '-'.join(str(v) for v in grouping_dict.values()) + + def _compute_base_line_taxes(base_line): + ''' Compute taxes amounts both in company currency / foreign currency as the ratio between + amount_currency & balance could not be the same as the expected currency rate. + The 'amount_currency' value will be set on compute_all(...)['taxes'] in multi-currency. + :param base_line: The account.move.line owning the taxes. + :return: The result of the compute_all method. + ''' + balance_taxes_res = base_line.tax_ids._origin.compute_all( + base_line.balance, + currency=base_line.company_currency_id, + partner=base_line.partner_id, + is_refund=self.type in ('out_refund', 'in_refund'), + handle_price_include=False, + ) + + if base_line.currency_id: + # Multi-currencies mode: Taxes are computed both in company's currency / foreign currency. + amount_currency_taxes_res = base_line.tax_ids._origin.compute_all( + base_line.amount_currency, + currency=base_line.currency_id, + partner=base_line.partner_id, + is_refund=self.type in ('out_refund', 'in_refund'), + handle_price_include=False, + ) + for b_tax_res, ac_tax_res in zip(balance_taxes_res['taxes'], amount_currency_taxes_res['taxes']): + b_tax_res['amount_currency'] = ac_tax_res['amount'] + return balance_taxes_res + + taxes_map = {} + + # ==== Add tax lines ==== + for line in self.line_ids.filtered('tax_repartition_line_id'): + grouping_dict = self._get_tax_grouping_key_from_tax_line(line) + grouping_key = _serialize_tax_grouping_key(grouping_dict) + taxes_map[grouping_key] = { + 'tax_line': line, + 'balance': 0.0, + 'amount_currency': 0.0, + 'tax_base_amount': 0.0, + 'grouping_dict': False, + } + + # ==== Mount base lines ==== + for line in self.line_ids.filtered(lambda line: not line.exclude_from_invoice_tab): + # Don't call compute_all if there is no tax. + if not line.tax_ids: + line.tag_ids = [(5, 0, 0)] + continue + + compute_all_vals = _compute_base_line_taxes(line) + + # Assign tags on base line + line.tag_ids = compute_all_vals['base_tags'] + + tax_exigible = True + for tax_vals in compute_all_vals['taxes']: + grouping_dict = self._get_tax_grouping_key_from_base_line(line, tax_vals) + grouping_key = _serialize_tax_grouping_key(grouping_dict) + + tax_repartition_line = self.env['account.tax.repartition.line'].browse(tax_vals['tax_repartition_line_id']) + tax = tax_repartition_line.invoice_tax_id or tax_repartition_line.refund_tax_id + + if tax.tax_exigibility == 'on_payment': + tax_exigible = False + + taxes_map_entry = taxes_map.setdefault(grouping_key, { + 'tax_line': None, + 'balance': 0.0, + 'amount_currency': 0.0, + 'tax_base_amount': 0.0, + 'grouping_dict': False, + }) + taxes_map_entry['balance'] += tax_vals['amount'] + taxes_map_entry['amount_currency'] += tax_vals.get('amount_currency', 0.0) + taxes_map_entry['tax_base_amount'] += tax_vals['base'] + taxes_map_entry['grouping_dict'] = grouping_dict + line.tax_exigible = tax_exigible + + # ==== Process taxes_map ==== + for taxes_map_entry in taxes_map.values(): + # Don't create tax lines with zero balance. + if self.currency_id.is_zero(taxes_map_entry['balance']) and self.currency_id.is_zero(taxes_map_entry['amount_currency']): + taxes_map_entry['grouping_dict'] = False + + tax_line = taxes_map_entry['tax_line'] + tax_base_amount = -taxes_map_entry['tax_base_amount'] if self.is_inbound() else taxes_map_entry['tax_base_amount'] + + if not tax_line and not taxes_map_entry['grouping_dict']: + continue + elif tax_line and not taxes_map_entry['grouping_dict']: + # The tax line is no longer used, drop it. + self.line_ids -= tax_line + elif tax_line: + tax_line.update({ + 'amount_currency': taxes_map_entry['amount_currency'], + 'debit': taxes_map_entry['balance'] > 0.0 and taxes_map_entry['balance'] or 0.0, + 'credit': taxes_map_entry['balance'] < 0.0 and -taxes_map_entry['balance'] or 0.0, + 'tax_base_amount': tax_base_amount, + }) + else: + create_method = in_draft_mode and self.env['account.move.line'].new or self.env['account.move.line'].create + tax_repartition_line_id = taxes_map_entry['grouping_dict']['tax_repartition_line_id'] + tax_repartition_line = self.env['account.tax.repartition.line'].browse(tax_repartition_line_id) + tax = tax_repartition_line.invoice_tax_id or tax_repartition_line.refund_tax_id + tax_line = create_method({ + 'name': tax.name, + 'move_id': self.id, + 'partner_id': line.partner_id.id, + 'company_id': line.company_id.id, + 'company_currency_id': line.company_currency_id.id, + 'quantity': 1.0, + 'date_maturity': False, + 'amount_currency': taxes_map_entry['amount_currency'], + 'debit': taxes_map_entry['balance'] > 0.0 and taxes_map_entry['balance'] or 0.0, + 'credit': taxes_map_entry['balance'] < 0.0 and -taxes_map_entry['balance'] or 0.0, + 'tax_base_amount': tax_base_amount, + 'exclude_from_invoice_tab': True, + 'tax_exigible': tax.tax_exigibility == 'on_invoice', + **taxes_map_entry['grouping_dict'], + }) + + if in_draft_mode: + tax_line._onchange_amount_currency() + tax_line._onchange_balance() @api.multi - @api.depends('line_ids.debit', 'line_ids.credit', 'line_ids.amount_currency', 'line_ids.currency_id') - def _amount_compute(self): - for move in self: - total = 0.0 - total_currency = 0.0 - currency_id = move.line_ids and move.line_ids[0].currency_id.id or False - for line in move.line_ids.filtered(lambda l: l.debit): - total += line.debit - if currency_id and line.currency_id.id == currency_id: - total_currency += line.amount_currency - elif currency_id: - currency_id = False - - if currency_id and total_currency: - move.amount = total_currency - move.currency_id = currency_id + def _recompute_cash_rounding_lines(self): + ''' Handle the cash rounding feature on invoices. + + In some countries, the smallest coins do not exist. For example, in Switzerland, there is no coin for 0.01 CHF. + For this reason, if invoices are paid in cash, you have to round their total amount to the smallest coin that + exists in the currency. For the CHF, the smallest coin is 0.05 CHF. + + There are two strategies for the rounding: + + 1) Add a line on the invoice for the rounding: The cash rounding line is added as a new invoice line. + 2) Add the rounding in the biggest tax amount: The cash rounding line is added as a new tax line on the tax + having the biggest balance. + ''' + self.ensure_one() + in_draft_mode = self != self._origin + + def _compute_cash_rounding(self, total_balance, total_amount_currency): + ''' Compute the amount differences due to the cash rounding. + :param self: The current account.move record. + :param total_balance: The invoice's total in company's currency. + :param total_amount_currency: The invoice's total in invoice's currency. + :return: The amount differences both in company's currency & invoice's currency. + ''' + if self.currency_id == self.company_id.currency_id: + diff_balance = self.invoice_cash_rounding_id.compute_difference(self.currency_id, total_balance) + diff_amount_currency = 0.0 + else: + diff_amount_currency = self.invoice_cash_rounding_id.compute_difference(self.currency_id, total_amount_currency) + diff_balance = self.currency_id._convert(diff_amount_currency, self.company_id.currency_id, self.company_id, self.date) + return diff_balance, diff_amount_currency + + def _apply_cash_rounding(self, diff_balance, diff_amount_currency, cash_rounding_line): + ''' Apply the cash rounding. + :param self: The current account.move record. + :param diff_balance: The computed balance to set on the new rounding line. + :param diff_amount_currency: The computed amount in invoice's currency to set on the new rounding line. + :param cash_rounding_line: The existing cash rounding line. + :return: The newly created rounding line. + ''' + rounding_line_vals = { + 'debit': diff_balance > 0.0 and diff_balance or 0.0, + 'credit': diff_balance < 0.0 and -diff_balance or 0.0, + 'quantity': 1.0, + 'amount_currency': diff_amount_currency, + 'partner_id': self.partner_id.id, + 'move_id': self.id, + 'currency_id': self.currency_id if self.currency_id != self.company_id.currency_id else False, + 'company_id': self.company_id.id, + 'company_currency_id': self.company_id.currency_id.id, + 'is_rounding_line': True, + 'sequence': 9999, + } + + if self.invoice_cash_rounding_id.strategy == 'biggest_tax': + biggest_tax_line = None + for tax_line in self.line_ids.filtered('tax_repartition_line_id'): + if not biggest_tax_line or tax_line.price_subtotal > biggest_tax_line.price_subtotal: + biggest_tax_line = tax_line + + # No tax found. + if not biggest_tax_line: + return + + rounding_line_vals.update({ + 'name': _('%s (rounding)') % biggest_tax_line.name, + 'account_id': biggest_tax_line.account_id.id, + 'tax_repartition_line_id': biggest_tax_line.tax_repartition_line_id.id, + 'tax_exigible': biggest_tax_line.tax_exigible, + 'exclude_from_invoice_tab': True, + }) + + elif self.invoice_cash_rounding_id.strategy == 'add_invoice_line': + rounding_line_vals.update({ + 'name': self.invoice_cash_rounding_id.name, + 'account_id': self.invoice_cash_rounding_id.account_id.id, + }) + + # Create or update the cash rounding line. + if cash_rounding_line: + cash_rounding_line.update({ + 'amount_currency': rounding_line_vals['amount_currency'], + 'debit': rounding_line_vals['debit'], + 'credit': rounding_line_vals['credit'], + }) + else: + create_method = in_draft_mode and self.env['account.move.line'].new or self.env['account.move.line'].create + cash_rounding_line = create_method(rounding_line_vals) + + if in_draft_mode: + cash_rounding_line._onchange_amount_currency() + cash_rounding_line._onchange_balance() + + existing_cash_rounding_line = self.line_ids.filtered(lambda line: line.is_rounding_line) + + # The cash rounding has been removed. + if not self.invoice_cash_rounding_id: + self.line_ids -= existing_cash_rounding_line + return + + # The cash rounding strategy has changed. + if self.invoice_cash_rounding_id and existing_cash_rounding_line: + strategy = self.invoice_cash_rounding_id.strategy + old_strategy = 'biggest_tax' if existing_cash_rounding_line.tax_line_id else 'add_invoice_line' + if strategy != old_strategy: + self.line_ids -= existing_cash_rounding_line + existing_cash_rounding_line = self.env['account.move.line'] + + others_lines = self.line_ids.filtered(lambda line: line.account_id.user_type_id.type not in ('receivable', 'payable')) + others_lines -= existing_cash_rounding_line + total_balance = sum(others_lines.mapped('balance')) + total_amount_currency = sum(others_lines.mapped('amount_currency')) + + diff_balance, diff_amount_currency = _compute_cash_rounding(self, total_balance, total_amount_currency) + + # The invoice is already rounded. + if self.currency_id.is_zero(diff_balance) and self.currency_id.is_zero(diff_amount_currency): + self.line_ids -= existing_cash_rounding_line + return + + _apply_cash_rounding(self, diff_balance, diff_amount_currency, existing_cash_rounding_line) + + @api.multi + def _recompute_payment_terms_lines(self): + ''' Compute the dynamic payment term lines of the journal entry.''' + self.ensure_one() + in_draft_mode = self != self._origin + today = fields.Date.context_today(self) + + def _get_payment_terms_computation_date(self): + ''' Get the date from invoice that will be used to compute the payment terms. + :param self: The current account.move record. + :return: A datetime.date object. + ''' + if self.invoice_payment_term_id: + return self.invoice_date or today else: - move.currency_id = move.company_id.currency_id or self.env.user.company_id.currency_id - move.amount = total + return self.invoice_date_due or self.invoice_date or today + + def _get_payment_terms_account(self, payment_terms_lines): + ''' Get the account from invoice that will be set as receivable / payable account. + :param self: The current account.move record. + :param payment_terms_lines: The current payment terms lines. + :return: An account.account record. + ''' + if payment_terms_lines: + # Retrieve account from previous payment terms lines in order to allow the user to set a custom one. + return payment_terms_lines[0].account_id + elif self.partner_id: + # Retrieve account from partner. + if self.is_sale_document(include_receipts=True): + return self.partner_id.property_account_receivable_id + else: + return self.partner_id.property_account_payable_id + else: + # Search new account. + domain = [ + ('company_id', '=', self.company_id.id), + ('internal_type', '=', 'receivable' if self.type in ('out_invoice', 'out_refund', 'out_receipt') else 'payable'), + ] + return self.env['account.account'].search(domain, limit=1) + + def _compute_payment_terms(self, date, total_balance, total_amount_currency): + ''' Compute the payment terms. + :param self: The current account.move record. + :param date: The date computed by '_get_payment_terms_computation_date'. + :param total_balance: The invoice's total in company's currency. + :param total_amount_currency: The invoice's total in invoice's currency. + :return: A list <to_pay_company_currency, to_pay_invoice_currency, due_date>. + ''' + if self.invoice_payment_term_id: + to_compute = self.invoice_payment_term_id.compute(total_balance, date_ref=date, currency=self.currency_id) + if self.currency_id != self.company_id.currency_id: + # Multi-currencies. + to_compute_currency = self.invoice_payment_term_id.compute(total_amount_currency, date_ref=date, currency=self.currency_id) + return [(b[0], b[1], ac[1]) for b, ac in zip(to_compute, to_compute_currency)] + else: + # Single-currency. + return [(b[0], b[1], 0.0) for b in to_compute] + else: + return [(fields.Date.to_string(date), total_balance, total_amount_currency)] + + def _compute_diff_payment_terms_lines(self, existing_terms_lines, account, to_compute): + ''' Process the result of the '_compute_payment_terms' method and creates/updates corresponding invoice lines. + :param self: The current account.move record. + :param existing_terms_lines: The current payment terms lines. + :param account: The account.account record returned by '_get_payment_terms_account'. + :param to_compute: The list returned by '_compute_payment_terms'. + ''' + # As we try to update existing lines, sort them by due date. + existing_terms_lines = existing_terms_lines.sorted(lambda line: line.date_maturity or today) + existing_terms_lines_index = 0 + + # Recompute amls: update existing line or create new one for each payment term. + new_terms_lines = self.env['account.move.line'] + for date_maturity, balance, amount_currency in to_compute: + if existing_terms_lines_index < len(existing_terms_lines): + # Update existing line. + candidate = existing_terms_lines[existing_terms_lines_index] + existing_terms_lines_index += 1 + candidate.update({ + 'date_maturity': date_maturity, + 'amount_currency': -amount_currency, + 'debit': balance < 0.0 and -balance or 0.0, + 'credit': balance > 0.0 and balance or 0.0, + }) + else: + # Create new line. + create_method = in_draft_mode and self.env['account.move.line'].new or self.env['account.move.line'].create + candidate = create_method({ + 'name': self.invoice_payment_ref or '', + 'debit': balance < 0.0 and -balance or 0.0, + 'credit': balance > 0.0 and balance or 0.0, + 'quantity': 1.0, + 'amount_currency': -amount_currency, + 'date_maturity': date_maturity, + 'move_id': self.id, + 'currency_id': self.currency_id.id if self.currency_id != self.company_id.currency_id else False, + 'account_id': account.id, + 'partner_id': self.commercial_partner_id.id, + 'exclude_from_invoice_tab': True, + }) + new_terms_lines += candidate + if in_draft_mode: + candidate._onchange_amount_currency() + candidate._onchange_balance() + return new_terms_lines + + existing_terms_lines = self.line_ids.filtered(lambda line: line.account_id.user_type_id.type in ('receivable', 'payable')) + others_lines = self.line_ids.filtered(lambda line: line.account_id.user_type_id.type not in ('receivable', 'payable')) + total_balance = sum(others_lines.mapped('balance')) + total_amount_currency = sum(others_lines.mapped('amount_currency')) + + if not others_lines: + self.line_ids -= existing_terms_lines + return + + computation_date = _get_payment_terms_computation_date(self) + account = _get_payment_terms_account(self, existing_terms_lines) + to_compute = _compute_payment_terms(self, computation_date, total_balance, total_amount_currency) + new_terms_lines = _compute_diff_payment_terms_lines(self, existing_terms_lines, account, to_compute) + + # Remove old terms lines that are no longer needed. + self.line_ids -= existing_terms_lines - new_terms_lines + + if new_terms_lines: + self.invoice_payment_ref = new_terms_lines[-1].name or '' + self.invoice_date_due = new_terms_lines[-1].date_maturity @api.multi - def _set_amount(self): + def _recompute_dynamic_lines(self, recompute_all_taxes=False): + ''' Recompute all lines that depend of others. + + For example, tax lines depends of base lines (lines having tax_ids set). This is also the case of cash rounding + lines that depend of base lines or tax lines depending the cash rounding strategy. When a payment term is set, + this method will auto-balance the move with payment term lines. + + :param recompute_all_taxes: Force the computation of taxes. If set to False, the computation will be done + or not depending of the field 'recompute_tax_line' in lines. + ''' + self.ensure_one() + + # Dispatch lines and pre-compute some aggregated values like taxes. + for line in self.line_ids: + if line.recompute_tax_line: + recompute_all_taxes = True + line.recompute_tax_line = False + + # Compute taxes. + if recompute_all_taxes: + self._recompute_tax_lines() + + if self.is_invoice(include_receipts=True): + + # Compute cash rounding. + self._recompute_cash_rounding_lines() + + # Compute payment terms. + self._recompute_payment_terms_lines() + + # Only synchronize one2many in onchange. + if self != self._origin: + self.invoice_line_ids = self.line_ids.filtered(lambda line: not line.exclude_from_invoice_tab) + + @api.multi + def onchange(self, values, field_name, field_onchange): + # OVERRIDE + # As the dynamic lines in this model are quite complex, we need to ensure some computations are done exactly + # at the beginning / at the end of the onchange mechanism. So, the onchange recursivity is disabled. + return super(AccountMove, self.with_context(recursive_onchanges=False)).onchange(values, field_name, field_onchange) + + # ------------------------------------------------------------------------- + # COMPUTE METHODS + # ------------------------------------------------------------------------- + + @api.depends('type') + def _compute_invoice_filter_type_domain(self): for move in self: - if len(move.line_ids) == 2 and move.amount != 0: - amount_in_company_currency = move.amount - if move.currency_id and move.currency_id != move.company_id.currency_id: - amount_in_company_currency = move.currency_id._convert(move.amount, move.company_id.currency_id, move.company_id, move.date) - for line in move.line_ids: - line.amount_currency = line.debit and move.amount or -move.amount - for line in move.with_context(check_move_validity=False).line_ids: - line.debit = line.debit and amount_in_company_currency or 0.0 - line.credit = line.credit and amount_in_company_currency or 0.0 - - @api.depends('line_ids.debit', 'line_ids.credit', 'line_ids.matched_debit_ids.amount', 'line_ids.matched_credit_ids.amount', 'line_ids.account_id.user_type_id.type') - def _compute_matched_percentage(self): - """Compute the percentage to apply for cash basis method. This value is relevant only for moves that - involve journal items on receivable or payable accounts. - """ + if move.is_sale_document(include_receipts=True): + move.invoice_filter_type_domain = 'sale' + elif move.is_purchase_document(include_receipts=True): + move.invoice_filter_type_domain = 'purchase' + else: + move.invoice_filter_type_domain = False + + @api.depends('partner_id') + def _compute_commercial_partner_id(self): + for move in self: + move.commercial_partner_id = move.partner_id.commercial_partner_id + + @api.depends('commercial_partner_id') + def _compute_bank_partner_id(self): + for move in self: + if move.is_outbound(): + move.bank_partner_id = move.commercial_partner_id + else: + move.bank_partner_id = move.company_id.partner_id + + @api.depends( + 'line_ids.debit', + 'line_ids.credit', + 'line_ids.currency_id', + 'line_ids.amount_currency', + 'line_ids.amount_residual', + 'line_ids.amount_residual_currency', + 'line_ids.payment_id.state') + def _compute_amount(self): + invoice_ids = [move.id for move in self if move.id and move.is_invoice(include_receipts=True)] + if invoice_ids: + self._cr.execute( + ''' + SELECT move.id + FROM account_move move + JOIN account_move_line line ON line.move_id = move.id + JOIN account_partial_reconcile part ON part.debit_move_id = line.id OR part.credit_move_id = line.id + JOIN account_move_line rec_line ON + (rec_line.id = part.credit_move_id AND line.id = part.debit_move_id) + OR + (rec_line.id = part.debit_move_id AND line.id = part.credit_move_id) + JOIN account_payment payment ON payment.id = rec_line.payment_id + JOIN account_journal journal ON journal.id = rec_line.journal_id + WHERE payment.state IN ('posted', 'sent') + AND journal.post_at_bank_rec IS TRUE + AND move.id IN %s + ''', [tuple(invoice_ids)] + ) + in_payment_set = set(res[0] for res in self._cr.fetchall()) + else: + in_payment_set = {} + for move in self: - total_amount = 0.0 - total_reconciled = 0.0 + total_untaxed = 0.0 + total_untaxed_currency = 0.0 + total_tax = 0.0 + total_tax_currency = 0.0 + total_residual = 0.0 + total_residual_currency = 0.0 + currencies = set() + for line in move.line_ids: - if line.account_id.user_type_id.type in ('receivable', 'payable'): - amount = abs(line.debit - line.credit) - total_amount += amount - precision_currency = move.currency_id or move.company_id.currency_id - if float_is_zero(total_amount, precision_rounding=precision_currency.rounding): - move.matched_percentage = 1.0 + if line.currency_id: + currencies.add(line.currency_id) + + # Untaxed amount. + if (move.is_invoice(include_receipts=True) and not line.exclude_from_invoice_tab)\ + or (move.type == 'entry' and line.debit and not line.tax_line_id): + total_untaxed += line.balance + total_untaxed_currency += line.amount_currency + + # Tax amount. + if line.tax_line_id: + total_tax += line.balance + total_tax_currency += line.amount_currency + + # Residual amount. + if move.type == 'entry' or line.account_id.user_type_id.type in ('receivable', 'payable'): + total_residual += line.amount_residual + total_residual_currency += line.amount_residual_currency + + total = total_untaxed + total_tax + total_currency = total_untaxed_currency + total_tax_currency + + if (move.type == 'entry' and total < 0.0) or move.is_inbound(): + sign = -1 else: - for line in move.line_ids: - if line.account_id.user_type_id.type in ('receivable', 'payable'): - for partial_line in (line.matched_debit_ids + line.matched_credit_ids): - total_reconciled += partial_line.amount - move.matched_percentage = total_reconciled / total_amount + sign = 1 + + move.amount_untaxed = sign * (total_untaxed_currency if len(currencies) == 1 else total_untaxed) + move.amount_tax = sign * (total_tax_currency if len(currencies) == 1 else total_tax) + move.amount_total = sign * (total_currency if len(currencies) == 1 else total) + move.amount_residual = -sign * (total_residual_currency if len(currencies) == 1 else total_residual) + move.amount_untaxed_signed = -total_untaxed + move.amount_tax_signed = -total_tax + move.amount_total_signed = -total + move.amount_residual_signed = total_residual + + currency = len(currencies) == 1 and currencies.pop() or move.company_id.currency_id + is_paid = currency and currency.is_zero(move.amount_residual) or not move.amount_residual + + # Compute 'invoice_payment_state'. + if move.state == 'posted' and is_paid: + if move.id in in_payment_set: + move.invoice_payment_state = 'in_payment' + else: + move.invoice_payment_state = 'paid' + else: + move.invoice_payment_state = 'not_paid' @api.multi - @api.depends('line_ids.reconcile_model_id') - def _compute_reconcile_model(self): + def _inverse_amount_total(self): for move in self: - move.reconcile_model_id = move.line_ids.mapped('reconcile_model_id') + if len(move.line_ids) != 2 or move.type != 'entry': + continue - @api.model - @api.depends('reconcile_model_id') - def _search_reconcile_model(self, operator, operand): - if operand: - rmi = self.search([('line_ids.reconcile_model_id', operator, operand)]) + to_write = [] + + if move.currency_id != move.company_id.currency_id: + amount_currency = abs(move.amount_total) + balance = move.currency_id._convert(amount_currency, move.currency_id, move.company_id, move.date) + else: + balance = abs(move.amount_total) + amount_currency = 0.0 + + for line in move.line_ids: + to_write.append((1, line.id, { + 'debit': line.balance > 0.0 and balance or 0.0, + 'credit': line.balance < 0.0 and balance or 0.0, + 'amount_currency': line.balance > 0.0 and amount_currency or -amount_currency, + })) + + move.write({'line_ids': to_write}) + + @api.multi + def _get_domain_matching_supsense_moves(self): + self.ensure_one() + domain = self.env['account.move.line']._get_suspense_moves_domain() + domain += ['|', ('partner_id', '=?', self.partner_id.id), ('partner_id', '=', False)] + if self.is_inbound(): + domain.append(('balance', '=', -self.amount_residual)) else: - rmi = self.search([('line_ids', operator, operand)]) - if rmi: - return [('id', 'in', rmi.ids)] - return [('id', '=', False)] + domain.append(('balance', '=', self.amount_residual)) + return domain + + def _compute_has_matching_suspense_amount(self): + for r in self: + res = False + if r.state == 'posted' and r.is_invoice() and r.invoice_payment_state == 'not_paid': + domain = r._get_domain_matching_supsense_moves() + #there are more than one but less than 5 suspense moves matching the residual amount + if (0 < self.env['account.move.line'].search_count(domain) < 5): + domain2 = [ + ('invoice_payment_state', '=', 'not_paid'), + ('state', '=', 'open'), + ('amount_residual', '=', r.amount_residual), + ('type', '=', r.type)] + #there are less than 5 other open invoices of the same type with the same residual + if self.env['account.move'].search_count(domain2) < 5: + res = True + r.invoice_has_matching_supsense_amount = res + + @api.depends('partner_id', 'invoice_source_email') + def _compute_invoice_vendor_display_info(self): + for move in self: + vendor_display_name = move.partner_id.name + move.invoice_icon = '' + if not vendor_display_name: + if move.invoice_source_email: + vendor_display_name = _('From: ') + move.invoice_source_email + move.invoice_vendor_icon = '@' + else: + vendor_display_name = ('Created by: ') + move.create_uid.name + move.invoice_vendor_icon = '#' + move.invoice_vendor_display_name = vendor_display_name + + @api.depends('state', 'journal_id', 'invoice_date') + def _compute_invoice_sequence_number_next(self): + """ computes the prefix of the number that will be assigned to the first invoice/bill/refund of a journal, in order to + let the user manually change it. + """ + # Check user group. + system_user = self.env.user._is_system() + if not system_user: + return + + # Check moves being candidates to set a custom number next. + moves = self.filtered(lambda move: move.is_invoice() and move.name == '/') + if not moves: + return + + for key, group in groupby(moves, key=lambda move: (move.journal_id, move._get_sequence())): + journal, sequence = key + domain = [('journal_id', '=', journal.id), ('state', '=', 'posted')] + if not isinstance(self.id, models.NewId): + domain.append(('id', '!=', self.id)) + if journal.type == 'sale': + domain.append(('type', 'in', ('out_invoice', 'out_refund'))) + elif journal.type == 'purchase': + domain.append(('type', 'in', ('in_invoice', 'in_refund'))) + else: + continue + if self.search_count(domain): + continue + + for move in group: + prefix, dummy = sequence._get_prefix_suffix(date=move.invoice_date or fields.Date.today(), date_range=move.invoice_date) + number_next = sequence._get_current_sequence().number_next_actual + move.invoice_sequence_number_next_prefix = prefix + move.invoice_sequence_number_next = '%%0%sd' % sequence.padding % number_next @api.multi - def _get_default_journal(self): - if self.env.context.get('default_journal_type'): - return self.env['account.journal'].search([('company_id', '=', self.env.company.id), ('type', '=', self.env.context['default_journal_type'])], limit=1).id + def _inverse_invoice_sequence_number_next(self): + ''' Set the number_next on the sequence related to the invoice/bill/refund''' + # Check user group. + if not self.env.user._is_admin(): + return + + # Set the next number in the sequence. + for move in self: + if not move.invoice_sequence_number_next: + continue + sequence = move._get_sequence() + nxt = re.sub("[^0-9]", '', move.invoice_sequence_number_next) + result = re.match("(0*)([0-9]+)", nxt) + if result and sequence: + date_sequence = sequence._get_current_sequence() + date_sequence.number_next_actual = int(result.group(2)) + + @api.multi + def _compute_payments_widget_to_reconcile_info(self): + for move in self: + move.invoice_outstanding_credits_debits_widget = json.dumps(False) + + if move.state != 'posted' or move.invoice_payment_state != 'not_paid' or not move.is_invoice(include_receipts=True): + continue + pay_term_line_ids = move.line_ids.filtered(lambda line: line.account_id.user_type_id.type in ('receivable', 'payable')) + + domain = [('account_id', 'in', pay_term_line_ids.mapped('account_id').ids), + ('move_id.state', '=', 'posted'), + ('partner_id', '=', move.commercial_partner_id.id), + ('reconciled', '=', False), '|', ('amount_residual', '!=', 0.0), + ('amount_residual_currency', '!=', 0.0)] + + if move.is_inbound(): + domain.extend([('credit', '>', 0), ('debit', '=', 0)]) + type_payment = _('Outstanding credits') + else: + domain.extend([('credit', '=', 0), ('debit', '>', 0)]) + type_payment = _('Outstanding debits') + info = {'title': '', 'outstanding': True, 'content': [], 'move_id': move.id} + lines = self.env['account.move.line'].search(domain) + currency_id = move.currency_id + if len(lines) != 0: + for line in lines: + # get the outstanding residual value in invoice currency + if line.currency_id and line.currency_id == move.currency_id: + amount_to_show = abs(line.amount_residual_currency) + else: + currency = line.company_id.currency_id + amount_to_show = currency._convert(abs(line.amount_residual), move.currency_id, move.company_id, + line.date or fields.Date.today()) + if float_is_zero(amount_to_show, precision_rounding=move.currency_id.rounding): + continue + info['content'].append({ + 'journal_name': line.ref or line.move_id.name, + 'amount': amount_to_show, + 'currency': currency_id.symbol, + 'id': line.id, + 'position': currency_id.position, + 'digits': [69, move.currency_id.decimal_places], + }) + info['title'] = type_payment + move.invoice_outstanding_credits_debits_widget = json.dumps(info) + move.invoice_has_outstanding = True @api.multi - @api.depends('line_ids.partner_id') - def _compute_partner_id(self): + def _get_reconciled_info_JSON_values(self): + self.ensure_one() + foreign_currency = self.currency_id if self.currency_id != self.company_id.currency_id else False + + reconciled_vals = [] + pay_term_line_ids = self.line_ids.filtered(lambda line: line.account_id.user_type_id.type in ('receivable', 'payable')) + partials = pay_term_line_ids.mapped('matched_debit_ids') + pay_term_line_ids.mapped('matched_credit_ids') + for partial in partials: + counterpart_lines = partial.debit_move_id + partial.credit_move_id + counterpart_line = counterpart_lines.filtered(lambda line: line not in self.line_ids) + + if foreign_currency and partial.currency_id == foreign_currency: + amount = partial.amount_currency + else: + amount = partial.company_currency_id._convert(partial.amount, self.currency_id, self.company_id, self.date) + + if float_is_zero(amount, precision_rounding=self.currency_id.rounding): + continue + + ref = counterpart_line.move_id.name + if counterpart_line.move_id.ref: + ref += ' (' + counterpart_line.move_id.ref + ')' + + reconciled_vals.append({ + 'name': counterpart_line.name, + 'journal_name': counterpart_line.journal_id.name, + 'amount': amount, + 'currency': self.currency_id.symbol, + 'digits': [69, self.currency_id.decimal_places], + 'position': self.currency_id.position, + 'date': counterpart_line.date, + 'payment_id': counterpart_line.id, + 'account_payment_id': counterpart_line.payment_id.id, + 'move_id': counterpart_line.move_id.id, + 'ref': ref, + }) + return reconciled_vals + + @api.depends('type', 'line_ids.amount_residual') + def _compute_payments_widget_reconciled_info(self): for move in self: - partner = move.line_ids.mapped('partner_id') - move.partner_id = partner.id if len(partner) == 1 else False - - @api.onchange('date') - def _onchange_date(self): - '''On the form view, a change on the date will trigger onchange() on account.move - but not on account.move.line even the date field is related to account.move. - Then, trigger the _onchange_amount_currency manually. + if move.state != 'posted' or not move.is_invoice(include_receipts=True): + continue + reconciled_vals = move._get_reconciled_info_JSON_values() + if reconciled_vals: + info = { + 'title': _('Less Payment'), + 'outstanding': False, + 'content': reconciled_vals, + } + move.invoice_payments_widget = json.dumps(info, default=date_utils.json_default) + else: + move.invoice_payments_widget = json.dumps(False) + + @api.depends('line_ids.price_subtotal', 'line_ids.tax_base_amount', 'line_ids.tax_line_id', 'partner_id', 'currency_id') + def _compute_invoice_taxes_by_group(self): + ''' Helper to get the taxes grouped according their account.tax.group. + This method is only used when printing the invoice. ''' - self.line_ids._onchange_amount_currency() - - name = fields.Char(string='Number', required=True, copy=False, default='/') - ref = fields.Char(string='Reference', copy=False) - date = fields.Date(required=True, states={'posted': [('readonly', True)]}, index=True, default=fields.Date.context_today) - journal_id = fields.Many2one('account.journal', string='Journal', required=True, states={'posted': [('readonly', True)]}, default=_get_default_journal) - state = fields.Selection([('draft', 'Unposted'), ('posted', 'Posted')], string='Status', - required=True, readonly=True, copy=False, default='draft', - help='All manually created new journal entries are usually in the status \'Unposted\', ' - 'but you can set the option to skip that status on the related journal. ' - 'In that case, they will behave as journal entries automatically created by the ' - 'system on document validation (invoices, bank statements...) and will be created ' - 'in \'Posted\' status.') - line_ids = fields.One2many('account.move.line', 'move_id', string='Journal Items', - states={'posted': [('readonly', True)]}, copy=True) - partner_id = fields.Many2one('res.partner', compute='_compute_partner_id', string="Partner", store=True, readonly=True) - amount = fields.Monetary(compute='_amount_compute', inverse="_set_amount", store=True) - currency_id = fields.Many2one('res.currency', compute='_amount_compute', store=True, string="Currency") - narration = fields.Text(string='Internal Note') - company_id = fields.Many2one('res.company', related='journal_id.company_id', string='Company', store=True, readonly=True) - matched_percentage = fields.Float('Percentage Matched', compute='_compute_matched_percentage', digits=0, store=True, readonly=True, help="Technical field used in cash basis method") - reconcile_model_id = fields.Many2many('account.reconcile.model', compute='_compute_reconcile_model', search='_search_reconcile_model', string="Reconciliation Model", readonly=True) - # Dummy Account field to search on account.move by account_id - dummy_account_id = fields.Many2one('account.account', related='line_ids.account_id', string='Account', store=False, readonly=True) - tax_cash_basis_rec_id = fields.Many2one( - 'account.partial.reconcile', - string='Tax Cash Basis Entry of', - help="Technical field used to keep track of the tax cash basis reconciliation. " - "This is needed when cancelling the source: it will post the inverse journal entry to cancel that part too.") - auto_post = fields.Boolean(string='Post Automatically', default=False, help='If this checkbox is ticked, this entry will be automatically posted at its date.') - reverse_entry_id = fields.Many2one('account.move', String="Reverse entry", store=True, readonly=True) - to_check = fields.Boolean(string='To Check', default=False, help='If this checkbox is ticked, it means that the user was not sure of all the related informations at the time of the creation of the move and that the move needs to be checked again.') - tax_type_domain = fields.Char(store=False, help='Technical field used to have a dynamic taxes domain on the form view.') + for move in self: + lang_env = move.with_context(lang=move.partner_id.lang).env + tax_lines = move.line_ids.filtered(lambda line: line.tax_line_id) + res = {} + for line in tax_lines: + res.setdefault(line.tax_line_id.tax_group_id, {'base': 0.0, 'amount': 0.0}) + res[line.tax_line_id.tax_group_id]['amount'] += line.price_subtotal + res[line.tax_line_id.tax_group_id]['base'] += line.tax_base_amount + res = sorted(res.items(), key=lambda l: l[0].sequence) + move.amount_by_group = [( + group.name, amounts['amount'], + amounts['base'], + formatLang(lang_env, amounts['amount'], currency_obj=move.currency_id), + formatLang(lang_env, amounts['base'], currency_obj=move.currency_id), + len(res), + ) for group, amounts in res] + + # ------------------------------------------------------------------------- + # CONSTRAINS METHODS + # ------------------------------------------------------------------------- @api.constrains('line_ids', 'journal_id') def _validate_move_modification(self): if 'posted' in self.mapped('line_ids.payment_id.state'): raise ValidationError(_("You cannot modify a journal entry linked to a posted payment.")) - @api.onchange('journal_id') - def _onchange_journal_id(self): - self.tax_type_domain = self.journal_id.type if self.journal_id.type in ('sale', 'purchase') else None + @api.constrains('name', 'journal_id') + def _check_unique_sequence_number(self): + if not self: + return + + self._cr.execute(''' + SELECT move.id + FROM account_move move + INNER JOIN account_move move2 ON + move2.name = move.name + AND move2.company_id = move.company_id + AND move2.journal_id = move.journal_id + AND move2.type = move.type + AND move2.id != move.id + WHERE move.id IN %s + AND move.state = 'posted' + AND move2.state = 'posted' + ''', [tuple(self.ids)]) + res = self._cr.fetchone() + if res: + raise ValidationError(_('Posted journal entry must have an unique sequence number per company.')) + + @api.constrains('ref') + def _check_duplicate_supplier_reference(self): + moves = self.filtered(lambda move: move.is_purchase_document() and move.ref) + if not moves: + return - @api.onchange('line_ids') - def _onchange_line_ids(self): - '''Compute additional lines corresponding to the taxes set on the line_ids. + self._cr.execute(''' + SELECT move.id + FROM account_move move + INNER JOIN account_move move2 ON + move2.ref = move.ref + AND move2.company_id = move.company_id + AND move2.commercial_partner_id = move.commercial_partner_id + AND move2.type = move.type + AND move2.id != move.id + WHERE move.id IN %s + AND move.type in ('in_invoice', 'in_refund') + AND move.ref IS NOT NULL + ''', [tuple(self.ids)]) + if self._cr.fetchone(): + raise ValidationError(_('Duplicated vendor reference detected. You probably encoded twice the same vendor bill/credit note.')) - For example, add a line with 1000 debit and 15% tax, this onchange will add a new - line with 150 debit. + @api.multi + def _check_balanced(self): + ''' Assert the move is fully balanced debit = credit. + An error is raised if it's not the case. ''' - def _str_to_list(string): - #remove heading and trailing brackets and return a list of int. This avoid calling safe_eval on untrusted field content - string = string[1:-1] - if string: - return [int(x) for x in string.split(',')] - return [] - - def _build_grouping_key(line): - #build a string containing all values used to create the tax line - return str(line.tax_ids.ids) + '-' + str(line.analytic_tag_ids.ids) + '-' + (line.analytic_account_id and str(line.analytic_account_id.id) or '') - - def _parse_grouping_key(line): - # Retrieve values computed the last time this method has been run. - if not line.tax_line_grouping_key: - return {'tax_ids': [], 'tag_ids': [], 'analytic_account_id': False} - tax_str, tags_str, analytic_account_str = line.tax_line_grouping_key.split('-') - return { - 'tax_ids': _str_to_list(tax_str), - 'tag_ids': _str_to_list(tags_str), - 'analytic_account_id': analytic_account_str and int(analytic_account_str) or False, - } + moves = self.filtered(lambda move: move.line_ids) + if not moves: + return + + # /!\ As this method is called in create / write, we can't make the assumption the computed stored fields + # are already done. Then, this query MUST NOT depend of computed stored fields (e.g. balance). + # It happens as the ORM makes the create with the 'no_recompute' statement. + self._cr.execute(''' + SELECT line.move_id + FROM account_move_line line + JOIN account_move move ON move.id = line.move_id + JOIN account_journal journal ON journal.id = move.journal_id + JOIN res_company company ON company.id = journal.company_id + JOIN res_currency currency ON currency.id = company.currency_id + WHERE line.move_id IN %s + GROUP BY line.move_id, currency.decimal_places + HAVING ROUND(SUM(debit - credit), currency.decimal_places) != 0.0; + ''', [tuple(self.ids)]) + + query_res = self._cr.fetchall() + if query_res: + ids = [res[0] for res in query_res] + raise UserError(_("Cannot create unbalanced journal entry. Ids: %s") % str(ids)) + + @api.multi + def _check_fiscalyear_lock_date(self): + for move in self: + lock_date = max(move.company_id.period_lock_date or date.min, move.company_id.fiscalyear_lock_date or date.min) + if self.user_has_groups('account.group_account_manager'): + lock_date = move.company_id.fiscalyear_lock_date + if move.date <= (lock_date or date.min): + if self.user_has_groups('account.group_account_manager'): + message = _("You cannot add/modify entries prior to and inclusive of the lock date %s") % lock_date + else: + message = _("You cannot add/modify entries prior to and inclusive of the lock date %s. Check the company settings or ask someone with the 'Adviser' role") % lock_date + raise UserError(message) + return True - def _find_existing_tax_line(line_ids, tax_repartition_line_id, analytic_tag_ids, analytic_account_id): - # tax_repartition_line_id, tag_ids and analytic_account_id are real ids - if tax.analytic: - return line_ids.filtered(lambda x: x.tax_repartition_line_id.id == tax_repartition_line_id and x.analytic_tag_ids.ids == analytic_tag_ids and x.analytic_account_id.id == analytic_account_id) - return line_ids.filtered(lambda x: x.tax_repartition_line_id.id == tax_repartition_line_id) + @api.multi + def _check_tax_lock_date(self): + if not self: + return + + self._cr.execute(''' + SELECT move.id, company.tax_lock_date + FROM account_move move + JOIN account_journal journal ON journal.id = move.journal_id + JOIN res_company company ON company.id = journal.company_id + WHERE move.id IN %s + AND move.date < company.tax_lock_date + ''', [tuple(self.ids)]) + + query_res = self._cr.fetchone() + if query_res: + raise UserError(_(''' + The operation is refused as it would impact an already issued tax statement. + Please change the journal entry date or the tax lock date set in the settings (%s) to proceed + ''' % query_res[1])) + + @api.multi + def _check_move_consistency(self): + for move in self: + if move.line_ids: + if not all([x.company_id.id == move.company_id.id for x in move.line_ids]): + raise UserError(_("Cannot create moves for different companies.")) - def _get_lines_to_sum(line_ids, tax, tag_ids, analytic_account_id): - # tax is a real record; tag_ids and analytic_account_id are real ids - if tax.analytic: - return line_ids.filtered(lambda x: tax in x.tax_ids._origin and x.analytic_tag_ids.ids == tag_ids and x.analytic_account_id.id == analytic_account_id) - return line_ids.filtered(lambda x: tax in x.tax_ids._origin) + self._check_balanced() + self._check_fiscalyear_lock_date() - # Cache the already computed tax to avoid useless recalculation (real records) - processed_taxes = self.env['account.tax'] + # ------------------------------------------------------------------------- + # LOW-LEVEL METHODS + # ------------------------------------------------------------------------- + @api.multi + def _move_autocomplete_invoice_lines_values(self): + ''' This method recomputes dynamic lines on the current journal entry that include taxes, cash rounding + and payment terms lines. + ''' self.ensure_one() - for line in self.line_ids.filtered(lambda x: x.recompute_tax_line): - # Retrieve old field values. - parsed_key = _parse_grouping_key(line) - - # Unmark the line. - line.recompute_tax_line = False - - # Manage group of taxes (new records) - group_taxes = line.tax_ids.filtered(lambda t: t.amount_type == 'group') - children_taxes = group_taxes.mapped('children_tax_ids') - if children_taxes: - line.tax_ids |= children_taxes - # Because the taxes on the line changed, we need to recompute them. - processed_taxes -= children_taxes._origin - - # Get the taxes to process (actual records) - taxes = self.env['account.tax'].browse(parsed_key['tax_ids']) - taxes |= line.tax_ids._origin - taxes |= children_taxes._origin - to_process_taxes = (taxes - processed_taxes).filtered(lambda t: t.amount_type != 'group') - processed_taxes += to_process_taxes - - # Apply tags on base line - line.tag_ids = taxes.mapped('invoice_repartition_line_ids').filtered(lambda x: x.repartition_type == 'base').mapped('tag_ids') - - # Process taxes (real records) - for tax in to_process_taxes: - lines_to_sum = _get_lines_to_sum(self.line_ids, tax, parsed_key['tag_ids'], parsed_key['analytic_account_id']) - - balance = sum([l.balance for l in lines_to_sum]) - - # Compute the tax amount one by one. - quantity = len(lines_to_sum) if tax.amount_type == 'fixed' else 1 - taxes_vals = tax.compute_all(balance, quantity=quantity, currency=line.currency_id, product=line.product_id, partner=line.partner_id) - - if taxes_vals.get('taxes'): - for line_vals in taxes_vals['taxes']: - tax_line = _find_existing_tax_line(self.line_ids, line_vals['tax_repartition_line_id'], parsed_key['tag_ids'], parsed_key['analytic_account_id']) - if tax_line: - if not lines_to_sum: - # Drop tax line because the originator tax is no longer used. - self.line_ids -= tax_line - elif balance: - # Update the existing tax_line. - # Update the debit/credit amount according to the new balance. - amount = line_vals['amount'] - account = line_vals['account_id'] or line.account_id - tax_line.debit = amount > 0 and amount or 0.0 - tax_line.credit = amount < 0 and -amount or 0.0 - tax_line.account_id = account - else: - # Reset debit/credit in case of the originator line is temporary set to 0 in both debit/credit. - tax_line.debit = tax_line.credit = 0.0 - else: - # Create a new tax_line. - amount = line_vals['amount'] - to_create_vals = { - 'account_id': line_vals['account_id'] or line.account_id.id, - 'name': line_vals['name'], - 'partner_id': line.partner_id.id, - 'debit': amount > 0 and amount or 0.0, - 'credit': amount < 0 and -amount or 0.0, - 'analytic_account_id': line.analytic_account_id.id if tax.analytic else False, - 'analytic_tag_ids': line.analytic_tag_ids.ids if tax.analytic else False, - 'move_id': self.id, - 'tax_exigible': tax.tax_exigibility == 'on_invoice', - 'company_id': self.company_id.id, - 'company_currency_id': self.company_id.currency_id.id, - 'tax_repartition_line_id': line_vals['tax_repartition_line_id'], - 'tag_ids': line_vals['tag_ids'], - 'tax_base_amount': line_vals['base'], - } - # N.B. currency_id/amount_currency are not set because if we have two lines with the same tax - # and different currencies, we have no idea which currency set on this line. - self.env['account.move.line'].new(to_create_vals) - - # Keep record of the values used as taxes the last time this method has been run. - line.tax_line_grouping_key = _build_grouping_key(line) - @api.model - def fields_view_get(self, view_id=None, view_type='form', toolbar=False, submenu=False): - res = super(AccountMove, self).fields_view_get( - view_id=view_id, view_type=view_type, toolbar=toolbar, submenu=submenu) - if self._context.get('vat_domain'): - res['fields']['line_ids']['views']['tree']['fields']['tax_line_id']['domain'] = [('tag_ids', 'in', [self.env.ref(self._context.get('vat_domain')).id])] - return res + line_currency = self.currency_id if self.currency_id != self.company_id.currency_id else False + for line in self.line_ids: + # Do something only on invoice lines. + if line.exclude_from_invoice_tab: + continue + + # Ensure related fields are well copied. + line.partner_id = self.partner_id + line.date = self.date + line.recompute_tax_line = True + line.currency_id = line_currency + + # Shortcut to load the demo data. + if not line.account_id: + line.account_id = line._get_computed_account() + if not line.account_id: + if self.is_sale_document(include_receipts=True): + line.account_id = self.journal_id.default_credit_account_id + elif self.is_purchase_document(include_receipts=True): + line.account_id = self.journal_id.default_debit_account_id - @api.constrains('company_id') - def _check_company_id(self): - if self.filtered(lambda record: any([x.company_id.id != record.company_id.id for x in record.line_ids])): - raise ValidationError(_("Cannot create moves for different companies.")) + self.line_ids._onchange_price_subtotal() + self._recompute_dynamic_lines(recompute_all_taxes=True) + + values = self._convert_to_write(self._cache) + values.pop('invoice_line_ids', None) + return values @api.model - def create(self, vals): - move = super(AccountMove, self.with_context(mail_create_nolog=True, check_move_validity=False, partner_id=vals.get('partner_id'))).create(vals) - move.assert_balanced() - return move + def _move_autocomplete_invoice_lines_create(self, vals_list): + ''' During the create of an account.move with only 'invoice_line_ids' set and not 'line_ids', this method is called + to auto compute accounting lines of the invoice. In that case, accounts will be retrieved and taxes, cash rounding + and payment terms will be computed. At the end, the values will contains all accounting lines in 'line_ids' + and the moves should be balanced. + + :param vals_list: The list of values passed to the 'create' method. + :return: Modified list of values. + ''' + new_vals_list = [] + for vals in vals_list: + if not vals.get('invoice_line_ids'): + new_vals_list.append(vals) + continue + if vals.get('line_ids'): + vals.pop('invoice_line_ids', None) + new_vals_list.append(vals) + continue + if not vals.get('type') and not self._context.get('default_type'): + vals.pop('invoice_line_ids', None) + new_vals_list.append(vals) + continue + vals['type'] = vals.get('type', self._context.get('default_type', 'entry')) + if not vals['type'] in self.get_invoice_types(include_receipts=True): + new_vals_list.append(vals) + continue + + vals['line_ids'] = vals.pop('invoice_line_ids') + + if vals.get('invoice_date') and not vals.get('date'): + vals['date'] = vals['invoice_date'] + + ctx_vals = {'default_type': vals.get('type') or self._context.get('default_type')} + if vals.get('journal_id'): + ctx_vals['default_journal_id'] = vals['journal_id'] + self_ctx = self.with_context(**ctx_vals) + new_vals = self_ctx._add_missing_default_values(vals) + + move = self_ctx.new(new_vals) + new_vals_list.append(move._move_autocomplete_invoice_lines_values()) + return new_vals_list + + @api.multi + def _move_autocomplete_invoice_lines_write(self, vals): + ''' During the write of an account.move with only 'invoice_line_ids' set and not 'line_ids', this method is called + to auto compute accounting lines of the invoice. In that case, accounts will be retrieved and taxes, cash rounding + and payment terms will be computed. At the end, the values will contains all accounting lines in 'line_ids' + and the moves should be balanced. + + :param vals_list: A python dict representing the values to write. + :return: True if the auto-completion did something, False otherwise. + ''' + enable_autocomplete = 'invoice_line_ids' in vals and 'line_ids' not in vals and True or False + + if not enable_autocomplete: + return False + + vals['line_ids'] = vals.pop('invoice_line_ids') + for invoice in self: + invoice_new = invoice.with_context(default_type=invoice.type, default_journal_id=invoice.journal_id.id).new(origin=invoice) + invoice_new.update(vals) + values = invoice_new._move_autocomplete_invoice_lines_values() + values.pop('invoice_line_ids', None) + invoice.write(values) + return True + + @api.model_create_multi + def create(self, vals_list): + # OVERRIDE + vals_list = self._move_autocomplete_invoice_lines_create(vals_list) + + moves = super(AccountMove, self).create(vals_list) + + # Trigger 'action_invoice_paid' when the invoice is directly paid at its creation. + moves.filtered(lambda move: move.is_invoice(include_receipts=True) and move.invoice_payment_state in ('paid', 'in_payment')).action_invoice_paid() + + return moves @api.multi def write(self, vals): - if 'line_ids' in vals: - res = super(AccountMove, self.with_context(check_move_validity=False)).write(vals) - self.assert_balanced() + not_paid_invoices = self.filtered(lambda move: move.is_invoice(include_receipts=True) and move.invoice_payment_state not in ('paid', 'in_payment')) + + if self._move_autocomplete_invoice_lines_write(vals): + res = True else: - res = super(AccountMove, self).write(vals) + vals.pop('invoice_line_ids', None) + res = super(AccountMove, self.with_context(check_move_validity=False)).write(vals) + + # Ensure the move is still well balanced. + if 'line_ids' in vals and self._context.get('check_move_validity', True): + # 'check_move_validity' is needed as the write will be done line per line. + self._check_move_consistency() + + # Trigger 'action_invoice_paid' when the invoice becomes paid after a write. + not_paid_invoices.filtered(lambda move: move.invoice_payment_state in ('paid', 'in_payment')).action_invoice_paid() + return res @api.multi + def unlink(self): + for move in self: + # check the lock date + check if some entries are reconciled + move.line_ids._update_check() + move.line_ids.unlink() + return super(AccountMove, self).unlink() + + @api.multi + @api.depends('name', 'state') + def name_get(self): + result = [] + for move in self: + if self._context.get('name_groupby') and not (move.type == 'entry' and move.state == 'draft'): + name = '**%s**, %s %s' % (format_date(self.env, move.date), move.name, move.partner_id.name or '') + elif move.type == 'entry': + # Miscellaneous operation. + if move.state == 'draft': + name = '* %s' % str(move.id) + else: + name = move.name + else: + # Invoice. + name = move._get_invoice_display_name(show_ref=True) + result.append((move.id, name)) + return result + + @api.multi + def _track_subtype(self, init_values): + # OVERRIDE to add custom subtype depending of the state. + self.ensure_one() + + if not self.is_invoice(include_receipts=True): + return super(AccountMove, self)._track_subtype(init_values) + + if 'invoice_payment_state' in init_values and self.invoice_payment_state == 'paid': + return self.env.ref('account.mt_invoice_paid') + elif 'state' in init_values and self.state == 'posted' and self.is_sale_document(include_receipts=True): + return self.env.ref('account.mt_invoice_validated') + elif 'state' in init_values and self.state == 'draft' and self.is_sale_document(include_receipts=True): + return self.env.ref('account.mt_invoice_created') + return super(AccountMove, self)._track_subtype(init_values) + + # ------------------------------------------------------------------------- + # BUSINESS METHODS + # ------------------------------------------------------------------------- + + @api.model + def get_invoice_types(self, include_receipts=False): + return ['out_invoice', 'out_refund', 'in_refund', 'in_invoice'] + (include_receipts and ['out_receipt', 'in_receipt'] or []) + + def is_invoice(self, include_receipts=False): + return self.type in self.get_invoice_types(include_receipts) + + @api.model + def get_sale_types(self, include_receipts=False): + return ['out_invoice', 'out_refund'] + (include_receipts and ['out_receipt'] or []) + + def is_sale_document(self, include_receipts=False): + return self.type in self.get_sale_types(include_receipts) + + @api.model + def get_purchase_types(self, include_receipts=False): + return ['in_invoice', 'in_refund'] + (include_receipts and ['in_receipt'] or []) + + def is_purchase_document(self, include_receipts=False): + return self.type in self.get_purchase_types(include_receipts) + + @api.model + def get_inbound_types(self, include_receipts=True): + return ['out_invoice', 'in_refund'] + (include_receipts and ['out_receipt'] or []) + + def is_inbound(self, include_receipts=True): + return self.type in self.get_inbound_types(include_receipts) + + @api.model + def get_outbound_types(self, include_receipts=True): + return ['in_invoice', 'out_refund'] + (include_receipts and ['in_receipt'] or []) + + def is_outbound(self, include_receipts=True): + return self.type in self.get_outbound_types(include_receipts) + + @api.multi + def _get_invoice_reference_euro_invoice(self): + """ This computes the reference based on the RF Creditor Reference. + The data of the reference is the database id number of the invoice. + For instance, if an invoice is issued with id 43, the check number + is 07 so the reference will be 'RF07 43'. + """ + self.ensure_one() + base = self.id + check_digits = mod_97_10.calc_check_digits('{}RF'.format(base)) + reference = 'RF{} {}'.format(check_digits, " ".join(["".join(x) for x in zip_longest(*[iter(str(base))]*4, fillvalue="")])) + return reference + + @api.multi + def _get_invoice_reference_euro_partner(self): + """ This computes the reference based on the RF Creditor Reference. + The data of the reference is the user defined reference of the + partner or the database id number of the parter. + For instance, if an invoice is issued for the partner with internal + reference 'food buyer 654', the digits will be extracted and used as + the data. This will lead to a check number equal to 00 and the + reference will be 'RF00 654'. + If no reference is set for the partner, its id in the database will + be used. + """ + self.ensure_one() + partner_ref = self.partner_id.ref + partner_ref_nr = re.sub('\D', '', partner_ref or '')[-21:] or str(self.partner_id.id)[-21:] + partner_ref_nr = partner_ref_nr[-21:] + check_digits = mod_97_10.calc_check_digits('{}RF'.format(partner_ref_nr)) + reference = 'RF{} {}'.format(check_digits, " ".join(["".join(x) for x in zip_longest(*[iter(partner_ref_nr)]*4, fillvalue="")])) + return reference + + @api.multi + def _get_invoice_reference_odoo_invoice(self): + """ This computes the reference based on the Odoo format. + We simply return the number of the invoice, defined on the journal + sequence. + """ + self.ensure_one() + return self.name + + @api.multi + def _get_invoice_reference_odoo_partner(self): + """ This computes the reference based on the Odoo format. + The data used is the reference set on the partner or its database + id otherwise. For instance if the reference of the customer is + 'dumb customer 97', the reference will be 'CUST/dumb customer 97'. + """ + ref = self.partner_id.ref or str(self.partner_id.id) + prefix = _('CUST') + return '%s/%s' % (prefix, ref) + + @api.multi + def _get_invoice_computed_reference(self): + self.ensure_one() + if self.journal_id.invoice_reference_type == 'none': + return '' + else: + ref_function = getattr(self, '_get_invoice_reference_{}_{}'.format(self.journal_id.invoice_reference_model, self.journal_id.invoice_reference_type)) + if ref_function: + return ref_function() + else: + raise UserError(_('The combination of reference model and reference type on the journal is not implemented')) + + @api.multi + def _get_sequence(self): + ''' Return the sequence to be used during the post of the current move. + :return: An ir.sequence record or False. + ''' + self.ensure_one() + + journal = self.journal_id + if self.type in ('entry', 'out_invoice', 'in_invoice') or not journal.refund_sequence: + return journal.sequence_id + if not journal.refund_sequence_id: + return + return journal.refund_sequence_id + + @api.multi + def _get_invoice_display_name(self, show_ref=False): + ''' Helper to get the display name of an invoice depending of its type. + :param show_ref: A flag indicating of the display name must include or not the journal entry reference. + :return: A string representing the invoice. + ''' + self.ensure_one() + if self.state == 'draft': + return { + 'out_invoice': _('Draft Invoice'), + 'out_refund': _('Credit Note'), + 'in_invoice': _('Vendor Bill'), + 'in_refund': _('Vendor Credit Note'), + 'out_receipt': _('Sales Receipt'), + 'in_receipt': _('Purchase Receipt'), + }[self.type] + else: + return ('%s' % self.name) + (show_ref and self.ref and '(%s)' % self.ref or '') + + @api.multi + def _get_invoice_delivery_partner_id(self): + ''' Hook allowing to retrieve the right delivery address depending of installed modules. + :return: A res.partner record's id representing the delivery address. + ''' + self.ensure_one() + return self.partner_id.address_get(['delivery'])['delivery'] + + @api.multi + def _get_invoice_intrastat_country_id(self): + ''' Hook allowing to retrieve the intrastat country depending of installed modules. + :return: A res.country record's id. + ''' + self.ensure_one() + return self.partner_id.country_id.id + + @api.multi + def _get_cash_basis_matched_percentage(self): + """Compute the percentage to apply for cash basis method. This value is relevant only for moves that + involve journal items on receivable or payable accounts. + """ + self.ensure_one() + query = ''' + SELECT + ( + SELECT COALESCE(SUM(line.balance), 0.0) + FROM account_move_line line + JOIN account_account account ON account.id = line.account_id + JOIN account_account_type account_type ON account_type.id = account.user_type_id + WHERE line.move_id = %s AND account_type.type IN ('receivable', 'payable') + ) AS total_amount, + ( + SELECT COALESCE(SUM(partial.amount), 0.0) + FROM account_move_line line + JOIN account_account account ON account.id = line.account_id + JOIN account_account_type account_type ON account_type.id = account.user_type_id + LEFT JOIN account_partial_reconcile partial ON + partial.debit_move_id = line.id + OR + partial.credit_move_id = line.id + WHERE line.move_id = %s AND account_type.type IN ('receivable', 'payable') + ) AS total_reconciled + ''' + params = [self.id, self.id] + self._cr.execute(query, params) + total_amount, total_reconciled = self._cr.fetchone() + if float_is_zero(total_amount, precision_rounding=self.company_id.currency_id.rounding): + return 1.0 + else: + return abs(total_reconciled / total_amount) + + @api.multi + def _reverse_move_vals(self, default_values, cancel=True): + ''' Reverse values passed as parameter being the copied values of the original journal entry. + For example, debit / credit must be switched. The tax lines must be edited in case of refunds. + + :param default_values: A copy_date of the original journal entry. + :param cancel: A flag indicating the reverse is made to cancel the original journal entry. + :return: The updated default_values. + ''' + self.ensure_one() + + def compute_tax_repartition_lines_mapping(move_vals): + ''' Computes and returns a mapping between the current repartition lines to the new expected one. + :param move_vals: The newly created invoice as a python dictionary to be passed to the 'create' method. + :return: A map invoice_repartition_line => refund_repartition_line. + ''' + # invoice_repartition_line => refund_repartition_line + mapping = {} + + # Do nothing if the move is not a credit note. + if move_vals['type'] not in ('out_refund', 'in_refund'): + return mapping + + for line_command in move_vals.get('line_ids', []): + line_vals = line_command[2] # (0, 0, {...}) + + if line_vals.get('tax_ids') and line_vals['tax_ids'][0][2]: + # Base line. + tax_ids = line_vals['tax_ids'][0][2] + elif line_vals.get('tax_line_id'): + # Tax line. + tax_ids = [line_vals['tax_line_id']] + else: + continue + + for tax in self.env['account.tax'].browse(tax_ids): + for inv_rep_line, ref_rep_line in zip(tax.invoice_repartition_line_ids, tax.refund_repartition_line_ids): + mapping[inv_rep_line] = ref_rep_line + return mapping + + move_vals = self.with_context(include_business_fields=True).copy_data(default=default_values)[0] + + tax_repartition_lines_mapping = compute_tax_repartition_lines_mapping(move_vals) + + for line_command in move_vals.get('line_ids', []): + line_vals = line_command[2] # (0, 0, {...}) + + # ==== Inverse debit / credit / amount_currency ==== + amount_currency = -line_vals.get('amount_currency', 0.0) + balance = line_vals['credit'] - line_vals['debit'] + + line_vals.update({ + 'amount_currency': amount_currency, + 'debit': balance > 0.0 and balance or 0.0, + 'credit': balance < 0.0 and -balance or 0.0, + }) + + if move_vals['type'] not in ('out_refund', 'in_refund'): + continue + + # ==== Map tax repartition lines ==== + if line_vals.get('tax_ids') and line_vals['tax_ids'][0][2]: + # Base line. + invoice_repartition_lines = self.env['account.tax'].browse(line_vals['tax_ids'][0][2])\ + .mapped('invoice_repartition_line_ids')\ + .filtered(lambda line: line.repartition_type == 'base') + refund_repartition_lines = invoice_repartition_lines\ + .mapped(lambda line: tax_repartition_lines_mapping[line]) + + line_vals['tag_ids'] = [(6, 0, refund_repartition_lines.mapped('tag_ids').ids)] + elif line_vals.get('tax_repartition_line_id'): + # Tax line. + invoice_repartition_line = self.env['account.tax.repartition.line'].browse(line_vals['tax_repartition_line_id']) + refund_repartition_line = tax_repartition_lines_mapping[invoice_repartition_line] + + # Find the right account. + account_id = self.env['account.move.line']._get_default_tax_account(refund_repartition_line).id + if not account_id: + if not invoice_repartition_line.account_id: + # Keep the current account as the current one comes from the base line. + account_id = line_vals['account_id'] + else: + tax = invoice_repartition_line.invoice_tax_id + base_line = self.line_ids.filtered(lambda line: tax in line.tax_ids)[0] + account_id = base_line.account_id.id + + line_vals.update({ + 'tax_repartition_line_id': refund_repartition_line.id, + 'account_id': account_id, + 'tag_ids': [(6, 0, refund_repartition_line.tag_ids.ids)], + }) + return move_vals + + @api.multi + def _reverse_moves(self, default_values_list=None, cancel=False): + ''' Reverse a recordset of account.move. + If cancel parameter is true, the reconcilable or liquidity lines + of each original move will be reconciled with its reverse's. + + :param default_values_list: A list of default values to consider per move. + ('type' & 'reversed_entry_id' are computed in the method). + :return: An account.move recordset, reverse of the current self. + ''' + if not default_values_list: + default_values_list = [{} for move in self] + + if cancel: + lines = self.mapped('line_ids') + # Avoid maximum recursion depth. + if lines: + lines.remove_move_reconcile() + + reverse_type_map = { + 'entry': 'entry', + 'out_invoice': 'out_refund', + 'in_invoice': 'in_refund', + 'in_refund': 'in_invoice', + 'out_receipt': 'in_receipt', + 'in_receipt': 'out_receipt', + } + + move_vals_list = [] + for move, default_values in zip(self, default_values_list): + default_values.update({ + 'type': reverse_type_map[self.type], + 'reversed_entry_id': self.id, + }) + move_vals_list.append(move._reverse_move_vals(default_values, cancel=cancel)) + reverse_moves = self.env['account.move'].create(move_vals_list) + + # Reconcile moves together to cancel the previous one. + if cancel: + reverse_moves.with_context(move_reverse_cancel=cancel).post() + for move, reverse_move in zip(self, reverse_moves): + accounts = move.mapped('line_ids.account_id') \ + .filtered(lambda account: account.reconcile or account.internal_type == 'liquidity') + for account in accounts: + (move.line_ids + reverse_move.line_ids)\ + .filtered(lambda line: line.account_id == account and line.balance)\ + .reconcile() + + return reverse_moves + + @api.multi + def open_reconcile_view(self): + return self.line_ids.open_reconcile_view() + + @api.multi def post(self): - self._post_validate() + for move in self: + if move.auto_post and move.date > fields.Date.today(): + date_msg = move.date.strftime(self.env['res.lang']._lang_get(self.env.user.lang).date_format) + raise UserError(_("This move is configured to be auto-posted on %s" % date_msg)) + + if not move.partner_id: + if move.is_sale_document(): + raise UserError(_("The field 'Customer' is required, please complete it to validate the Customer Invoice.")) + elif move.is_purchase_document(): + raise UserError(_("The field 'Vendor' is required, please complete it to validate the Vendor Bill.")) + + if move.is_invoice(include_receipts=True) and float_compare(move.amount_total, 0.0, precision_rounding=move.currency_id.rounding) < 0: + raise UserError(_("You cannot validate an invoice with a negative total amount. You should create a credit note instead.")) + + # Handle case when the invoice_date is not set. In that case, the invoice_date is set at today and then, + # lines are recomputed accordingly. + if not move.invoice_date and move.is_invoice(include_receipts=True): + move.invoice_date = fields.Date.context_today(self) + move.with_context(check_move_validity=False)._onchange_invoice_date() + + self._check_tax_lock_date() + self._check_move_consistency() # Create the analytic lines in batch is faster as it leads to less cache invalidation. self.mapped('line_ids').create_analytic_lines() for move in self: if move.auto_post and move.date > fields.Date.today(): raise UserError(_("This move is configured to be auto-posted on {}".format(move.date.strftime(self.env['res.lang']._lang_get(self.env.user.lang).date_format)))) + to_write = {'state': 'posted'} if move.name == '/': - new_name = False - journal = move.journal_id + # Get the journal's sequence. + sequence = move._get_sequence() + if not sequence: + raise UserError(_('Please define a sequence on your journal.')) - if journal.sequence_id: - sequence = journal.sequence_id - new_name = sequence.with_context(ir_sequence_date=move.date).next_by_id() - else: - raise UserError(_('Please define a sequence on the journal.')) + # Consume a new number. + to_write['name'] = sequence.next_by_id(sequence_date=move.date) + + move.write(to_write) - if new_name: - move.name = new_name + # Compute 'ref' for 'out_invoice'. + if move.type == 'out_invoice' and not move.invoice_payment_ref: + to_write = { + 'invoice_payment_ref': move._get_invoice_computed_reference(), + 'line_ids': [] + } + for line in move.line_ids.filtered(lambda line: line.account_id.user_type_id.type in ('receivable', 'payable')): + to_write['line_ids'].append((1, line.id, {'name': to_write['invoice_payment_ref']})) + move.write(to_write) if move == move.company_id.account_opening_move_id and not move.company_id.account_bank_reconciliation_start: # For opening moves, we set the reconciliation date threshold @@ -357,22 +1906,41 @@ class AccountMove(models.Model): # installing Accounting- with bank statements) move.company_id.account_bank_reconciliation_start = move.date - return self.write({'state': 'posted'}) + @api.multi + def action_reverse(self): + action = self.env.ref('account.action_view_account_move_reversal').read()[0] + + if self.is_invoice(): + action['name'] = _('Credit Note') + + return action @api.multi def action_post(self): - if self.mapped('line_ids.payment_id'): - if any(self.mapped('journal_id.post_at_bank_rec')): - raise UserError(_("A payment journal entry generated in a journal configured to post entries only when payments are reconciled with a bank statement cannot be manually posted. Those will be posted automatically after performing the bank reconciliation.")) + if self.mapped('line_ids.payment_id') and any(self.mapped('journal_id.post_at_bank_rec')): + raise UserError(_("A payment journal entry generated in a journal configured to post entries only when payments are reconciled with a bank statement cannot be manually posted. Those will be posted automatically after performing the bank reconciliation.")) return self.post() @api.multi + def js_assign_outstanding_line(self, line_id): + self.ensure_one() + lines = self.env['account.move.line'].browse(line_id) + lines += self.line_ids.filtered(lambda line: line.account_id == lines[0].account_id and not line.reconciled) + return lines.reconcile() + + @api.multi + def button_draft(self): + if any(move.state != 'cancel' for move in self): + raise UserError(_('Only cancelled journal entries can be reset to draft.')) + self.write({'state': 'draft'}) + + @api.multi def button_cancel(self): AccountMoveLine = self.env['account.move.line'] excluded_move_ids = [] - if self._context.get('edition_mode'): - excluded_move_ids = AccountMoveLine.search(AccountMoveLine._get_domain_for_edition_mode() + [('move_id', 'in', self.ids)]).mapped('move_id').ids + if self._context.get('suspense_moves_mode'): + excluded_move_ids = AccountMoveLine.search(AccountMoveLine._get_suspense_moves_domain() + [('move_id', 'in', self.ids)]).mapped('move_id').ids for move in self: if not move.journal_id.update_posted and move.id not in excluded_move_ids: @@ -382,122 +1950,130 @@ class AccountMove(models.Model): if self.ids: self.check_access_rights('write') self.check_access_rule('write') - self._check_lock_date() - self._cr.execute('UPDATE account_move '\ - 'SET state=%s '\ - 'WHERE id IN %s', ('draft', tuple(self.ids),)) + self._check_fiscalyear_lock_date() + self._cr.execute('UPDATE account_move SET state=%s WHERE id IN %s', ('cancel', tuple(self.ids))) self.invalidate_cache() - self._check_lock_date() + self._check_fiscalyear_lock_date() + self.mapped('line_ids').remove_move_reconcile() return True @api.multi - def unlink(self): - for move in self: - #check the lock date + check if some entries are reconciled - move.line_ids._update_check() - move.line_ids.unlink() - return super(AccountMove, self).unlink() + def action_invoice_sent(self): + """ Open a window to compose an email, with the edi invoice template + message loaded by default + """ + self.ensure_one() + template = self.env.ref('account.email_template_edi_invoice', raise_if_not_found=False) + compose_form = self.env.ref('account.account_invoice_send_wizard_form', raise_if_not_found=False) + ctx = dict( + default_model='account.move', + default_res_id=self.id, + default_use_template=bool(template), + default_template_id=template and template.id or False, + default_composition_mode='comment', + mark_invoice_as_sent=True, + custom_layout="mail.mail_notification_paynow", + force_email=True + ) + return { + 'name': _('Send Invoice'), + 'type': 'ir.actions.act_window', + 'view_type': 'form', + 'view_mode': 'form', + 'res_model': 'account.invoice.send', + 'views': [(compose_form.id, 'form')], + 'view_id': compose_form.id, + 'target': 'new', + 'context': ctx, + } @api.multi - def _post_validate(self): - for move in self: - if move.line_ids: - if not all([x.company_id.id == move.company_id.id for x in move.line_ids]): - raise UserError(_("Cannot create moves for different companies.")) - self.assert_balanced() - return self._check_lock_date() + def action_invoice_print(self): + """ Print the invoice and mark it as sent, so that we can see more + easily the next step of the workflow + """ + if any(not move.is_invoice(include_receipts=True) for move in self): + raise UserError(_("Only invoices could be printed.")) + + self.filtered(lambda inv: not inv.invoice_sent).write({'invoice_sent': True}) + if self.user_has_groups('account.group_account_invoice'): + return self.env.ref('account.account_invoices').report_action(self) + else: + return self.env.ref('account.account_invoices_without_payment').report_action(self) @api.multi - def _check_lock_date(self): - for move in self: - lock_date = max(move.company_id.period_lock_date or date.min, move.company_id.fiscalyear_lock_date or date.min) - if self.user_has_groups('account.group_account_manager'): - lock_date = move.company_id.fiscalyear_lock_date - if move.date <= (lock_date or date.min): - if self.user_has_groups('account.group_account_manager'): - message = _("You cannot add/modify entries prior to and inclusive of the lock date %s") % (lock_date) - else: - message = _("You cannot add/modify entries prior to and inclusive of the lock date %s. Check the company settings or ask someone with the 'Adviser' role") % (lock_date) - raise UserError(message) - return True + def action_invoice_paid(self): + ''' Hook to be overrided called when the invoice moves to the paid state. ''' + pass @api.multi - def assert_balanced(self): - if not self.ids: - return True - prec = self.env.company.currency_id.decimal_places - - self._cr.execute("""\ - SELECT move_id - FROM account_move_line - WHERE move_id in %s - GROUP BY move_id - HAVING abs(sum(debit) - sum(credit)) > %s - """, (tuple(self.ids), 10 ** (-max(5, prec)))) - if len(self._cr.fetchall()) != 0: - raise UserError(_("Cannot create unbalanced journal entry.")) - return True + def action_open_matching_suspense_moves(self): + self.ensure_one() + domain = self._get_domain_matching_supsense_moves() + ids = self.env['account.move.line'].search(domain).mapped('statement_line_id').ids + action_context = {'show_mode_selector': False, 'company_ids': self.mapped('company_id').ids} + action_context.update({'suspense_moves_mode': True}) + action_context.update({'statement_line_ids': ids}) + action_context.update({'partner_id': self.partner_id.id}) + action_context.update({'partner_name': self.partner_id.name}) + return { + 'type': 'ir.actions.client', + 'tag': 'bank_statement_reconciliation_view', + 'context': action_context, + } + + @api.multi + def action_invoice_register_payment(self): + return self.env['account.payment']\ + .with_context(active_ids=self.ids, active_model='account.move', active_id=self.id)\ + .action_register_payment() @api.multi - def _reverse_move(self, date=None, journal_id=None): + def _get_report_base_filename(self): + if any(not move.is_invoice() for move in self): + raise UserError(_("Only invoices could be printed.")) + return self._get_invoice_display_name() + + @api.multi + def preview_invoice(self): self.ensure_one() - date = date or fields.Date.today() - with self.env.norecompute(): - reversed_move = self.copy(default={ - 'date': date, - 'journal_id': journal_id.id if journal_id else self.journal_id.id, - 'ref': _('Reversal of: %s') % (self.name), - }) - for acm_line in reversed_move.line_ids.with_context(check_move_validity=False): - acm_line.write({ - 'debit': acm_line.credit, - 'credit': acm_line.debit, - 'amount_currency': -acm_line.amount_currency - }) - self.reverse_entry_id = reversed_move - self.recompute() - return reversed_move - - @api.multi - def reverse_moves(self, date=None, journal_id=None): - date = date or fields.Date.today() - reversed_moves = self.env['account.move'] - for ac_move in self: - #unreconcile all lines reversed - aml = ac_move.line_ids.filtered(lambda x: x.account_id.reconcile or x.account_id.internal_type == 'liquidity') - aml.remove_move_reconcile() - reversed_move = ac_move._reverse_move(date=date, journal_id=journal_id) - reversed_moves |= reversed_move - #reconcile together the reconcilable (or the liquidity aml) and their newly created counterpart - for account in set([x.account_id for x in aml]): - to_rec = aml.filtered(lambda y: y.account_id == account) - to_rec |= reversed_move.line_ids.filtered(lambda y: y.account_id == account) - #reconciliation will be full, so speed up the computation by using skip_full_reconcile_check in the context - to_rec.reconcile() - if reversed_moves: - reversed_moves._post_validate() - reversed_moves.post() - return [x.id for x in reversed_moves] - return [] + return { + 'type': 'ir.actions.act_url', + 'target': 'self', + 'url': self.get_portal_url(), + } @api.multi - def open_reconcile_view(self): - return self.line_ids.open_reconcile_view() + def _compute_access_url(self): + super(AccountMove, self)._compute_access_url() + for move in self.filtered(lambda move: move.is_invoice()): + move.access_url = '/my/invoices/%s' % (move.id) - # FIXME: Clarify me and change me in master @api.multi - def action_duplicate(self): + def action_view_reverse_entry(self): self.ensure_one() - action = self.env.ref('account.action_move_journal_line').read()[0] - action['context'] = dict(self.env.context) - action['context']['form_view_initial_mode'] = 'edit' - action['context']['view_no_maturity'] = False - action['views'] = [(self.env.ref('account.view_move_form').id, 'form')] - action['res_id'] = self.copy().id + + # Create action. + action = { + 'name': _('Reverse Moves'), + 'type': 'ir.actions.act_window', + 'res_model': 'account.move', + } + reverse_entries = self.env['account.move'].search([('reversed_entry_id', '=', self.id)]) + if len(reverse_entries) == 1: + action.update({ + 'view_mode': 'form', + 'res_id': reverse_entries.id, + }) + else: + action.update({ + 'view_mode': 'tree', + 'domain': [('id', 'in', reverse_entries.ids)], + }) return action @api.model - def _run_post_draft_to_post(self): + def _autopost_draft_entries(self): ''' This method is called from a cron job. It is used to post entries such as those created by the module account_asset. @@ -509,46 +2085,546 @@ class AccountMove(models.Model): ]) records.post() - @api.multi - def action_view_reverse_entry(self): - action = self.env.ref('account.action_move_journal_line').read()[0] - action['views'] = [(self.env.ref('account.view_move_form').id, 'form')] - action['res_id'] = self.reverse_entry_id.id - return action class AccountMoveLine(models.Model): _name = "account.move.line" _description = "Journal Item" - _order = "date desc, move_name desc, id desc" + _order = "sequence, date desc, move_name desc, id" + + # ==== Business fields ==== + move_id = fields.Many2one('account.move', string='Journal Entry', + index=True, required=True, auto_join=True, ondelete="cascade", + help="The move of this entry line.") + move_name = fields.Char(string='Number', related='move_id.name', store=True, index=True) + date = fields.Date(related='move_id.date', store=True, readonly=True, index=True, copy=False) + ref = fields.Char(related='move_id.ref', store=True, copy=False, index=True, readonly=False) + parent_state = fields.Selection(related='move_id.state', store=True, readonly=True) + journal_id = fields.Many2one(related='move_id.journal_id', store=True, readonly=False, index=True, copy=False) + company_id = fields.Many2one(related='move_id.company_id', store=True, readonly=True) + company_currency_id = fields.Many2one(related='company_id.currency_id', string='Company Currency', + readonly=True, store=True, + help='Utility field to express amount currency') + account_id = fields.Many2one('account.account', string='Account', + index=True, ondelete="cascade", + domain=[('deprecated', '=', False)]) + account_internal_type = fields.Selection(related='account_id.user_type_id.type', string="Internal Type", store=True, readonly=True) + sequence = fields.Integer(default=10) + name = fields.Char(string='Label') + quantity = fields.Float(string='Quantity', + default=1.0, digits=dp.get_precision('Product Unit of Measure'), + help="The optional quantity expressed by this line, eg: number of product sold." + "The quantity is not a legal requirement but is very useful for some reports.") + price_unit = fields.Float(string='Unit Price', digits=dp.get_precision('Product Price')) + discount = fields.Float(string='Discount (%)', digits=dp.get_precision('Discount'), default=0.0) + debit = fields.Monetary(string='Debit', default=0.0, currency_field='company_currency_id') + credit = fields.Monetary(string='Credit', default=0.0, currency_field='company_currency_id') + balance = fields.Monetary(string='Balance', store=True, + currency_field='company_currency_id', + compute='_compute_balance', + help="Technical field holding the debit - credit in order to open meaningful graph views from reports") + amount_currency = fields.Monetary(string='Balance in Currency', store=True, copy=True, + help="The amount expressed in an optional other currency if it is a multi-currency entry.") + price_subtotal = fields.Monetary(string='Subtotal', store=True, readonly=True, + currency_field='always_set_currency_id') + price_total = fields.Monetary(string='Total', store=True, readonly=True, + currency_field='always_set_currency_id') + reconciled = fields.Boolean(compute='_amount_residual', store=True) + blocked = fields.Boolean(string='No Follow-up', default=False, + help="You can check this box to mark this journal item as a litigation with the associated partner") + date_maturity = fields.Date(string='Due date', index=True, + help="This field is used for payable and receivable journal entries. You can put the limit date for the payment of this line.") + currency_id = fields.Many2one('res.currency', string='Currency') + partner_id = fields.Many2one('res.partner', string='Partner', ondelete='restrict') + product_uom_id = fields.Many2one('uom.uom', string='Unit of Measure') + product_id = fields.Many2one('product.product', string='Product') + + # ==== Origin fields ==== + reconcile_model_id = fields.Many2one('account.reconcile.model', string="Reconciliation Model", copy=False) + payment_id = fields.Many2one('account.payment', string="Originator Payment", copy=False, + help="Payment that created this entry") + statement_line_id = fields.Many2one('account.bank.statement.line', + string='Bank statement line reconciled with this entry', + index=True, copy=False, readonly=True) + statement_id = fields.Many2one(related='statement_line_id.statement_id', store=True, index=True, copy=False, + help="The bank statement used for bank reconciliation") + + # ==== Tax fields ==== + tax_ids = fields.Many2many('account.tax', string='Taxes') + tax_line_id = fields.Many2one('account.tax', string='Originator tax', ondelete='restrict', store=True, + compute='_compute_tax_line_id') + tax_base_amount = fields.Monetary(string="Base Amount", store=True, + currency_field='company_currency_id') + tax_exigible = fields.Boolean(string='Appears in VAT report', default=True, + help="Technical field used to mark a tax line as exigible in the vat report or not (only exigible journal items" + " are displayed). By default all new journal items are directly exigible, but with the feature cash_basis" + " on taxes, some will become exigible only when the payment is recorded.") + tax_repartition_line_id = fields.Many2one(comodel_name='account.tax.repartition.line', + string="Originator Tax Repartition Line", ondelete='restrict', + help="Tax repartition line that caused the creation of this move line, if any") + tag_ids = fields.Many2many(string="Tags", comodel_name='account.account.tag', ondelete='restrict', + help="Tags assigned to this line by the tax creating it, if any. It determines its impact on financial reports.") + tax_audit = fields.Char(string="Tax Audit String", compute="_compute_tax_audit", store=True, + help="Computed field, listing the tax grids impacted by this line, and the amount it applies to each of them.") + + # ==== Reconciliation fields ==== + amount_residual = fields.Monetary(string='Residual Amount', store=True, + currency_field='company_currency_id', + compute='_amount_residual', + help="The residual amount on a journal item expressed in the company currency.") + amount_residual_currency = fields.Monetary(string='Residual Amount in Currency', store=True, + compute='_amount_residual', + help="The residual amount on a journal item expressed in its currency (possibly not the company currency).") + full_reconcile_id = fields.Many2one('account.full.reconcile', string="Matching Number", copy=False, index=True) + matched_debit_ids = fields.One2many('account.partial.reconcile', 'credit_move_id', String='Matched Debits', + help='Debit journal items that are matched with this journal item.') + matched_credit_ids = fields.One2many('account.partial.reconcile', 'debit_move_id', String='Matched Credits', + help='Credit journal items that are matched with this journal item.') + + # ==== Analytic fields ==== + analytic_line_ids = fields.One2many('account.analytic.line', 'move_id', string='Analytic lines') + analytic_account_id = fields.Many2one('account.analytic.account', string='Analytic Account', index=True) + analytic_tag_ids = fields.Many2many('account.analytic.tag', string='Analytic Tags') + + # ==== Onchange / display purpose fields ==== + recompute_tax_line = fields.Boolean(store=False, readonly=True, + help="Technical field used to know on which lines the taxes must be recomputed.") + display_type = fields.Selection([ + ('line_section', 'Section'), + ('line_note', 'Note'), + ], default=False, help="Technical field for UX purpose.") + is_rounding_line = fields.Boolean(help="Technical field used to retrieve the cash rounding line.") + exclude_from_invoice_tab = fields.Boolean(help="Technical field used to exclude some lines from the invoice_line_ids tab in the form view.") + always_set_currency_id = fields.Many2one('res.currency', string='Foreign Currency', + compute='_compute_always_set_currency_id', + help="Technical field used to compute the monetary field. As currency_id is not a required field, we need to use either the foreign currency, either the company one.") + + _sql_constraints = [ + ( + 'check_credit_debit', + 'CHECK(credit + debit>=0 AND credit * debit=0)', + 'Wrong credit or debit value in accounting entry !' + ), + ( + 'check_accountable_required_fields', + "CHECK(display_type IN ('line_section', 'line_note') OR account_id IS NOT NULL)", + "Missing required account on accountable invoice line." + ), + ( + 'check_non_accountable_fields_null', + "CHECK(display_type NOT IN ('line_section', 'line_note') OR (amount_currency = 0 AND debit = 0 AND credit = 0 AND account_id IS NULL))", + "Forbidden unit price, account and quantity on non-accountable invoice line" + ), + ( + 'check_amount_currency_balance_sign', + '''CHECK( + currency_id IS NULL + OR + company_currency_id IS NULL + OR + ( + (currency_id != company_currency_id) + AND + ( + (balance > 0 AND amount_currency > 0) + OR (balance <= 0 AND amount_currency <= 0) + OR (balance >= 0 AND amount_currency >= 0) + ) + ) + )''', + "The amount expressed in the secondary currency must be positive when account is debited and negative when account is credited." + ), + ] + + # ------------------------------------------------------------------------- + # HELPERS + # ------------------------------------------------------------------------- - @api.onchange('debit', 'credit', 'tax_ids', 'analytic_account_id', 'analytic_tag_ids') - def onchange_tax_ids_create_aml(self): + @api.model + def _get_default_tax_account(self, repartition_line): + tax = repartition_line.invoice_tax_id or repartition_line.refund_tax_id + if tax.tax_exigibility == 'on_payment': + account = tax.cash_basis_transition_account_id + else: + account = repartition_line.account_id + return account + + @api.multi + def _get_computed_name(self): + self.ensure_one() + + if not self.product_id: + return '' + + if self.partner_id.lang: + product = self.product_id.with_context(lang=self.partner_id.lang) + else: + product = self.product_id + + values = [] + if product.partner_ref: + values.append(product.partner_ref) + if self.journal_id.type == 'sale': + if product.description_sale: + values.append(product.description_sale) + elif self.journal_id.type == 'purchase': + if product.description_purchase: + values.append(product.description_purchase) + return '\n'.join(values) + + @api.multi + def _get_computed_price_unit(self): + self.ensure_one() + + if not self.product_id: + return self.price_unit + elif self.move_id.is_sale_document(include_receipts=True): + # Out invoice. + price_unit = self.product_id.lst_price + elif self.move_id.is_purchase_document(include_receipts=True): + # In invoice. + price_unit = self.product_id.standard_price + else: + return self.price_unit + + if self.product_uom_id != self.product_id.uom_id: + price_unit = self.product_id.uom_id._compute_price(price_unit, self.product_uom_id) + + company = self.move_id.company_id + if self.move_id.currency_id != company.currency_id: + price_unit = company.currency_id._convert( + price_unit, self.move_id.currency_id, company, self.move_id.date) + return price_unit + + @api.multi + def _get_computed_account(self): + self.ensure_one() + + if not self.product_id: + return + + fiscal_position = self.move_id.fiscal_position_id + accounts = self.product_id.product_tmpl_id.get_product_accounts(fiscal_pos=fiscal_position) + if self.move_id.is_sale_document(include_receipts=True): + # Out invoice. + return accounts['income'] + elif self.move_id.is_purchase_document(include_receipts=True): + # In invoice. + return accounts['expense'] + + @api.multi + def _get_computed_taxes(self): + self.ensure_one() + + if self.move_id.is_sale_document(include_receipts=True): + # Out invoice. + if self.product_id.taxes_id: + tax_ids = self.product_id.taxes_id.filtered(lambda tax: tax.company_id == self.move_id.company_id) + elif self.account_id.tax_ids: + tax_ids = self.account_id.tax_ids + else: + tax_ids = self.env['account.tax'] + if not tax_ids and not self.exclude_from_invoice_tab: + tax_ids = self.move_id.company_id.account_sale_tax_id + elif self.move_id.is_purchase_document(include_receipts=True): + # In invoice. + if self.product_id.supplier_taxes_id: + tax_ids = self.product_id.supplier_taxes_id.filtered(lambda tax: tax.company_id == self.move_id.company_id) + elif self.account_id.tax_ids: + tax_ids = self.account_id.tax_ids + else: + tax_ids = self.env['account.tax'] + if not tax_ids and not self.exclude_from_invoice_tab: + tax_ids = self.move_id.company_id.account_purchase_tax_id + else: + # Miscellaneous operation. + tax_ids = self.account_id.tax_ids + + if self.company_id: + tax_ids = tax_ids.filtered(lambda tax: tax.company_id == self.company_id) + + fiscal_position = self.move_id.fiscal_position_id + if tax_ids and fiscal_position: + return fiscal_position.map_tax(tax_ids, partner=self.partner_id) + else: + return tax_ids + + @api.multi + def _get_computed_uom(self): + self.ensure_one() + if self.product_id: + return self.product_id.uom_id + return False + + @api.multi + def _get_price_total_and_subtotal(self, price_unit=None, quantity=None, discount=None, currency=None, product=None, partner=None, taxes=None, move_type=None): + self.ensure_one() + return self._get_price_total_and_subtotal_model( + price_unit=price_unit or self.price_unit, + quantity=quantity or self.quantity, + discount=discount or self.discount, + currency=currency or self.currency_id, + product=product or self.product_id, + partner=partner or self.partner_id, + taxes=taxes or self.tax_ids, + move_type=move_type or self.move_id.type, + ) + + @api.model + def _get_price_total_and_subtotal_model(self, price_unit, quantity, discount, currency, product, partner, taxes, move_type): + ''' This method is used to compute 'price_total' & 'price_subtotal'. + + :param price_unit: The current price unit. + :param quantity: The current quantity. + :param discount: The current discount. + :param currency: The line's currency. + :param product: The line's product. + :param partner: The line's partner. + :param taxes: The applied taxes. + :param move_type: The type of the move. + :return: A dictionary containing 'price_subtotal' & 'price_total'. + ''' + res = {} + + # Compute 'price_subtotal'. + price_unit_wo_discount = price_unit * (1 - (discount / 100.0)) + subtotal = quantity * price_unit_wo_discount + + # Compute 'price_total'. + if taxes: + taxes_res = taxes._origin.compute_all(price_unit_wo_discount, + quantity=quantity, currency=currency, product=product, partner=partner, is_refund=move_type in ('out_refund', 'in_refund')) + res['price_subtotal'] = taxes_res['total_excluded'] + res['price_total'] = taxes_res['total_included'] + else: + res['price_total'] = res['price_subtotal'] = subtotal + return res + + @api.multi + def _get_fields_onchange_subtotal(self, price_subtotal=None, move_type=None, currency=None, company=None, date=None): + self.ensure_one() + return self._get_fields_onchange_subtotal_model( + price_subtotal=price_subtotal or self.price_subtotal, + move_type=move_type or self.move_id.type, + currency=currency or self.currency_id, + company=company or self.move_id.company_id, + date=date or self.move_id.date, + ) + + @api.model + def _get_fields_onchange_subtotal_model(self, price_subtotal, move_type, currency, company, date): + ''' This method is used to recompute the values of 'amount_currency', 'debit', 'credit' due to a change made + in some business fields (affecting the 'price_subtotal' field). + + :param price_subtotal: The untaxed amount. + :param move_type: The type of the move. + :param currency: The line's currency. + :param company: The move's company. + :param date: The move's date. + :return: A dictionary containing 'debit', 'credit', 'amount_currency'. + ''' + if move_type in self.move_id.get_outbound_types(): + sign = 1 + elif move_type in self.move_id.get_inbound_types(): + sign = -1 + else: + sign = 1 + price_subtotal *= sign + + if currency and currency != company.currency_id: + # Multi-currencies. + balance = currency._convert(price_subtotal, company.currency_id, company, date) + return { + 'amount_currency': price_subtotal, + 'debit': balance > 0.0 and balance or 0.0, + 'credit': balance < 0.0 and -balance or 0.0, + } + else: + # Single-currency. + return { + 'amount_currency': 0.0, + 'debit': price_subtotal > 0.0 and price_subtotal or 0.0, + 'credit': price_subtotal < 0.0 and -price_subtotal or 0.0, + } + + @api.multi + def _get_fields_onchange_balance(self, quantity=None, discount=None, balance=None, move_type=None, currency=None, taxes=None): + self.ensure_one() + return self._get_fields_onchange_balance_model( + quantity=quantity or self.quantity, + discount=discount or self.discount, + balance=balance or self.balance, + move_type=move_type or self.move_id.type, + currency=currency or self.currency_id, + taxes=taxes or self.tax_ids, + ) + + @api.model + def _get_fields_onchange_balance_model(self, quantity, discount, balance, move_type, currency, taxes): + ''' This method is used to recompute the values of 'quantity', 'discount', 'price_unit' due to a change made + in some accounting fields such as 'balance'. + + This method is a bit complex as we need to handle some special cases. + For example, setting a positive balance with a 100% discount. + + :param quantity: The current quantity. + :param discount: The current discount. + :param balance: The new balance. + :param move_type: The type of the move. + :param currency: The currency. + :param taxes: The applied taxes. + :return: A dictionary containing 'quantity', 'discount', 'price_unit'. + ''' + if move_type in self.move_id.get_outbound_types(): + sign = 1 + elif move_type in self.move_id.get_inbound_types(): + sign = -1 + else: + sign = 1 + balance *= sign + + taxes = taxes.flatten_taxes_hierarchy() + if taxes and any(tax.price_include for tax in taxes): + # Inverse taxes. E.g: + # + # Price Unit | Taxes | Originator Tax |Price Subtotal | Price Total + # ----------------------------------------------------------------------------------- + # 110 | 10% incl, 5% | | 100 | 115 + # 10 | | 10% incl | 10 | 10 + # 5 | | 5% | 5 | 5 + # + # When setting the balance to -200, the expected result is: + # + # Price Unit | Taxes | Originator Tax |Price Subtotal | Price Total + # ----------------------------------------------------------------------------------- + # 220 | 10% incl, 5% | | 200 | 230 + # 20 | | 10% incl | 20 | 20 + # 10 | | 5% | 10 | 10 + taxes_res = taxes._origin.compute_all(balance, currency=currency, handle_price_include=False) + for tax_res in taxes_res['taxes']: + tax = self.env['account.tax'].browse(tax_res['id']) + if tax.price_include: + balance += tax_res['amount'] + + discount_factor = 1 - (discount / 100.0) + if balance and discount_factor: + # discount != 100% + vals = { + 'quantity': quantity or 1.0, + 'price_unit': balance / discount_factor / (quantity or 1.0), + } + elif balance and not discount_factor: + # discount == 100% + vals = { + 'quantity': quantity or 1.0, + 'discount': 0.0, + 'price_unit': balance / (quantity or 1.0), + } + else: + vals = {} + return vals + + # ------------------------------------------------------------------------- + # ONCHANGE METHODS + # ------------------------------------------------------------------------- + + @api.onchange('amount_currency', 'currency_id', 'debit', 'credit', 'tax_ids', 'account_id', 'analytic_account_id', 'analytic_tag_ids') + def _onchange_mark_recompute_taxes(self): + ''' Recompute the dynamic onchange based on taxes. + If the edited line is a tax line, don't recompute anything as the user must be able to + set a custom value. + ''' for line in self: - line.recompute_tax_line = True + if not line.tax_repartition_line_id: + line.recompute_tax_line = True + + @api.onchange('product_id') + def _onchange_product_id(self): + for line in self: + if not line.product_id or line.display_type in ('line_section', 'line_note'): + continue + + line.name = line._get_computed_name() + line.account_id = line._get_computed_account() + line.tax_ids = line._get_computed_taxes() + line.product_uom_id = line._get_computed_uom() + line.price_unit = line._get_computed_price_unit() + + if len(self) == 1: + return {'domain': {'product_uom_id': [('category_id', '=', self.product_uom_id.category_id.id)]}} + + @api.onchange('product_uom_id') + def _onchange_uom_id(self): + ''' Recompute the 'price_unit' depending of the unit of measure. ''' + self.price_unit = self._get_computed_price_unit() + + @api.onchange('account_id') + def _onchange_account_id(self): + ''' Recompute 'tax_ids' based on 'account_id'. ''' + if not self.display_type in ('line_section', 'line_note'): + self.tax_ids = self._get_computed_taxes() + + @api.multi + def _onchange_balance(self): + for line in self: + if line.currency_id: + continue + if not line.move_id.is_invoice(include_receipts=True): + continue + line.update(line._get_fields_onchange_balance()) + line.update(line._get_price_total_and_subtotal()) @api.onchange('debit') def _onchange_debit(self): - self.ensure_one() - if self.debit != 0: - self.credit = 0 + if self.debit: + self.credit = 0.0 + self._onchange_balance() @api.onchange('credit') def _onchange_credit(self): - self.ensure_one() - if self.credit != 0: - self.debit = 0 + if self.credit: + self.debit = 0.0 + self._onchange_balance() - @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 - same way when we search on partner_id, with the addition of being optimal when having a query that will - search on partner_id and ref at the same time (which is the case when we open the bank reconciliation widget) - """ - cr = self._cr - cr.execute('DROP INDEX IF EXISTS account_move_line_partner_id_index') - cr.execute('SELECT indexname FROM pg_indexes WHERE indexname = %s', ('account_move_line_partner_id_ref_idx',)) - if not cr.fetchone(): - cr.execute('CREATE INDEX account_move_line_partner_id_ref_idx ON account_move_line (partner_id, ref)') + @api.onchange('amount_currency') + def _onchange_amount_currency(self): + for line in self: + if not line.currency_id: + continue + if not line.move_id.is_invoice(include_receipts=True): + continue + line.update(line._get_fields_onchange_balance( + balance=line.amount_currency, + )) + line.update(line._get_price_total_and_subtotal()) + + @api.onchange('quantity', 'discount', 'price_unit', 'tax_ids') + def _onchange_price_subtotal(self): + for line in self: + if not line.move_id.is_invoice(include_receipts=True): + continue + + line.update(line._get_price_total_and_subtotal()) + line.update(line._get_fields_onchange_subtotal()) + + @api.onchange('currency_id') + def _onchange_currency(self): + for line in self: + if line.move_id.is_invoice(include_receipts=True): + line._onchange_price_subtotal() + + # ------------------------------------------------------------------------- + # COMPUTE METHODS + # ------------------------------------------------------------------------- + + @api.depends('currency_id') + def _compute_always_set_currency_id(self): + for line in self: + line.always_set_currency_id = line.currency_id or line.company_currency_id + + @api.depends('debit', 'credit') + def _compute_balance(self): + for line in self: + line.balance = line.debit - line.credit @api.depends('debit', 'credit', 'amount_currency', 'currency_id', 'matched_debit_ids', 'matched_credit_ids', 'matched_debit_ids.amount', 'matched_credit_ids.amount', 'move_id.state') def _amount_residual(self): @@ -601,163 +2677,9 @@ class AccountMoveLine(models.Model): reconciled = True line.reconciled = reconciled - line.amount_residual = line.company_id.currency_id.round(amount * sign) + line.amount_residual = line.move_id.company_id.currency_id.round(amount * sign) line.amount_residual_currency = line.currency_id and line.currency_id.round(amount_residual_currency * sign) or 0.0 - @api.depends('debit', 'credit') - def _store_balance(self): - for line in self: - line.balance = line.debit - line.credit - - @api.model - def _get_currency(self): - currency = False - context = self._context or {} - if context.get('default_journal_id', False): - currency = self.env['account.journal'].browse(context['default_journal_id']).currency_id - return currency - - @api.depends('move_id') - def _compute_parent_state(self): - for record in self.filtered('move_id'): - record.parent_state = record.move_id.state - - name = fields.Char(string="Label") - move_name = fields.Char(string='Number', related='move_id.name', store=True, index=True) - quantity = fields.Float(digits=dp.get_precision('Product Unit of Measure'), - help="The optional quantity expressed by this line, eg: number of product sold. The quantity is not a legal requirement but is very useful for some reports.") - product_uom_id = fields.Many2one('uom.uom', string='Unit of Measure') - product_id = fields.Many2one('product.product', string='Product') - debit = fields.Monetary(default=0.0, currency_field='company_currency_id') - credit = fields.Monetary(default=0.0, currency_field='company_currency_id') - balance = fields.Monetary(compute='_store_balance', store=True, currency_field='company_currency_id', - help="Technical field holding the debit - credit in order to open meaningful graph views from reports") - amount_currency = fields.Monetary(default=0.0, help="The amount expressed in an optional other currency if it is a multi-currency entry.") - company_currency_id = fields.Many2one('res.currency', related='company_id.currency_id', string="Company Currency", readonly=True, - help='Utility field to express amount currency', store=True) - currency_id = fields.Many2one('res.currency', string='Currency', default=_get_currency, - help="The optional other currency if it is a multi-currency entry.") - amount_residual = fields.Monetary(compute='_amount_residual', string='Residual Amount', store=True, currency_field='company_currency_id', - help="The residual amount on a journal item expressed in the company currency.") - amount_residual_currency = fields.Monetary(compute='_amount_residual', string='Residual Amount in Currency', store=True, - help="The residual amount on a journal item expressed in its currency (possibly not the company currency).") - tax_base_amount = fields.Monetary(string="Base Amount", currency_field='company_currency_id') - account_id = fields.Many2one('account.account', string='Account', required=True, index=True, - ondelete="cascade", domain=[('deprecated', '=', False)], default=lambda self: self._context.get('account_id', False)) - move_id = fields.Many2one('account.move', string='Journal Entry', ondelete="cascade", - help="The move of this entry line.", index=True, required=True, auto_join=True) - narration = fields.Text(related='move_id.narration', string='Narration', readonly=False) - ref = fields.Char(related='move_id.ref', string='Reference', store=True, copy=False, index=True, readonly=False) - payment_id = fields.Many2one('account.payment', string="Originator Payment", help="Payment that created this entry", copy=False) - statement_line_id = fields.Many2one('account.bank.statement.line', index=True, string='Bank statement line reconciled with this entry', copy=False, readonly=True) - statement_id = fields.Many2one('account.bank.statement', related='statement_line_id.statement_id', string='Statement', store=True, - help="The bank statement used for bank reconciliation", index=True, copy=False) - reconciled = fields.Boolean(compute='_amount_residual', store=True) - reconcile_model_id = fields.Many2one('account.reconcile.model', string="Reconciliation Model", copy=False) - full_reconcile_id = fields.Many2one('account.full.reconcile', string="Matching Number", copy=False, index=True) - matched_debit_ids = fields.One2many('account.partial.reconcile', 'credit_move_id', String='Matched Debits', - help='Debit journal items that are matched with this journal item.') - matched_credit_ids = fields.One2many('account.partial.reconcile', 'debit_move_id', String='Matched Credits', - help='Credit journal items that are matched with this journal item.') - journal_id = fields.Many2one('account.journal', related='move_id.journal_id', string='Journal', readonly=False, - index=True, store=True, copy=False) # related is required - blocked = fields.Boolean(string='No Follow-up', default=False, - help="You can check this box to mark this journal item as a litigation with the associated partner") - date_maturity = fields.Date(string='Due date', index=True, required=True, copy=False, - help="This field is used for payable and receivable journal entries. You can put the limit date for the payment of this line.") - date = fields.Date(related='move_id.date', string='Date', index=True, store=True, copy=False, readonly=False) # related is required - analytic_line_ids = fields.One2many('account.analytic.line', 'move_id', string='Analytic lines', oldname="analytic_lines") - tax_ids = fields.Many2many('account.tax', string='Taxes', domain=['|', ('active', '=', False), ('active', '=', True)]) - tax_line_id = fields.Many2one('account.tax', string='Originator tax', ondelete='restrict', compute='_compute_tax_line_id', store=True) - tax_repartition_line_id = fields.Many2one(comodel_name='account.tax.repartition.line', string="Originator Tax Repartition Line", ondelete='restrict', help="Tax repartition line that caused the creation of this move line, if any") - analytic_account_id = fields.Many2one('account.analytic.account', string='Analytic Account', index=True) - analytic_tag_ids = fields.Many2many('account.analytic.tag', string='Analytic Tags') - company_id = fields.Many2one('res.company', related='account_id.company_id', string='Company', store=True, readonly=True) - - # TODO: put the invoice link and partner_id on the account_move - invoice_id = fields.Many2one('account.invoice', oldname="invoice") - partner_id = fields.Many2one('res.partner', string='Partner', ondelete='restrict') - tax_exigible = fields.Boolean(string='Appears in VAT report', default=True, - help="Technical field used to mark a tax line as exigible in the vat report or not (only exigible journal items are displayed). By default all new journal items are directly exigible, but with the feature cash_basis on taxes, some will become exigible only when the payment is recorded.") - parent_state = fields.Char(compute="_compute_parent_state", help="State of the parent account.move") - - recompute_tax_line = fields.Boolean(store=False, help="Technical field used to know if the tax_ids field has been modified in the UI.") - tax_line_grouping_key = fields.Char(store=False, string='Old Taxes', help="Technical field used to store the old values of fields used to compute tax lines (in account.move form view) between the moment the user changed it and the moment the ORM reflects that change in its one2many") - tag_ids = fields.Many2many(string="Tags", comodel_name='account.account.tag', ondelete='restrict', help="Tags assigned to this line by the tax creating it, if any. It determines its impact on financial reports.") - tax_audit = fields.Char(string="Tax Audit String", compute="_compute_tax_audit", store=True, help="Computed field, listing the tax grids impacted by this line, and the amount it applies to each of them.") - - _sql_constraints = [ - ('credit_debit1', 'CHECK (credit*debit=0)', 'Wrong credit or debit value in accounting entry! Credit or debit should be zero.'), - ('credit_debit2', 'CHECK (credit+debit>=0)', 'Wrong credit or debit value in accounting entry! Credit and debit should be positive.'), - ] - - @api.model - def default_get(self, fields): - rec = super(AccountMoveLine, self).default_get(fields) - if 'line_ids' not in self._context or "account_move_line_default_get" in self._context: - return rec - if {'debit', 'credit', 'partner_id', 'account_id'}.isdisjoint(fields): - return rec - # compute the default credit/debit of the next line in case of a manual entry - move = self.env['account.move'].with_context(account_move_line_default_get=True).new({'line_ids': self._context['line_ids']}) - balance = 0 - for line in move.line_ids: - balance += line.debit - line.credit - if balance < 0: - rec.update({'debit': -balance}) - if balance > 0: - rec.update({'credit': balance}) - - # Set the partner/account to the same value as the two last lines if they are equal - if len(move.line_ids) > 1: - last_lines = move.line_ids[-2:] - if last_lines[0].account_id == last_lines[1].account_id: - rec['account_id'] = last_lines[0].account_id.id - if last_lines[0].partner_id == last_lines[1].partner_id: - rec['partner_id'] = last_lines[0].partner_id.id - - return rec - - @api.multi - @api.constrains('tax_ids', 'tax_line_id') - def _check_tax_lock_date1(self): - for line in self: - if line.date <= (line.company_id.tax_lock_date or date.min): - raise ValidationError(_("The operation is refused as it would impact an already issued tax statement. " + - "Please change the journal entry date or the tax lock date set in the settings ({}) to proceed").format(line.company_id.tax_lock_date or date.min)) - - @api.multi - @api.constrains('credit', 'debit', 'date') - def _check_tax_lock_date2(self): - for line in self: - if (line.tax_ids or line.tax_line_id) and line.date <= (line.company_id.tax_lock_date or date.min): - raise ValidationError(_("The operation is refused as it would impact an already issued tax statement. " + - "Please change the journal entry date or the tax lock date set in the settings ({}) to proceed").format(line.company_id.tax_lock_date or date.min)) - - @api.multi - @api.constrains('currency_id', 'account_id') - def _check_currency(self): - for line in self: - account_currency = line.account_id.currency_id - if account_currency and account_currency != line.company_id.currency_id: - if not line.currency_id or line.currency_id != account_currency: - raise ValidationError(_('The selected account of your Journal Entry forces to provide a secondary currency. You should remove the secondary currency on the account.')) - - @api.multi - @api.constrains('currency_id', 'amount_currency') - def _check_currency_and_amount(self): - for line in self: - if (line.amount_currency and not line.currency_id): - raise ValidationError(_("You cannot create journal items with a secondary currency without filling both 'currency' and 'amount currency' fields.")) - - @api.multi - @api.constrains('amount_currency', 'debit', 'credit') - def _check_currency_amount(self): - for line in self: - if line.amount_currency: - if (line.amount_currency > 0.0 and line.credit > 0.0) or (line.amount_currency < 0.0 and line.debit > 0.0): - raise ValidationError(_('The amount expressed in the secondary currency must be positive when account is debited and negative when account is credited.')) - @api.depends('tax_repartition_line_id.invoice_tax_id', 'tax_repartition_line_id.refund_tax_id') def _compute_tax_line_id(self): """ tax_line_id is computed as the tax linked to the repartition line creating @@ -774,7 +2696,7 @@ class AccountMoveLine(models.Model): unaffected_earnings_type = self.env.ref("account.data_unaffected_earnings") record.is_unaffected_earnings_line = unaffected_earnings_type == record.account_id.user_type_id - @api.depends('tag_ids', 'debit', 'credit', 'journal_id', 'invoice_id') + @api.depends('tag_ids', 'debit', 'credit', 'journal_id') def _compute_tax_audit(self): separator = ' ' @@ -782,7 +2704,7 @@ class AccountMoveLine(models.Model): currency = record.company_id.currency_id audit_str = '' for tag in record.tag_ids: - tag_amount = (tag.tax_negate and -1 or 1) * (record.journal_id.type == 'sale' and -1 or 1) * (record.invoice_id.type in ('in_refund', 'out_refund') and -1 or 1) * record.balance + tag_amount = (tag.tax_negate and -1 or 1) * (record.move_id.is_outbound() and -1 or 1) * record.balance if tag.tax_report_line_ids: #Then, the tag comes from a report line, and hence has a + or - sign (also in its name) @@ -796,24 +2718,279 @@ class AccountMoveLine(models.Model): record.tax_audit = audit_str - @api.onchange('amount_currency', 'currency_id', 'account_id') - def _onchange_amount_currency(self): - '''Recompute the debit/credit based on amount_currency/currency_id and date. - However, date is a related field on account.move. Then, this onchange will not be triggered - by the form view by changing the date on the account.move. - To fix this problem, see _onchange_date method on account.move. - ''' + # ------------------------------------------------------------------------- + # CONSTRAINT METHODS + # ------------------------------------------------------------------------- + + @api.constrains('account_id') + def _check_constrains_account_id(self): for line in self: - company_currency_id = line.account_id.company_id.currency_id - amount = line.amount_currency - if line.currency_id and company_currency_id and line.currency_id != company_currency_id: - amount = line.currency_id._convert(amount, company_currency_id, line.company_id, line.date or fields.Date.today()) - line.debit = amount > 0 and amount or 0.0 - line.credit = amount < 0 and -amount or 0.0 + account = line.account_id + journal = line.journal_id - #################################################### - # Reconciliation methods - #################################################### + if account.deprecated: + raise UserError(_('The account %s (%s) is deprecated.') % (account.name, account.code)) + + control_type_failed = journal.type_control_ids and account.user_type_id not in journal.type_control_ids + control_account_failed = journal.account_control_ids and account not in journal.account_control_ids + if control_type_failed or control_account_failed: + raise UserError(_('You cannot use this general account in this journal, check the tab \'Entry Controls\' on the related journal.')) + + @api.constrains('tax_ids', 'tax_line_id') + def _check_tax_lock_date1(self): + for line in self: + if line.date <= (line.company_id.tax_lock_date or date.min): + raise ValidationError( + _("The operation is refused as it would impact an already issued tax statement. " + + "Please change the journal entry date or the tax lock date set in the settings ({}) to proceed").format( + line.company_id.tax_lock_date or date.min)) + + @api.constrains('credit', 'debit', 'date') + def _check_tax_lock_date2(self): + for line in self: + if (line.tax_ids or line.tax_line_id) and line.date <= (line.company_id.tax_lock_date or date.min): + raise ValidationError( + _("The operation is refused as it would impact an already issued tax statement. " + + "Please change the journal entry date or the tax lock date set in the settings ({}) to proceed").format( + line.company_id.tax_lock_date or date.min)) + + @api.multi + def _update_check(self): + """ Raise Warning to cause rollback if the move is posted, some entries are reconciled or the move is older than the lock date""" + move_ids = set() + for line in self: + err_msg = _('Move name (id): %s (%s)') % (line.move_id.name, str(line.move_id.id)) + if line.move_id.state != 'draft': + raise UserError(_('You cannot do this modification on a posted journal entry, you can just change some non legal fields. You must revert the journal entry to cancel it.\n%s.') % err_msg) + if line.reconciled and not (line.debit == 0 and line.credit == 0): + raise UserError(_('You cannot do this modification on a reconciled entry. You can just change some non legal fields or you must unreconcile first.\n%s.') % err_msg) + if line.move_id.id not in move_ids: + move_ids.add(line.move_id.id) + self.env['account.move'].browse(list(move_ids))._check_fiscalyear_lock_date() + return True + + # ------------------------------------------------------------------------- + # LOW-LEVEL METHODS + # ------------------------------------------------------------------------- + + @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 + same way when we search on partner_id, with the addition of being optimal when having a query that will + search on partner_id and ref at the same time (which is the case when we open the bank reconciliation widget) + """ + cr = self._cr + cr.execute('DROP INDEX IF EXISTS account_move_line_partner_id_index') + cr.execute('SELECT indexname FROM pg_indexes WHERE indexname = %s', ('account_move_line_partner_id_ref_idx',)) + if not cr.fetchone(): + cr.execute('CREATE INDEX account_move_line_partner_id_ref_idx ON account_move_line (partner_id, ref)') + + @api.model_create_multi + def create(self, vals_list): + # OVERRIDE + ACCOUNTING_FIELDS = ('debit', 'credit', 'amount_currency') + BUSINESS_FIELDS = ('price_unit', 'quantity', 'discount', 'tax_ids') + + for vals in vals_list: + move = self.env['account.move'].browse(vals['move_id']) + + if move.is_invoice(include_receipts=True): + currency = self.env['res.currency'].browse(vals.get('currency_id')) + partner = self.env['res.partner'].browse(vals.get('partner_id')) + taxes = self.resolve_2many_commands('tax_ids', vals.get('tax_ids', []), fields=['id']) + tax_ids = set(tax['id'] for tax in taxes) + taxes = self.env['account.tax'].browse(tax_ids) + + # Ensure consistency between accounting & business fields. + # As we can't express such synchronization as computed fields without cycling, we need to do it both + # in onchange and in create/write. So, if something changed in accounting [resp. business] fields, + # business [resp. accounting] fields are recomputed. + if any(vals.get(field) for field in ACCOUNTING_FIELDS): + if vals.get('currency_id'): + balance = vals.get('amount_currency', 0.0) + else: + balance = vals.get('debit', 0.0) - vals.get('credit', 0.0) + vals.update(self._get_fields_onchange_balance_model( + vals.get('quantity', 0.0), + vals.get('discount', 0.0), + balance, + move.type, + currency, + taxes + )) + vals.update(self._get_price_total_and_subtotal_model( + vals.get('price_unit', 0.0), + vals.get('quantity', 0.0), + vals.get('discount', 0.0), + currency, + self.env['product.product'].browse(vals.get('product_id')), + partner, + taxes, + move.type, + )) + elif any(vals.get(field) for field in BUSINESS_FIELDS): + vals.update(self._get_price_total_and_subtotal_model( + vals.get('price_unit', 0.0), + vals.get('quantity', 0.0), + vals.get('discount', 0.0), + currency, + self.env['product.product'].browse(vals.get('product_id')), + partner, + taxes, + move.type, + )) + vals.update(self._get_fields_onchange_subtotal_model( + vals['price_subtotal'], + move.type, + currency, + move.company_id, + move.date, + )) + + # Ensure consistency between taxes & tax exigibility fields. + if 'tax_exigible' in vals: + continue + if vals.get('tax_repartition_line_id'): + repartition_line = self.env['account.tax.repartition.line'].browse(vals['tax_repartition_line_id']) + tax = repartition_line.invoice_tax_id or repartition_line.refund_tax_id + vals['tax_exigible'] = tax.tax_exigibility == 'on_invoice' + elif vals.get('tax_ids'): + taxes = self.resolve_2many_commands('tax_ids', vals['tax_ids']) + vals['tax_exigible'] = not any([tax['tax_exigibility'] == 'on_payment' for tax in taxes]) + + lines = super(AccountMoveLine, self).create(vals_list) + + # Check the move is balanced debit = credit. + if self._context.get('check_move_validity', True): + lines.mapped('move_id')._check_move_consistency() + + return lines + + @api.multi + def write(self, vals): + # OVERRIDE + ACCOUNTING_FIELDS = ('debit', 'credit', 'amount_currency') + BUSINESS_FIELDS = ('price_unit', 'quantity', 'discount', 'tax_ids') + + if ('account_id' in vals) and self.env['account.account'].browse(vals['account_id']).deprecated: + raise UserError(_('You cannot use a deprecated account.')) + if any(key in vals for key in ('account_id', 'journal_id', 'date', 'move_id', 'debit', 'credit')): + self._update_check() + if not self._context.get('allow_amount_currency') and any( + key in vals for key in ('amount_currency', 'currency_id')): + # hackish workaround to write the amount_currency when assigning a payment to an invoice through the 'add' button + # this is needed to compute the correct amount_residual_currency and potentially create an exchange difference entry + self._update_check() + # when making a reconciliation on an existing liquidity journal item, mark the payment as reconciled + for record in self: + if 'statement_line_id' in vals and record.payment_id: + # In case of an internal transfer, there are 2 liquidity move lines to match with a bank statement + if all(line.statement_id for line in record.payment_id.move_line_ids.filtered( + lambda r: r.id != record.id and r.account_id.internal_type == 'liquidity')): + record.payment_id.state = 'reconciled' + + result = super(AccountMoveLine, self).write(vals) + + for line in self: + if not line.move_id.is_invoice(include_receipts=True): + continue + + # Ensure consistency between accounting & business fields. + # As we can't express such synchronization as computed fields without cycling, we need to do it both + # in onchange and in create/write. So, if something changed in accounting [resp. business] fields, + # business [resp. accounting] fields are recomputed. + if any(field in vals for field in ACCOUNTING_FIELDS): + price_subtotal = line.currency_id and line.amount_currency or line.debit - line.credit + to_write = line._get_fields_onchange_balance( + balance=price_subtotal, + ) + to_write.update(line._get_price_total_and_subtotal( + price_unit=to_write.get('price_unit', line.price_unit), + quantity=to_write.get('quantity', line.quantity), + discount=to_write.get('discount', line.discount), + )) + super(AccountMoveLine, line).write(to_write) + elif any(field in vals for field in BUSINESS_FIELDS): + to_write = self._get_price_total_and_subtotal() + to_write.update(line._get_fields_onchange_subtotal( + price_subtotal=to_write['price_subtotal'], + )) + super(AccountMoveLine, line).write(to_write) + + if self._context.get('check_move_validity', True) and any(key in vals for key in ('account_id', 'journal_id', 'date', 'move_id', 'debit', 'credit')): + self.env['account.move'].browse(self.mapped('move_id.id'))._check_move_consistency() + + return result + + @api.multi + def unlink(self): + self._update_check() + self._check_tax_lock_date2() + move_ids = set() + for line in self: + if line.move_id.id not in move_ids: + move_ids.add(line.move_id.id) + result = super(AccountMoveLine, self).unlink() + if self._context.get('check_move_validity', True) and move_ids: + self.env['account.move'].browse(list(move_ids))._check_move_consistency() + return result + + @api.model + def default_get(self, default_fields): + # OVERRIDE + values = super(AccountMoveLine, self).default_get(default_fields) + + if 'account_id' in default_fields \ + and (self._context.get('journal_id') or self._context.get('default_journal_id')) \ + and not values.get('account_id') \ + and self._context.get('default_type') in self.move_id.get_inbound_types(): + # Fill missing 'account_id'. + journal = self.env['account.journal'].browse(self._context.get('default_journal_id') or self._context['journal_id']) + values['account_id'] = journal.default_credit_account_id.id + elif 'account_id' in default_fields \ + and (self._context.get('journal_id') or self._context.get('default_journal_id')) \ + and not values.get('account_id') \ + and self._context.get('default_type') in self.move_id.get_outbound_types(): + # Fill missing 'account_id'. + journal = self.env['account.journal'].browse(self._context.get('default_journal_id') or self._context['journal_id']) + values['account_id'] = journal.default_debit_account_id.id + elif self._context.get('line_ids') and any(field_name in default_fields for field_name in ('debit', 'credit', 'account_id', 'partner_id')): + move = self.env['account.move'].new({'line_ids': self._context['line_ids']}) + + # Suggest default value for debit / credit to balance the journal entry. + balance = sum(line['debit'] - line['credit'] for line in move.line_ids) + if balance < 0.0: + values.update({'debit': -balance}) + if balance > 0.0: + values.update({'credit': balance}) + + # Suggest default value for 'partner_id'. + if 'partner_id' in default_fields and not values.get('partner_id'): + partners = move.line_ids[-2:].mapped('partner_id') + if len(partners) == 1: + values['partner_id'] = partners.id + + # Suggest default value for 'account_id'. + if 'account_id' in default_fields and not values.get('account_id'): + accounts = move.line_ids[-2:].mapped('account_id') + if len(accounts) == 1: + values['account_id'] = accounts.id + return values + + @api.multi + @api.depends('ref', 'move_id') + def name_get(self): + result = [] + for line in self: + if line.ref: + result.append((line.id, (line.move_id.name or '') + '(' + line.ref + ')')) + else: + result.append((line.id, line.move_id.name)) + return result + + # ------------------------------------------------------------------------- + # RECONCILIATION + # ------------------------------------------------------------------------- @api.multi def check_full_reconcile(self): @@ -894,7 +3071,7 @@ class AccountMoveLine(models.Model): exchange_move_id = False # Eventually create a journal entry to book the difference due to foreign currency's exchange rate that fluctuates if to_balance and any([not float_is_zero(residual, precision_rounding=digits_rounding_precision) for aml, residual in to_balance.values()]): - exchange_move = self.env['account.move'].create( + exchange_move = self.env['account.move'].with_context(default_type='entry').create( self.env['account.full.reconcile']._prepare_exchange_diff_move(move_date=maxdate, company=amls[0].company_id)) part_reconcile = self.env['account.partial.reconcile'] for aml_to_balance, total in to_balance.values(): @@ -985,13 +3162,12 @@ class AccountMoveLine(models.Model): for after_rec_dict in cash_basis_subjected: new_rec = part_rec.create(after_rec_dict) # if the pair belongs to move being reverted, do not create CABA entry - if cash_basis and not (new_rec.debit_move_id + new_rec.credit_move_id).mapped('move_id').mapped('reverse_entry_id'): + if cash_basis and not (new_rec.debit_move_id + new_rec.credit_move_id).mapped('move_id.reversed_entry_id'): new_rec.create_tax_cash_basis_entry(cash_basis_percentage_before_rec) self.recompute() return debit_moves+credit_moves - @api.multi def auto_reconcile_lines(self): # Create list of debit and list of credit move ordered by date-currency @@ -1032,7 +3208,7 @@ class AccountMoveLine(models.Model): # Empty self can happen if the user tries to reconcile entries which are already reconciled. # The calling method might have filtered out reconciled lines. if not self: - return True + return self._check_reconcile_validity() #reconcile everything that can be @@ -1058,7 +3234,6 @@ class AccountMoveLine(models.Model): def _create_writeoff(self, writeoff_vals): """ Create a writeoff move per journal for the account.move.lines in self. If debit/credit is not specified in vals, the writeoff amount will be computed as the sum of amount_residual of the given recordset. - :param writeoff_vals: list of dicts containing values suitable for account_move_line.create(). The data in vals will be processed to create bot writeoff acount.move.line and their enclosing account.move. """ @@ -1138,208 +3313,38 @@ class AccountMoveLine(models.Model): 'line_ids': [(0, 0, line) for line in writeoff_lines], }) writeoff_moves += writeoff_move - # writeoff_move.post() - line_to_reconcile += writeoff_move.line_ids.filtered(lambda r: r.account_id == self[0].account_id).sorted(key='id')[-1:] + + #post all the writeoff moves at once if writeoff_moves: writeoff_moves.post() + # Return the writeoff move.line which is to be reconciled return line_to_reconcile @api.multi def remove_move_reconcile(self): """ Undo a reconciliation """ - if not self: - return True - rec_move_ids = self.env['account.partial.reconcile'] - for account_move_line in self: - account_move_line.reconcile_model_id = False - for invoice in account_move_line.payment_id.invoice_ids: - if invoice.id == self.env.context.get('invoice_id') and account_move_line in invoice.payment_move_line_ids: - account_move_line.payment_id.write({'invoice_ids': [(3, invoice.id, None)]}) - rec_move_ids += account_move_line.matched_debit_ids - rec_move_ids += account_move_line.matched_credit_ids - if self.env.context.get('invoice_id'): - current_invoice = self.env['account.invoice'].browse(self.env.context['invoice_id']) - aml_to_keep = current_invoice.move_id.line_ids | current_invoice.move_id.line_ids.mapped('full_reconcile_id.exchange_move_id.line_ids') - rec_move_ids = rec_move_ids.filtered( - lambda r: (r.debit_move_id + r.credit_move_id) & aml_to_keep - ) - return rec_move_ids.unlink() - - def _apply_taxes(self, vals, amount): - tax_lines_vals = [] - # Get ids from triplets : https://www.odoo.com/documentation/10.0/reference/orm.html#odoo.models.Model.write - tax_ids = [tax['id'] for tax in self.resolve_2many_commands('tax_ids', vals['tax_ids']) if tax.get('id')] - # Since create() receives ids instead of recordset, let's just use the old-api bridge - taxes = self.env['account.tax'].browse(tax_ids) - currency = self.env['res.currency'].browse(vals.get('currency_id')) - partner = self.env['res.partner'].browse(vals.get('partner_id')) - ctx = dict(self._context) - ctx['round'] = ctx.get('round', True) - res = taxes.with_context(ctx).compute_all(amount, - currency, 1, vals.get('product_id'), partner) - # Adjust line amount if any tax is price_include - if abs(res['total_excluded']) < abs(amount): - if vals['debit'] != 0.0: vals['debit'] = res['total_excluded'] - if vals['credit'] != 0.0: vals['credit'] = -res['total_excluded'] - if vals.get('amount_currency'): - vals['amount_currency'] = self.env['res.currency'].browse(vals['currency_id']).round(vals['amount_currency'] * (res['total_excluded']/amount)) - # Create tax lines - for tax_vals in res['taxes']: - if tax_vals['amount']: - tax = self.env['account.tax'].browse([tax_vals['id']]) - account_id = (amount > 0 and tax_vals['account_id'] or tax_vals['refund_account_id']) - if not account_id: account_id = vals['account_id'] - temp = { - 'account_id': account_id, - 'name': vals['name'] + ' ' + tax_vals['name'], - 'tax_line_id': tax_vals['id'], - 'move_id': vals['move_id'], - 'partner_id': vals.get('partner_id'), - 'statement_id': vals.get('statement_id'), - 'debit': tax_vals['amount'] > 0 and tax_vals['amount'] or 0.0, - 'credit': tax_vals['amount'] < 0 and -tax_vals['amount'] or 0.0, - 'analytic_account_id': vals.get('analytic_account_id') if tax.analytic else False, - } - bank = self.env["account.bank.statement.line"].browse(vals.get('statement_line_id')).statement_id - if bank.currency_id != bank.company_id.currency_id: - ctx = {} - if 'date' in vals: - ctx['date'] = vals['date'] - elif 'date_maturity' in vals: - ctx['date'] = vals['date_maturity'] - temp['currency_id'] = bank.currency_id.id - temp['amount_currency'] = bank.company_id.currency_id.with_context(ctx).compute(tax_vals['amount'], bank.currency_id, round=True) - if vals.get('tax_exigible'): - temp['tax_exigible'] = True - temp['account_id'] = tax.cash_basis_account.id or account_id - tax_lines_vals.append(temp) - return tax_lines_vals - - #################################################### - # CRUD methods - #################################################### - - @api.model_create_multi - def create(self, vals_list): - """ :context's key `check_move_validity`: check data consistency after move line creation. Eg. set to false to disable verification that the move - debit-credit == 0 while creating the move lines composing the move. - """ - for vals in vals_list: - amount = vals.get('debit', 0.0) - vals.get('credit', 0.0) - move = self.env['account.move'].browse(vals['move_id']) - account = self.env['account.account'].browse(vals['account_id']) - if account.deprecated: - raise UserError(_('The account %s (%s) is deprecated.') %(account.name, account.code)) - journal = vals.get('journal_id') and self.env['account.journal'].browse(vals['journal_id']) or move.journal_id - vals['date_maturity'] = vals.get('date_maturity') or vals.get('date') or move.date - - ok = ( - (not journal.type_control_ids and not journal.account_control_ids) - or account.user_type_id in journal.type_control_ids - or account in journal.account_control_ids - ) - if not ok: - raise UserError(_('You cannot use this general account in this journal, check the tab \'Entry Controls\' on the related journal.')) - - # Automatically convert in the account's secondary currency if there is one and - # the provided values were not already multi-currency - if account.currency_id and 'amount_currency' not in vals and account.currency_id.id != account.company_id.currency_id.id: - vals['currency_id'] = account.currency_id.id - date = vals.get('date') or vals.get('date_maturity') or fields.Date.today() - vals['amount_currency'] = account.company_id.currency_id._convert(amount, account.currency_id, account.company_id, date) - - #Toggle the 'tax_exigible' field to False in case it is not yet given and the tax in 'tax_line_id' or one of - #the 'tax_ids' is a cash based tax. - taxes = False - if vals.get('tax_repartition_line_id'): - taxes = [{'tax_exigibility': self.env['account.tax.repartition.line'].browse(vals['tax_repartition_line_id']).tax_id.tax_exigibility}] - if vals.get('tax_ids'): - taxes = self.env['account.move.line'].resolve_2many_commands('tax_ids', vals['tax_ids']) - if taxes and any([tax['tax_exigibility'] == 'on_payment' for tax in taxes]) and not vals.get('tax_exigible'): - vals['tax_exigible'] = False - - lines = super(AccountMoveLine, self).create(vals_list) - - if self._context.get('check_move_validity', True): - lines.mapped('move_id')._post_validate() - - return lines - - @api.multi - def unlink(self): - self._update_check() - self._check_tax_lock_date2() - move_ids = set() - for line in self: - if line.move_id.id not in move_ids: - move_ids.add(line.move_id.id) - result = super(AccountMoveLine, self).unlink() - if self._context.get('check_move_validity', True) and move_ids: - self.env['account.move'].browse(list(move_ids))._post_validate() - return result + (self.mapped('matched_debit_ids') + self.mapped('matched_credit_ids')).unlink() @api.multi - def write(self, vals): - if ('account_id' in vals) and self.env['account.account'].browse(vals['account_id']).deprecated: - raise UserError(_('You cannot use a deprecated account.')) - if any(key in vals for key in ('account_id', 'journal_id', 'date', 'move_id', 'debit', 'credit')): - self._update_check() - if not self._context.get('allow_amount_currency') and any(key in vals for key in ('amount_currency', 'currency_id')): - #hackish workaround to write the amount_currency when assigning a payment to an invoice through the 'add' button - #this is needed to compute the correct amount_residual_currency and potentially create an exchange difference entry - self._update_check() - #when we set the expected payment date, log a note on the invoice_id related (if any) - if vals.get('expected_pay_date') and self.invoice_id: - str_expected_pay_date = vals['expected_pay_date'] - if isinstance(str_expected_pay_date, date): - str_expected_pay_date = fields.Date.to_string(str_expected_pay_date) - msg = _('New expected payment date: ') + str_expected_pay_date + '.\n' + vals.get('internal_note', '') - self.invoice_id.message_post(body=msg) #TODO: check it is an internal note (not a regular email)! - #when making a reconciliation on an existing liquidity journal item, mark the payment as reconciled - for record in self: - if 'statement_line_id' in vals and record.payment_id: - # In case of an internal transfer, there are 2 liquidity move lines to match with a bank statement - if all(line.statement_id for line in record.payment_id.move_line_ids.filtered(lambda r: r.id != record.id and r.account_id.internal_type=='liquidity')): - record.payment_id.state = 'reconciled' - - result = super(AccountMoveLine, self).write(vals) - if self._context.get('check_move_validity', True) and any(key in vals for key in ('account_id', 'journal_id', 'date', 'move_id', 'debit', 'credit')): - move_ids = set() - for line in self: - if line.move_id.id not in move_ids: - move_ids.add(line.move_id.id) - self.env['account.move'].browse(list(move_ids))._post_validate() - return result + def _copy_data_extend_business_fields(self, values): + ''' Hook allowing copying business fields under certain conditions. + E.g. The link to the sale order lines must be preserved in case of a refund. + ''' + self.ensure_one() @api.multi - def _update_check(self): - """ Raise Warning to cause rollback if the move is posted, some entries are reconciled or the move is older than the lock date""" - move_ids = set() - for line in self: - err_msg = _('Move name (id): %s (%s)') % (line.move_id.name, str(line.move_id.id)) - if line.reconciled and not (line.debit == 0 and line.credit == 0): - raise UserError(_('You cannot do this modification on a reconciled entry. You can just change some non legal fields or you must unreconcile first.\n%s.') % err_msg) - if line.move_id.id not in move_ids: - move_ids.add(line.move_id.id) - self.env['account.move'].browse(list(move_ids))._check_lock_date() - return True - - #################################################### - # Misc / utility methods - #################################################### + def copy_data(self, default=None): + res = super(AccountMoveLine, self).copy_data(default=default) + if self._context.get('include_business_fields'): + for line, values in zip(self, res): + line._copy_data_extend_business_fields(values) + return res - @api.multi - @api.depends('ref', 'move_id') - def name_get(self): - result = [] - for line in self: - if line.ref: - result.append((line.id, (line.move_id.name or '') + '(' + line.ref + ')')) - else: - result.append((line.id, line.move_id.name)) - return result + # ------------------------------------------------------------------------- + # MISC + # ------------------------------------------------------------------------- def _get_matched_percentage(self): """ This function returns a dictionary giving for each move_id of self, the percentage to consider as cash basis factor. @@ -1370,7 +3375,7 @@ class AccountMoveLine(models.Model): break if not all_same_currency: #we cannot rely on amount_currency fields as it is not present on all partial reconciliation - matched_percentage_per_move[line.move_id.id] = line.move_id.matched_percentage + matched_percentage_per_move[line.move_id.id] = line.move_id._get_cash_basis_matched_percentage() else: #we can rely on amount_currency fields, which allow us to post a tax cash basis move at the initial rate #to avoid currency rate difference issues. @@ -1380,22 +3385,6 @@ class AccountMoveLine(models.Model): matched_percentage_per_move[line.move_id.id] = total_reconciled_currency / total_amount_currency return matched_percentage_per_move - @api.model - def _compute_amount_fields(self, amount, src_currency, company_currency): - """ Helper function to compute value for fields debit/credit/amount_currency based on an amount and the currencies given in parameter""" - amount_currency = False - currency_id = False - date = self.env.context.get('date') or fields.Date.today() - company = self.env.context.get('company_id') - company = self.env['res.company'].browse(company) if company else self.env.company - if src_currency and src_currency != company_currency: - amount_currency = amount - amount = src_currency._convert(amount, company_currency, company, date) - currency_id = src_currency.id - debit = amount > 0 and amount or 0.0 - credit = amount < 0 and -amount or 0.0 - return debit, credit, amount_currency, currency_id - def _get_analytic_tag_ids(self): self.ensure_one() return self.analytic_tag_ids.filtered(lambda r: not r.active_analytic_distribution).ids @@ -1441,7 +3430,7 @@ class AccountMoveLine(models.Model): 'general_account_id': move_line.account_id.id, 'ref': move_line.ref, 'move_id': move_line.id, - 'user_id': move_line.invoice_id.user_id.id or self._uid, + 'user_id': move_line.move_id.invoice_user_id.id or self._uid, 'partner_id': move_line.partner_id.id, 'company_id': move_line.analytic_account_id.company_id.id or self.env.company.id, }) @@ -1467,7 +3456,7 @@ class AccountMoveLine(models.Model): 'general_account_id': self.account_id.id, 'ref': self.ref, 'move_id': self.id, - 'user_id': self.invoice_id.user_id.id or self._uid, + 'user_id': self.move_id.invoice_user_id.id or self._uid, 'company_id': distribution.account_id.company_id.id or self.env.company.id, } @@ -1531,6 +3520,9 @@ class AccountMoveLine(models.Model): where_clause_params = [] tables = '' if domain: + domain.append(('display_type', 'not in', ('line_section', 'line_note'))) + domain.append(('move_id.state', '!=', 'cancel')) + query = self._where_calc(domain) # Wrap the query with 'company_id IN (...)' to avoid bypassing company access rights. @@ -1539,6 +3531,19 @@ class AccountMoveLine(models.Model): tables, where_clause, where_clause_params = query.get_sql() return tables, where_clause, where_clause_params + # FIXME: Clarify me and change me in master + @api.multi + def action_duplicate(self): + self.ensure_one() + action = self.env.ref('account.action_move_journal_line').read()[0] + action['target'] = 'inline' + action['context'] = dict(self.env.context) + action['context']['form_view_initial_mode'] = 'edit' + action['context']['view_no_maturity'] = False + action['views'] = [(self.env.ref('account.view_move_form').id, 'form')] + action['res_id'] = self.copy().id + return action + @api.multi def open_reconcile_view(self): [action] = self.env.ref('account.action_account_moves_all').read() @@ -1551,7 +3556,7 @@ class AccountMoveLine(models.Model): return action @api.model - def _get_domain_for_edition_mode(self): + def _get_suspense_moves_domain(self): return [ ('move_id.to_check', '=', True), ('full_reconcile_id', '=', False), @@ -1603,7 +3608,6 @@ class AccountPartialReconcile(models.Model): new journal item will be made into the given `move` in the company `currency_exchange_journal_id`, and one of its journal items is matched with the other lines to balance the full reconciliation. - :param aml_to_fix: recordset of account.move.line (possible several but sharing the same currency) :param move: account.move @@ -1653,7 +3657,6 @@ class AccountPartialReconcile(models.Model): def _get_tax_cash_basis_base_account(self, line, tax): ''' Get the account of lines that will contain the base amount of taxes. - :param line: An account.move.line record :param tax: An account.tax record :return: An account record @@ -1711,7 +3714,6 @@ class AccountPartialReconcile(models.Model): 'account_id': line.tax_repartition_line_id.account_id.id, 'analytic_account_id': line.analytic_account_id.id, 'analytic_tag_ids': line.analytic_tag_ids.ids, - 'tax_line_id': line.tax_line_id.id, 'tax_exigible': True, 'amount_currency': line.amount_currency and line.currency_id.round(line.amount_currency * amount / line.balance) or 0.0, 'currency_id': line.currency_id.id, @@ -1792,9 +3794,9 @@ class AccountPartialReconcile(models.Model): #reverse the tax basis move created at the reconciliation time for move in self.env['account.move'].search([('tax_cash_basis_rec_id', 'in', self._ids)]): if move.date > (move.company_id.period_lock_date or date.min): - move.reverse_moves(date=move.date) + move._reverse_moves([{'ref': _('Reversal of %s') % move.name}], cancel=True) else: - move.reverse_moves() + move._reverse_moves([{'date': fields.Date.today(), 'ref': _('Reversal of %s') % move.name}], cancel=True) res = super(AccountPartialReconcile, self).unlink() if full_to_unlink: full_to_unlink.unlink() @@ -1824,7 +3826,10 @@ class AccountFullReconcile(models.Model): # (reversing will cause a nested attempt to drop the full reconciliation) to_reverse = rec.exchange_move_id rec.exchange_move_id = False - to_reverse.reverse_moves() + to_reverse._reverse_moves([{ + 'date': fields.Date.today(), + 'ref': _('Reversal of: %s') % to_reverse.name, + }], cancel=True) return super(AccountFullReconcile, self).unlink() @api.model |